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
prev 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).