From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp12.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms5.migadu.com with LMTPS id YPq1Ep4cvGIwaQAAbAwnHQ (envelope-from ) for ; Wed, 29 Jun 2022 11:34:22 +0200 Received: from aspmx1.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp12.migadu.com with LMTPS id OI1uEp4cvGIDgQEAauVa8A (envelope-from ) for ; Wed, 29 Jun 2022 11:34:22 +0200 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id EA41542C53 for ; Wed, 29 Jun 2022 11:34:21 +0200 (CEST) Received: from localhost ([::1]:59206 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1o6U5U-00063e-R2 for larch@yhetil.org; Wed, 29 Jun 2022 05:34:20 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42882) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1o6U5C-0005zu-Ns for bug-guix@gnu.org; Wed, 29 Jun 2022 05:34:06 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:35308) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1o6U5C-0000bR-El for bug-guix@gnu.org; Wed, 29 Jun 2022 05:34:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1o6U5C-00046O-Bx for bug-guix@gnu.org; Wed, 29 Jun 2022 05:34:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#56297: Guix style imperfections Resent-From: Maxime Devos Original-Sender: "Debbugs-submit" Resent-CC: bug-guix@gnu.org Resent-Date: Wed, 29 Jun 2022 09:34:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 56297 X-GNU-PR-Package: guix X-GNU-PR-Keywords: To: 56297@debbugs.gnu.org X-Debbugs-Original-To: bug-guix@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.165649520415722 (code B ref -1); Wed, 29 Jun 2022 09:34:02 +0000 Received: (at submit) by debbugs.gnu.org; 29 Jun 2022 09:33:24 +0000 Received: from localhost ([127.0.0.1]:57438 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1o6U4Z-00045W-P5 for submit@debbugs.gnu.org; Wed, 29 Jun 2022 05:33:23 -0400 Received: from lists.gnu.org ([209.51.188.17]:37714) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1o6U4X-00045N-1A for submit@debbugs.gnu.org; Wed, 29 Jun 2022 05:33:21 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42688) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1o6U4W-0005Sb-QN for bug-guix@gnu.org; Wed, 29 Jun 2022 05:33:20 -0400 Received: from michel.telenet-ops.be ([2a02:1800:110:4::f00:18]:39314) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1o6U4U-0008W2-Cp for bug-guix@gnu.org; Wed, 29 Jun 2022 05:33:20 -0400 Received: from ptr-bvsjgyhxw7psv60dyze.18120a2.ip6.access.telenet.be ([IPv6:2a02:1811:8c09:9d00:3c5f:2eff:feb0:ba5a]) by michel.telenet-ops.be with bizsmtp id oxZB270034UW6Th06xZBX5; Wed, 29 Jun 2022 11:33:11 +0200 Message-ID: <9499300db3fe4222f7126240fb2acad3cdf4371b.camel@telenet.be> From: Maxime Devos Date: Wed, 29 Jun 2022 11:33:05 +0200 Content-Type: multipart/signed; micalg="pgp-sha512"; protocol="application/pgp-signature"; boundary="=-Eo50Gz7S8HCjnRg6uFzR" User-Agent: Evolution 3.38.3-1 MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=telenet.be; s=r22; t=1656495191; bh=/VLN/hd/oX7gtRpEAOLqp6lS2VEcBsJhmqdAmRPgrtQ=; h=Subject:From:To:Date; b=duRSLMnwUpkspzwCyWeqnWAHDzhDFNrHxHQLZiz5ZLrGA3IrACUPW5ElyKCVoLVxJ M2K7OgbhQX95vNgxQk9xLR1jVzi2D8WHNgJc85tq5mqmMYBpEi5QNq4DUJ4HxxsU7w vUV1X2kWbaT6RCg4Ow+NM+yDizX2DQ/rVzdXOeK19Jo6vbkOPLyE2aZpgYoEqUD76I 5eJAj/c8DeKEOSJAUP/IOTD/wbZnuXwB6R6b2u5XR7dOwPB7DXdvj6XK+mjXBOL5tp M3CychfkKWnC7YdjXYzOibqbD175WUHyKvYfPXQWp18r/Db4box8Qg9B7lMYdgUmNW yItdjD1nOAV4g== Received-SPF: pass client-ip=2a02:1800:110:4::f00:18; envelope-from=maximedevos@telenet.be; helo=michel.telenet-ops.be X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-guix@gnu.org List-Id: Bug reports for GNU Guix List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-guix-bounces+larch=yhetil.org@gnu.org Sender: "bug-Guix" X-Migadu-Flow: FLOW_IN X-Migadu-To: larch@yhetil.org X-Migadu-Country: US ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1656495262; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:list-id:list-help:list-unsubscribe:list-subscribe: list-post:dkim-signature; bh=/VLN/hd/oX7gtRpEAOLqp6lS2VEcBsJhmqdAmRPgrtQ=; b=G+11tizxenCuSBrNFXgfYnVYs1oj6J7LEeliehRi+B/4a5EP0J47bYaKbdRLdvDdaZY272 +E8s/zIMu5zL/rQG3WEpDjdD19uR0szUlsdwdyY+M0O162G0aOm2WwuORDl+jelvgeY29t zBzq6UkrpXmwhkL1WwtL6V0fpdwCd4Bcc0+tNX1pPVfdtMIVHs8ZoCsQfMyvaYEb6Qux3O GXoQ4eEM4ar53o+L6WoHwNZLQWy/hvOWnbcbcxA7bU5DYVsWH51jxFGIJjwh5mnsMYg0Ha plCIaHSmcrOO7fUBxXRD5ZAFJhm+fL2pr0WqKgK1iB7zZw8ThV5ledasnplB3w== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1656495262; a=rsa-sha256; cv=none; b=siiyX4HAIdjtIh0OffVSXdvT0kKYHToKcqJ6SayIYYxMmdAzfAbftvX/uPjxz9E335wioe QQ3Rlb88RDScMgw8SsLex8PZo2qD1/OfW6hAUkogoepRgOpdcjGmw6bOBDZZSVUxlY8H5d EFDTSCMVdHkMGZXiNSHxgQa+Yhqm1xZ6olDOQW9U4e2V71SHsaaE8kptImUAoXdkoGp5OC OrYfCwmzC+vZbeqBzIEOQwvNKEfLiQeus3nBkTVbtXCTv+enECLll1IlPpR84fKOL52z5a iBbSWrLW2crMTwPV7hUc2BKeChXUL7ZbxlUKDbSZnP9/JfGhbVIiEDZHufAHeg== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=telenet.be header.s=r22 header.b=duRSLMnw; dmarc=fail reason="SPF not aligned (relaxed)" header.from=telenet.be (policy=none); spf=pass (aspmx1.migadu.com: domain of "bug-guix-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="bug-guix-bounces+larch=yhetil.org@gnu.org" X-Migadu-Spam-Score: 5.14 Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=telenet.be header.s=r22 header.b=duRSLMnw; dmarc=fail reason="SPF not aligned (relaxed)" header.from=telenet.be (policy=none); spf=pass (aspmx1.migadu.com: domain of "bug-guix-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="bug-guix-bounces+larch=yhetil.org@gnu.org" X-Migadu-Queue-Id: EA41542C53 X-Spam-Score: 5.14 X-Migadu-Scanner: scn1.migadu.com X-TUID: 5rryFucUPLXl --=-Eo50Gz7S8HCjnRg6uFzR Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi, "guix style" occasionally makes some decision that seem a bit questionable to me. More concretely, copy the definition of guile- next, put it in a .scm and rename it, and run "guix style -L . guile-next-styleme". I get: > (define-module (test)) > (use-modules (guix packages) (guix git-download) (gnu packages autotools)= (gnu packages guile) (guix utils) > (define-public guile-next > (let ((version "3.0.7") (revision "0") > (commit "d70c1dbebf9ac0fd45af4578c23983ec4a7da535")) Conventionally 'revision' is put on another line -- for these kind of let b= indings, (maybe all?), I would recommend to put all of them on separate lines. > (package > (inherit guile-3.0) > (name "guile-next-styleme") > (version (git-version version revision commit)) > (source [snip, LGTM]) > (arguments > (substitute-keyword-arguments (package-arguments guile-3.0) > ((#:phases phases > '%standard-phases) `(modify-phases ,phases Put %standard-phases on the same line ad #:phases phases and `(modify-phase= s ,phases on a new lineg=20 > (add-before 'check 'skip-failing-tests > (lambda _ > (substitute* "test-suite/standalone/t= est-out-of-memory" > (("!#") "!# > >(exit 77) >")) I'd prefer the original "!#\n\n(exit 77)\n" here, but I don't know if that'= s something 'Guix style' could feasibly do (there might be situations where a newline might be appropriate, how could "guix style" which is the case?). > (delete-file > "test-suite/tests/version.test") #t)= ))))) (Would be nice if "guix style" could be taught to remove those #t, but that= seems more a feature limitation than a bug to me.) > (native-inputs (modify-inputs (package-native-inputs guile-3.0) > (prepend autoconf > automake > libtool > flex > gnu-gettext > texinfo > gperf))) I'd consider it tidier to put (modify-inputs ...) on a new line > (synopsis "Development version of GNU Guile")))) Question: do people agree with these style choices? Greetings, Maxime. --=-Eo50Gz7S8HCjnRg6uFzR Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYrwcURccbWF4aW1lZGV2 b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7oJFAP4hReQBKh4e79ZjH4zlWDSf0wxF fIxX69bkVPNSJJIyAQEAlwFXbrhB8Fk0IRrOgnGHf7qfLhintGcnSxfbthuyQgI= =yODs -----END PGP SIGNATURE----- --=-Eo50Gz7S8HCjnRg6uFzR--