From patchwork Tue Feb 4 12:53:14 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ludovic_Court=C3=A8s?= X-Patchwork-Id: 20089 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id 04C00168C9; Tue, 4 Feb 2020 12:54:11 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-2.9 required=5.0 tests=BAYES_00,MAILING_LIST_MULTI autolearn=unavailable autolearn_force=no version=3.4.2 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id CAF58168BE for ; Tue, 4 Feb 2020 12:54:10 +0000 (GMT) Received: from localhost ([::1]:57624 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iyxiU-00071K-8t for patchwork@mira.cbaines.net; Tue, 04 Feb 2020 07:54:10 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:54927) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iyxiP-00071E-RN for guix-patches@gnu.org; Tue, 04 Feb 2020 07:54:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iyxiO-0007so-Q2 for guix-patches@gnu.org; Tue, 04 Feb 2020 07:54:05 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:36786) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1iyxiM-0007b6-6o; Tue, 04 Feb 2020 07:54:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1iyxiM-0006Yu-65; Tue, 04 Feb 2020 07:54:02 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#39414] [PATCH core-updates 0/2] Clarify search path handling in commencement.scm Resent-From: Ludovic =?utf-8?q?Court=C3=A8s?= Original-Sender: "Debbugs-submit" Resent-CC: janneke@gnu.org, guix-patches@gnu.org Resent-Date: Tue, 04 Feb 2020 12:54:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 39414 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 39414@debbugs.gnu.org Cc: Ludovic =?utf-8?q?Court=C3=A8s?= , janneke@gnu.org X-Debbugs-Original-To: guix-patches@gnu.org X-Debbugs-Original-Xcc: janneke@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.158082080725182 (code B ref -1); Tue, 04 Feb 2020 12:54:02 +0000 Received: (at submit) by debbugs.gnu.org; 4 Feb 2020 12:53:27 +0000 Received: from localhost ([127.0.0.1]:42759 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1iyxhn-0006Y6-EF for submit@debbugs.gnu.org; Tue, 04 Feb 2020 07:53:27 -0500 Received: from lists.gnu.org ([209.51.188.17]:51450) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1iyxhl-0006Xy-3e for submit@debbugs.gnu.org; Tue, 04 Feb 2020 07:53:25 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:54082) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iyxhj-0006yc-TZ for guix-patches@gnu.org; Tue, 04 Feb 2020 07:53:24 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:57640) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1iyxhj-0003pL-PQ; Tue, 04 Feb 2020 07:53:23 -0500 Received: from [2001:660:6102:320:e120:2c8f:8909:cdfe] (port=50230 helo=gnu.org) by fencepost.gnu.org with esmtpsa (TLS1.2:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1iyxhj-0002T8-Cw; Tue, 04 Feb 2020 07:53:23 -0500 From: Ludovic =?utf-8?q?Court=C3=A8s?= Date: Tue, 4 Feb 2020 13:53:14 +0100 Message-Id: <20200204125314.27494-1-ludo@gnu.org> X-Mailer: git-send-email 2.25.0 MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches 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(-)