Re: [PATCH 0/4] pull: allow disabling command builds

Sami Kerola
2014-03-07T14:52:43Z

On 7 March 2014 04:44, Karel Zak <kzak@redhat.com> wrote:
> On Sun, Feb 16, 2014 at 03:22:52PM +0000, Sami Kerola wrote:
>> This pull request has four patches that will allow a builder to disable
>> all commands.
>
> Note that I don't silently ignore these patches,

Hi Karel,

I know you don't, and it was expected this is not the usual patch set
that gets review and comments immediately.

> but I'd like to play
> with it later and try to provide better granularity, maybe something
> like
>
> --enable-util=
> --disable-util=
> --enable-all
> --disable-all
>
> where is comma separated list of the utils names (we already
> use thing like this for static programs), then we can avoid huge
> number of --enable-* options.

The --enable-util=<item1,item2> sounds much better approach. I wonder
if this approach could be converted to reasonably reusable m4
definition, resulting to deprecation of various command specific
disable/enables.

--
Sami Kerola