From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: Allow disabling of CONFIG_SHELL in the configure phase Date: Tue, 13 Sep 2016 14:07:29 +0200 Message-ID: <87fup457ge.fsf@gnu.org> References: 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]:60398) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bjmUy-0000jJ-Cs for guix-devel@gnu.org; Tue, 13 Sep 2016 08:07:37 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bjmUu-00019g-4j for guix-devel@gnu.org; Tue, 13 Sep 2016 08:07:35 -0400 In-Reply-To: (Arun Isaac's message of "Mon, 12 Sep 2016 12:18:31 +0530") 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" To: Arun Isaac Cc: guix-devel Arun Isaac skribis: > I am packaging optipng (http://optipng.sourceforge.net/) with the > gnu-build-system, and its configure script does not like being passed > CONFIG_SHELL. It fails with the following error: > > ./configure: error: unknown option: CONFIG_SHELL=3D/gnu/store/ykzwykkvr2c= 80rw4l1qh3mvfdkl7jibi-bash-4.3.42/bin/bash > > Can we allow disabling of CONFIG_SHELL by passing some keyword argument > to the configure phase? Right now, the CONFIG_SHELL argument is always > passed to ./configure, and the only way to get around this is to replace > the configure phase for the optipng package with a specialized function > that does not pass CONFIG_SHELL. That seems like too much work, and it > will be nice if we can just disable CONFIG_SHELL with a keyword argument. I find that it=E2=80=99s not that much work to replace the =E2=80=98configu= re=E2=80=99 phase, and it=E2=80=99s not too common (Autoconf-generated =E2=80=98configure=E2= =80=99 scripts all honor CONFIG_SHELL.) We could add a Boolean flag for CONFIG_SHELL, but I bet manually written =E2=80=98configure=E2=80=99 scripts may need more than just that anyway, so= I=E2=80=99m not sure it would buy us much. WDYT? Ludo=E2=80=99.