From debbugs-submit-bounces@debbugs.gnu.org Mon Feb 17 09:36:15 2020 Received: (at 38408) by debbugs.gnu.org; 17 Feb 2020 14:36:15 +0000 Received: from localhost ([127.0.0.1]:38198 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1j3hVP-0003hr-Gs for submit@debbugs.gnu.org; Mon, 17 Feb 2020 09:36:15 -0500 Received: from eggs.gnu.org ([209.51.188.92]:49125) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1j3hVO-0003hf-8X for 38408@debbugs.gnu.org; Mon, 17 Feb 2020 09:36:14 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:53117) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1j3hVJ-00007Q-4Q; Mon, 17 Feb 2020 09:36:09 -0500 Received: from [2001:660:6102:320:e120:2c8f:8909:cdfe] (port=48602 helo=ribbon) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1j3hVH-0000yN-Az; Mon, 17 Feb 2020 09:36:07 -0500 From: =?utf-8?Q?Ludovic_Court=C3=A8s?= To: Efraim Flashner Subject: Re: [PATCH v9 3/8] Added Guile-Semver as a dependency to guix References: <8a86631d201313b1da427a5ceb2ca5f201e6546c.1580817140.git.mjbecze@riseup.net> <20200217100345.GI1968@E5400> X-URL: http://www.fdn.fr/~lcourtes/ X-Revolutionary-Date: 29 =?utf-8?Q?Pluvi=C3=B4se?= an 228 de la =?utf-8?Q?R=C3=A9volution?= X-PGP-Key-ID: 0x090B11993D9AEBB5 X-PGP-Key: http://www.fdn.fr/~lcourtes/ludovic.asc X-PGP-Fingerprint: 3CE4 6455 8A84 FDC6 9DB4 0CFB 090B 1199 3D9A EBB5 X-OS: x86_64-pc-linux-gnu Date: Mon, 17 Feb 2020 15:36:05 +0100 In-Reply-To: <20200217100345.GI1968@E5400> (Efraim Flashner's message of "Mon, 17 Feb 2020 12:03:45 +0200") Message-ID: <87wo8l702y.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Spam-Score: -0.7 (/) X-Debbugs-Envelope-To: 38408 Cc: 38408@debbugs.gnu.org, jsoo1@asu.edu, Martin Becze X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -1.7 (-) Hi, Efraim Flashner skribis: > From 578d6f023c706df999c1b1b1bb23c9771b279857 Mon Sep 17 00:00:00 2001 > From: Martin Becze > Date: Tue, 4 Feb 2020 07:18:20 -0500 > Subject: [PATCH 3/8] Added Guile-Semver as a dependency to guix > > * configure.ac: added check for guile-semver > * gnu/packages/package-management.scm (guix): added guile-semver as dep [...] > +dnl Check for Guile-Semver > +GUILE_MODULE_AVAILABLE([have_guile_semver], [(semver)]) > +if test "x$have_guile_semver" !=3D "xyes"; then > + AC_MSG_ERROR([Guile-Semver is missing; please install it.]) > +fi I think a hard dependency like this is too much. I would very much prefer to deal with it similar to how we deal with Guile-Newt or Guile-Charting: a soft dependency that=E2=80=99s entirely optional. But I guess that also depends on what Guile-Semver is used for. I just posted a question on this topic in that thread. :-) Thanks for reviving this patch series! Ludo=E2=80=99.