From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:45637) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gqUhv-0005wW-N0 for guix-patches@gnu.org; Sun, 03 Feb 2019 22:14:04 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gqUhu-0000Ee-WE for guix-patches@gnu.org; Sun, 03 Feb 2019 22:14:03 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:58665) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gqUhu-0000EM-DS for guix-patches@gnu.org; Sun, 03 Feb 2019 22:14:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1gqUhu-0000q4-5m for guix-patches@gnu.org; Sun, 03 Feb 2019 22:14:02 -0500 Subject: [bug#34217] [PATCH 11/11] gnu: Add ruby-cucumber and ruby-aruba. Resent-Message-ID: References: <87zhrmgy0x.fsf@cbaines.net> <20190127114956.26570-1-mail@cbaines.net> <20190127114956.26570-11-mail@cbaines.net> <20190203231817.0a722915@alma-ubu> From: Ricardo Wurmus In-reply-to: <20190203231817.0a722915@alma-ubu> Date: Mon, 04 Feb 2019 00:26:59 +0100 Message-ID: <87r2cowj0c.fsf@elephly.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: =?UTF-8?Q?Bj=C3=B6rn_?= =?UTF-8?Q?H=C3=B6fling?= Cc: 34217@debbugs.gnu.org Bj=C3=B6rn H=C3=B6fling writes: >> + (propagated-inputs >> + (map (lambda (input) >> + (if (string=3D? (car input) "ruby-cucumber") >> + `("ruby-cucumber" ,ruby-cucumber-without-tests) >> + input)) >> + (package-propagated-inputs ruby-aruba))) > > This is really nit-picking, I haven't tried it out myself and I haven't > used it much myself, but I think this could be written more elegant with > match-lambda [syntax errors might be included]: > > (map (match-lambda > (("ruby-cucumber" . pkg) > `("ruby-cucumber" ,ruby-cucumber-without-tests)) > (input input) > ...) Or like this `(("ruby-cucumber" ,ruby-cucumber-without-tests) ,@(alist-delete "ruby-cucumber" (package-propagated-inputs ruby-aruba))) --=20 Ricardo