unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Liliana Marie Prikler <liliana.prikler@gmail.com>
To: Greg Hogan <code@greghogan.com>
Cc: guix-devel <guix-devel@gnu.org>,
	"Efraim Flashner" <efraim@flashner.co.il>,
	"Ludovic Courtès" <ludo@gnu.org>
Subject: Re: Creating a C/C++ team?
Date: Fri, 06 Dec 2024 20:45:11 +0100	[thread overview]
Message-ID: <f982aa9717f0491887fd4a59089d471ea3cc84f1.camel@gmail.com> (raw)
In-Reply-To: <CA+3U0Zkt3zLggyu=bUKDdr_OjK-NfrZoSvb_HzG4+J+dB8mBYQ@mail.gmail.com>

Am Donnerstag, dem 05.12.2024 um 18:35 -0500 schrieb Greg Hogan:
> v2 below. Removed compiler (llvm[-meta].scm) scope and removed
> "compilers" from description.
> 
> +1 to an LLVM team or llvm[-meta].scm in the core-packages scope.
> 
> I think cmake-build-system belongs in the C++ team scope. Our use of
> cmake is rather simple, even with the enhancements collected in
> #70031 (which began with a desire to parallelize tests). We should
> keep cmake up-to-date without bothering core-packages.
I think we can do smaller topic-branches, regardless of which team
ought to do the review work.  There could even be multiple teams on a
particular series, to push it more quickly.

> ninja.scm and valgrind.scm are essentially single package modules.
> The suggested build-tools.scm, check.scm, and debug.scm are a mix.
> For example, check.scm includes python-pytest among other python
> packages. Not sure why these are not in python-check.scm unless there
> would be circular dependencies. Given the success of the teams
> workflow, should we look to divide these mixed modules by team rather
> than grouping by theme?
In my opinion, we shouldn't be that exclusionary w.r.t. non-C
languages.  Tools being written in Python historically hasn't stopped
C++ users from adopting them – see Meson or Conan for some popular
examples.

> $ ./etc/teams.scm show c++
> id: c++
> name: C/C++ team
> description: C and C++ libraries and tools and the CMake build
> system.
> + Note that updates to fundamental packages are the competency of the
> + core-packages team.
I would just shorten that to "C/C++ libraries and tools", with the
CMake build system being one of said tools :)
Blasting notes into the team description is imho not a good idea.
> scope:
> + gnu/build-system/cmake.scm
> + gnu/build/cmake-build-system.scm
> + gnu/packages/c.scm
> + gnu/packages/cmake.scm
> + gnu/packages/cpp.scm
> + gnu/packages/ninja.scm
> + gnu/packages/valgrind.scm
Scope looks good enough, even while bikeshedding.

Cheers


      parent reply	other threads:[~2024-12-06 19:45 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-25 18:27 Creating a C/C++ team? Greg Hogan
2024-11-28  8:26 ` Ludovic Courtès
2024-11-29  9:36 ` Liliana Marie Prikler
2024-11-30 15:19   ` Ludovic Courtès
2024-12-01 10:02   ` Efraim Flashner
2024-12-05 23:35   ` Greg Hogan
2024-12-06  8:27     ` indieterminacy
2024-12-06  9:39       ` Andreas Enge
2024-12-06 19:45     ` Liliana Marie Prikler [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f982aa9717f0491887fd4a59089d471ea3cc84f1.camel@gmail.com \
    --to=liliana.prikler@gmail.com \
    --cc=code@greghogan.com \
    --cc=efraim@flashner.co.il \
    --cc=guix-devel@gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).