Received: (at 32661) by debbugs.gnu.org; 10 Sep 2018 08:39:00 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Mon Sep 10 04:39:00 2018 Received: from localhost ([127.0.0.1]:51735 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1fzHik-0006Ax-MQ for submit <at> debbugs.gnu.org; Mon, 10 Sep 2018 04:39:00 -0400 Received: from mail-it0-f47.google.com ([209.85.214.47]:52219) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <boskovits@HIDDEN>) id 1fzHii-0006Af-Ez for 32661 <at> debbugs.gnu.org; Mon, 10 Sep 2018 04:38:56 -0400 Received: by mail-it0-f47.google.com with SMTP id e14-v6so28472621itf.1 for <32661 <at> debbugs.gnu.org>; Mon, 10 Sep 2018 01:38:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=sRTJazRprDFr30qVf0LFq9tqV5Q/ecrl/6bKmNu0gEU=; b=XMgUkMMyH+WGNcryzB//8SjXJBRzJaZ2k1qGxqZfu8gJjAA0OXChMvC9IQM6LjULT4 LuJHfMTTMa9utlYRmSfIn3IsRWBzU6gIntZ39qqYlK5Ey5IgoN4jAPByfszZACuDRZrJ djCc9hMrU1JNM8zVLkx75FtFrRpJMEeMeSCW9XdtYC2oPu5BUiHmx0QTVKcQZ9oG6kVW SXJe7gX4/c2YRVoffxZ05Qr9vjW43ID69WSLKFNRgQx/HW80y7Kpfyn+fEzAVS+4i9Nf oYn0egjRDHRTWWp1Pv2Ia6BKFXPl91FK8pWNeyqK0drYy1SQJWBgNb9o1x2dOIpBPi3D j8Wg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=sRTJazRprDFr30qVf0LFq9tqV5Q/ecrl/6bKmNu0gEU=; b=AQQqEPaYifEPDTLXiYrShTM+iqTYNtl6tToDa1VDh8ePceYnBnKLS9iIGD+eCtX4r+ Ah1gLsHn0RMh6eDtIi/dSzaHjYtHt9Z3X50FquH/bKHErY6O/RT1sH6FkHNbxlxCGF24 ZQAcMwYyqBM7daP5Rzr9KOf+k8ruBMgGJk9PsSBHXwbmpf221U4PLd0ecRNM+Dj66VVq sbtkkFCy6nElfyc5aCRYxyDSjs9E5o6PtT7J5hWUhSVzioabF/E0IV0MzTZO1lbTbK+S a1HC73eIZtFhYMMUvMeq+DT73+xTYt8gm0Hvbi9JzvoeIP/4cPrYlPMxTVFee4XbyGSO lNUA== X-Gm-Message-State: APzg51AMXjIuT52q5bkoNjx7EWg7UFPPJ3ev2NW/o/1WFWJnHjRMT9R5 Xh82LLyyPuqT7PJKO1dyWa0JHV1+COgKgeattwYm X-Google-Smtp-Source: ANB0Vdby0UZvEy0tiOK/eBW8X70oNBX5dAvbdZwT32py6KjKjXcUinNYNZ33Egu2nDNzXRMnzUD7LjrlOEKttchPdOw= X-Received: by 2002:a02:5c46:: with SMTP id q67-v6mr17381100jab.85.1536568730852; Mon, 10 Sep 2018 01:38:50 -0700 (PDT) MIME-Version: 1.0 References: <877ejxm0z9.fsf@HIDDEN> In-Reply-To: <877ejxm0z9.fsf@HIDDEN> From: =?UTF-8?Q?G=C3=A1bor_Boskovits?= <boskovits@HIDDEN> Date: Mon, 10 Sep 2018 10:38:39 +0200 Message-ID: <CAE4v=phrpqog=3nQfXbwbrZjCFU3Acst_+k4aBRHNfPgE4NzBA@HIDDEN> Subject: Re: bug#32661: modify-phases: detect invalid phase addition To: Ricardo Wurmus <rekado@HIDDEN> Content-Type: multipart/alternative; boundary="0000000000006022340575804928" X-Spam-Score: 1.2 (+) X-Spam-Report: Spam detection software, running on the system "debbugs.gnu.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: Ricardo Wurmus ezt írta (időpont: 2018. szept. 7., P 23:51): > It is possible to add a build phase to a non-existing phase without an > error. The new build phase will simply not be executed. > > This is problematic when an inherited package augments the original > package’s phases and the original package is changed. Imagine that the > parent package’s “pre-configure” phase is removed: > > ((#:phases original-phases) > `(modify-phases ,original-phases > (add-after 'pre-configure 'patch-this > (lambda _ …)))) > > This would cause the “patch-this” phase in the child package to be > skipped without warning. > > This kind of thing can be especially confusing in the bootstrap, which > makes heavy use of inheritance. > It would be nice to fix this. Also I have noticed that it is easier to make mistakes when inheritance is involved, but these tend to only break derived packages. I am not aware of a facility to make it easier to build all derived packages. Am I missing something? I guess this would be a useful tool, but not sure about implementation. [...] Content analysis details: (1.2 points, 10.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- 1.2 MIME_BOUND_DIGITS_15 Spam tool pattern in MIME boundary -0.0 SPF_PASS SPF: sender matches SPF record -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/, no trust [209.85.214.47 listed in list.dnswl.org] 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider (boskovits[at]gmail.com) 0.0 HTML_MESSAGE BODY: HTML included in message 0.0 T_DKIM_INVALID DKIM-Signature header exists but is not valid X-Debbugs-Envelope-To: 32661 Cc: 32661 <at> debbugs.gnu.org X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: 0.2 (/) --0000000000006022340575804928 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Ricardo Wurmus <rekado@HIDDEN> ezt =C3=ADrta (id=C5=91pont: 2018. szep= t. 7., P 23:51): > It is possible to add a build phase to a non-existing phase without an > error. The new build phase will simply not be executed. > > This is problematic when an inherited package augments the original > package=E2=80=99s phases and the original package is changed. Imagine th= at the > parent package=E2=80=99s =E2=80=9Cpre-configure=E2=80=9D phase is removed= : > > ((#:phases original-phases) > `(modify-phases ,original-phases > (add-after 'pre-configure 'patch-this > (lambda _ =E2=80=A6)))) > > This would cause the =E2=80=9Cpatch-this=E2=80=9D phase in the child pack= age to be > skipped without warning. > > This kind of thing can be especially confusing in the bootstrap, which > makes heavy use of inheritance. > It would be nice to fix this. Also I have noticed that it is easier to make mistakes when inheritance is involved, but these tend to only break derived packages. I am not aware of a facility to make it easier to build all derived packages. Am I missing something? I guess this would be a useful tool, but not sure about implementation. > > -- > Ricardo > > > > > --0000000000006022340575804928 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr">= Ricardo Wurmus <<a href=3D"mailto:rekado@HIDDEN">rekado@HIDDEN= </a>> ezt =C3=ADrta (id=C5=91pont: 2018. szept. 7., P 23:51):<br></div><= blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px= #ccc solid;padding-left:1ex">It is possible to add a build phase to a non-= existing phase without an<br> error.=C2=A0 The new build phase will simply not be executed.<br> <br> This is problematic when an inherited package augments the original<br> package=E2=80=99s phases and the original package is changed.=C2=A0 Imagine= that the<br> parent package=E2=80=99s =E2=80=9Cpre-configure=E2=80=9D phase is removed:<= br> <br> ((#:phases original-phases)<br> `(modify-phases ,original-phases<br> =C2=A0 (add-after 'pre-configure 'patch-this<br> =C2=A0 =C2=A0 (lambda _ =E2=80=A6))))<br> <br> This would cause the =E2=80=9Cpatch-this=E2=80=9D phase in the child packag= e to be<br> skipped without warning.<br> <br> This kind of thing can be especially confusing in the bootstrap, which<br> makes heavy use of inheritance.<br></blockquote></div></div><div dir=3D"aut= o">It would be nice to fix this. Also I have noticed that it is easier to m= ake mistakes when inheritance is involved, but these tend to only break der= ived packages. I am not aware of a facility to make it easier to build all = derived packages. Am I missing something? I guess this would be a useful to= ol, but not sure about implementation.</div><div dir=3D"auto"><div class=3D= "gmail_quote"><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;= border-left:1px #ccc solid;padding-left:1ex"> <br> --<br> Ricardo<br> <br> <br> <br> <br> </blockquote></div></div></div> --0000000000006022340575804928--
bug-guix@HIDDEN
:bug#32661
; Package guix
.
Full text available.Received: (at submit) by debbugs.gnu.org; 7 Sep 2018 21:50:38 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Fri Sep 07 17:50:38 2018 Received: from localhost ([127.0.0.1]:49902 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1fyOeE-0002Br-0N for submit <at> debbugs.gnu.org; Fri, 07 Sep 2018 17:50:38 -0400 Received: from eggs.gnu.org ([208.118.235.92]:39477) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <rekado@HIDDEN>) id 1fyOeB-0002Bd-TN for submit <at> debbugs.gnu.org; Fri, 07 Sep 2018 17:50:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from <rekado@HIDDEN>) id 1fyOe5-0004X2-O3 for submit <at> debbugs.gnu.org; Fri, 07 Sep 2018 17:50:30 -0400 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on eggs.gnu.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,T_DKIM_INVALID autolearn=disabled version=3.3.2 Received: from lists.gnu.org ([2001:4830:134:3::11]:41818) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from <rekado@HIDDEN>) id 1fyOe5-0004WA-Eu for submit <at> debbugs.gnu.org; Fri, 07 Sep 2018 17:50:29 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60447) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from <rekado@HIDDEN>) id 1fyOe4-0008DU-Lf for bug-guix@HIDDEN; Fri, 07 Sep 2018 17:50:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from <rekado@HIDDEN>) id 1fyOe1-0004Oj-K0 for bug-guix@HIDDEN; Fri, 07 Sep 2018 17:50:28 -0400 Received: from sender-of-o51.zoho.com ([135.84.80.216]:21061) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from <rekado@HIDDEN>) id 1fyOe1-0004LA-93 for bug-guix@HIDDEN; Fri, 07 Sep 2018 17:50:25 -0400 ARC-Seal: i=1; a=rsa-sha256; t=1536357007; cv=none; d=zoho.com; s=zohoarc; b=S2sW7ZMVWDq+qohzaqOXvy4HmAQbIhCocSfIOpLOkSgfs/WQs08N+ULeMwQlUav7whPQZMo7coW0MY0qAOk6Gk/VHj1fLJ1Glpei7busARbEn7apRSdLnMu1Uwp4Od3RuyvvUuE3HGJwwWjQywqiisD8PDWXx2XPIwadeZn1vSQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1536357007; h=Content-Type:Content-Transfer-Encoding:Date:From:MIME-Version:Message-ID:Subject:To:ARC-Authentication-Results; bh=S1OnV5JiMxZpafJBAhW6Hdv5io9Pk2h9ZkrJSr6E9yM=; b=Tkw2IEWURyAT2M5VnTbRMiBDO9lUNzerMW4QROUUOW1/Xe4NuEteMzVkhgRyC1tJ+j22SYTMaBqxQIbrfEK0hHDfUMhZ302Z0ttRLoaWQoOIw8/I9GcsSItDl3Iv5BDw2XcCnj2005vSNqYf+G4Uqa5FmlDa1m5DWnHU8iaq7DY= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass header.i=elephly.net; spf=pass smtp.mailfrom=rekado@HIDDEN; dmarc=pass header.from=<rekado@HIDDEN> header.from=<rekado@HIDDEN> DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1536357007; s=zoho; d=elephly.net; i=rekado@HIDDEN; h=From:To:Subject:Date:Message-ID:MIME-Version:Content-Type:Content-Transfer-Encoding; l=727; bh=S1OnV5JiMxZpafJBAhW6Hdv5io9Pk2h9ZkrJSr6E9yM=; b=Gzk7H9XXQl2ah1mnDAuqKryCtpjb/Gu6ts28ZKbqoVYpgyzrScIeSb7bMVft3T0x +67BConPSoNxZD9jbF1ioAAMmrLwShMm46+lTovOP4NbuZ3OxwrfzTMPythuYKfVDoe cGXmRUOkbW4oUMr2Cbxl+OAmqpohmlyjxPOKu7jU= Received: from localhost (port-92-200-2-74.dynamic.qsc.de [92.200.2.74]) by mx.zohomail.com with SMTPS id 1536357005599175.68790066938016; Fri, 7 Sep 2018 14:50:05 -0700 (PDT) User-agent: mu4e 1.0; emacs 26.1 From: Ricardo Wurmus <rekado@HIDDEN> To: bug-guix@HIDDEN Subject: modify-phases: detect invalid phase addition X-URL: https://elephly.net X-PGP-Key: https://elephly.net/rekado.pubkey X-PGP-Fingerprint: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC Date: Fri, 07 Sep 2018 23:50:02 +0200 Message-ID: <877ejxm0z9.fsf@HIDDEN> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-ZohoMailClient: External X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Received-From: 2001:4830:134:3::11 X-Spam-Score: -4.0 (----) X-Debbugs-Envelope-To: submit X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -5.0 (-----) It is possible to add a build phase to a non-existing phase without an error. The new build phase will simply not be executed. This is problematic when an inherited package augments the original package=E2=80=99s phases and the original package is changed. Imagine that= the parent package=E2=80=99s =E2=80=9Cpre-configure=E2=80=9D phase is removed: ((#:phases original-phases) `(modify-phases ,original-phases (add-after 'pre-configure 'patch-this (lambda _ =E2=80=A6)))) This would cause the =E2=80=9Cpatch-this=E2=80=9D phase in the child packag= e to be skipped without warning. This kind of thing can be especially confusing in the bootstrap, which makes heavy use of inheritance. -- Ricardo
Ricardo Wurmus <rekado@HIDDEN>
:bug-guix@HIDDEN
.
Full text available.bug-guix@HIDDEN
:bug#32661
; Package guix
.
Full text available.
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997 nCipher Corporation Ltd,
1994-97 Ian Jackson.