GNU bug report logs - #39414
[PATCH core-updates 0/2] Clarify search path handling in commencement.scm

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#39414: closed ([PATCH core-updates 0/2] Clarify search path
 handling in commencement.scm)
Date: Thu, 06 Feb 2020 17:49:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Thu, 06 Feb 2020 18:48:11 +0100
with message-id <87y2tf6204.fsf <at> gnu.org>
and subject line Re: [bug#39414] [PATCH 2/2] gnu: commencement: Rationalize search path handling.
has caused the debbugs.gnu.org bug report #39414,
regarding [PATCH core-updates 0/2] Clarify search path handling in commencement.scm
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
39414: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=39414
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Ludovic Courtès <ludo <at> gnu.org>
To: guix-patches <at> gnu.org
Cc: Ludovic Courtès <ludo <at> gnu.org>
Subject: [PATCH core-updates 0/2] Clarify search path handling in
 commencement.scm
Date: Tue,  4 Feb 2020 13:53:14 +0100

Hello Guix!

The patch below for current ‘core-updates’ is an attempt to clarify
search path handling in commencement.scm by:

  1. Having ‘native-search-paths’ fields only for compilers—e.g.,
     “C_INCLUDE_PATH” belongs to the compiler, not to libc.

  2. Avoiding phases that manually fiddle with search path
     environment variables: normally, this is handled automatically
     by the ‘set-paths’ phase based on the declared search paths,
     so manual fiddling should be a last resort and it should be
     well commented so we remember why it’s there.

This is an attempt to reduce complexity and keep things declarative
as much as possible.  I’ve tested it on top of
46312064de8ae0cca531fddbc4b5ec8421e5d866 and “guix build coreutils”
succeeds.

There’s another pattern that I found hard to follow that has to do
with the reuse of build phases.

For example, there’s a build phase named ‘setenv’ (perhaps we should
find a more descriptive name :-)) in the various GCCs that is reused
or replaced; when looking at a specific package, it’s difficult to
see which phases it really runs because this particular phase is
inherited and modified on several layers.  If I can make time for it,
I’ll see if I can come up with a proposal to clarify this, but at any
rate, it’s probably something to keep in mind for future changes.

Thoughts?  (I’m particularly interested in your feedback, janneke!)

Ludo’.

Ludovic Courtès (2):
  gnu: commencement: Avoid hard-coded GCC version numbers.
  gnu: commencement: Rationalize search path handling.

 gnu/packages/commencement.scm | 201 ++++++++--------------------------
 1 file changed, 48 insertions(+), 153 deletions(-)

-- 
2.25.0

[Message part 3 (message/rfc822, inline)]

From: Ludovic Courtès <ludo <at> gnu.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: 39414-done <at> debbugs.gnu.org
Subject: Re: [bug#39414] [PATCH 2/2] gnu: commencement: Rationalize search
 path handling.
Date: Thu, 06 Feb 2020 18:48:11 +0100

Pushed as 558b0bbe291b2f2cd38b7f4eadc827e2ed102c54!

Ludo'.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.