From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [Patch] Remove unused lambda arguments and prefer separate phases over augmented phases. Date: Mon, 31 Mar 2014 14:06:55 +0200 Message-ID: <87lhvqh8rk.fsf@gnu.org> References: <87ob0n6z3p.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:43805) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WUazX-0006bu-OP for guix-devel@gnu.org; Mon, 31 Mar 2014 08:07:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WUazS-0000lz-33 for guix-devel@gnu.org; Mon, 31 Mar 2014 08:07:03 -0400 Received: from hera.aquilenet.fr ([2a01:474::1]:53842) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WUazR-0000lc-Rq for guix-devel@gnu.org; Mon, 31 Mar 2014 08:06:57 -0400 In-Reply-To: <87ob0n6z3p.fsf@gmail.com> (Eric Bavier's message of "Sun, 30 Mar 2014 18:33:30 -0500") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Eric Bavier Cc: guix-devel@gnu.org Eric Bavier skribis: > My previous patch raised the question about whether it's wise to use > something like:: > > #:phases > (alist-replace > 'foo > (lambda* (#:key #:allow-other-keys #:rest args) > (let ((foo (assoc-ref 'foo %standard-phases))) > (frobnicate) > (apply foo args)))) > > in recipe arguments, or to just add another phase before the foo phase. > > There was some concensus that adding another phase before or after was > easier to understand (and less verbose), so I went ahead and tried to > reformulate the package recipes that did such things. Along the way I > "cleaned up" some other things, mostly in the package arguments. > > I tried not to change anything semantically in the packate definitions, > but I did verify that all packages build with these changes. Nice, thanks for the taking the time to do that. Mark and Andreas were fine with the idea, so OK to push, given that you=E2=80=99ve already tested it. > From d92b819c12b873fc3b99ec6771049a6bc4badbac Mon Sep 17 00:00:00 2001 > From: Eric Bavier > Date: Sun, 30 Mar 2014 18:09:10 -0500 > Subject: [PATCH] gnu: Remove unused lambda arguments and prefer separate > phases over augmented phases. > > * gnu/packages/compression.scm (sharutils) [argumets]: Remove unused =E2=80=9Carguments=E2=80=9D > * gnu/packages/gl.scm (mesa) [arguments]: Same > * gnu/packages/fltk.scm [arguments]: Same. Substitute const check > phase with #:tests? #f. Add pre-configure phase. > * gnu/packages/ghostscript.scm (ghostscript) [arguments]: Put makefile > patches in separate phase. Put so steps oi separate phases. Typo. > * gnu/packages/gnupg.scm (gnupg) [arguments]: Put patch in > pre-configure phase. > (pius) [arguments]: Delete const #t configure and build phases. > (signing-party) [arguments]: Factor patches into post-unpack phase. > (paperkey) [arguments]: Remove unused lambda args. Factor out patches > into pre-check phase. This (and a couple of others below) should be indented, like those above. Thanks! Ludo=E2=80=99.