From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56726) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gTaS8-0002ur-NL for guix-patches@gnu.org; Sun, 02 Dec 2018 17:43:05 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gTaPF-0007u3-3q for guix-patches@gnu.org; Sun, 02 Dec 2018 17:40:06 -0500 Received: from debbugs.gnu.org ([208.118.235.43]:54214) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gTaPE-0007to-GZ for guix-patches@gnu.org; Sun, 02 Dec 2018 17:40:05 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1gTaPE-00077P-B9 for guix-patches@gnu.org; Sun, 02 Dec 2018 17:40:04 -0500 Subject: [bug#33572] [PATCH] Make Guile-JSON a required dependency. Resent-Message-ID: From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=) References: <20181202051822.8344-1-ericbavier@centurylink.net> Date: Sun, 02 Dec 2018 23:39:04 +0100 In-Reply-To: <20181202051822.8344-1-ericbavier@centurylink.net> (ericbavier's message of "Sat, 1 Dec 2018 23:18:22 -0600") Message-ID: <875zwbk0zb.fsf@gnu.org> 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: ericbavier@centurylink.net Cc: Eric Bavier , 33572@debbugs.gnu.org Hello Eric, ericbavier@centurylink.net skribis: > From: Eric Bavier > > * README (Requirements): Remove "optional" verbiage. > * configure.ac (HAVE_GUILE_JSON): Remove Automake conditional. > (have_guile_json): Error if not "yes". > * Makefile.am (MODULE, SCM_TESTS)[HAVE_GUILE_JSON]: Add modules and tests > unconditionally. > * gnu/packages/package-mangement.scm (guix-minimal)[propagated-inputs]: L= eave > guile-json input. Could you include guix.texi changes along the lines of what Joshua proposed on guix-devel? Otherwise LGTM, thanks! Ludo=E2=80=99.