[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#30683: [PATCH] build: add a configure flag to force --sandbox
From: |
Assaf Gordon |
Subject: |
bug#30683: [PATCH] build: add a configure flag to force --sandbox |
Date: |
Fri, 2 Mar 2018 16:44:40 -0700 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
Hello Eric, Mike,
(replying to both recent messages)
On Fri, Mar 02, 2018 at 05:20:07PM -0600, Eric Blake wrote:
> On 03/02/2018 05:07 PM, Assaf Gordon wrote:
>
> >Adding such "--enable" options to "./configure" goes against the gnu coding
> >standards,
>
> Would a different spelling, such as '--with-forced-sandbox-default=on/off'
> be better?
On Fri, Mar 2, 2018 at 4:27 PM, Mike Frysinger <address@hidden> wrote:
> [...] so if you tell me what name you
> want, i'll happily adjust the patch/code.
I generally do not object to this feature (regardless of the name).
However when I previously suggested something similar for coreutils [1]
(a ./configure flag to change the default 'ls' quoting style),
it was explained that such things should be avoided [2].
[1] https://lists.gnu.org/archive/html/bug-coreutils/2016-02/msg00057.html
[2] https://lists.gnu.org/archive/html/bug-coreutils/2016-02/msg00058.html
Perhaps there's no issue in adding it to sed - in which case I'm happy to
commit it.
Jim - what do you think?
regards,
- assaf
bug#30683: [PATCH] build: add a configure flag to force --sandbox, Mike Frysinger, 2018/03/02