GNU bug report logs - #30683
[PATCH] build: add a configure flag to force --sandbox

Previous Next

Full log

Message #18 received at 30683 <at> debbugs.gnu.org (full text, mbox):

From: Mike Frysinger <vapier <at> gentoo.org>
To: Assaf Gordon <assafgordon <at> gmail.com>
Cc: 30683 <at> debbugs.gnu.org
Subject: Re: bug#30683: [PATCH] build: add a configure flag to force --sandbox
Date: Fri, 2 Mar 2018 18:27:37 -0500

[Message part 1 (text/plain, inline)]

On 02 Mar 2018 16:07, Assaf Gordon wrote:
> tag 30683 notabug
> close 30683
> stop
> 
> Hello Mike,
> 
> On Fri, Mar 02, 2018 at 05:28:15PM -0500, Mike Frysinger wrote:
> > * configure.ac: Add --enable-forced-sandbox option, and define
> > ENABLE_FORCED_SANDBOX when enabled.
> 
> Conceptually I like your idea (since I've added the original
> --sandbox option to both gnu sed and gawk).
> 
> However,
> Adding such "--enable" options to "./configure" goes against the gnu coding standards,
> which say:
> 
>    No ‘--enable’ option should ever cause one feature to replace another.
>    No ‘--enable’ option should ever substitute one useful behavior for
>    another useful behavior. The only proper use for ‘--enable’ is for
>    questions of whether to build part of the program or exclude it.
>    (source: https://www.gnu.org/prep/standards/html_node/Configuration.html)

frankly, i consider this a bikeshed aspect, and i have no interest in
debating what would be acceptable.  so if you tell me what name you
want, i'll happily adjust the patch/code.
-mike

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.