[Vmdebootstrap-devel] Bug#770722: Bug#770722: [PATCH] add --debootstrap-opts to pass additional options to debootstrap

[Jan Gerber]

On 05/03/2015 06:45 PM, Lars Wirzenius wrote:
> On Sun, May 03, 2015 at 02:55:58PM +0200, j at mailb.org wrote:
>> +        self.settings.string(
>> +            ['debootstrap-opts'],
>> +            'pass additional options to debootstrap (check debootstrap --help for valid options)')
>
> I would make this a self.settings.string_list type option instead.
> This is an option type provided by cliapp that means the value is a
> list of strings. Thus
>
>      vmdebootstrap --debootstrap-option --foo --debootstrap-option --bar
>
> would result in a value ["--foo", "--bar"], rather than having the
> program have to parse the option value itself. This also means that
> the cliapp configuration file parsing can handle this automatically.
>

That was my first thought too, but after looking at common options one 
would pass to debootstrap, most of them are

  --key value

with self.settings.string_list that would become

  --debootstrap-option --key --debootstrap-option value

that looks ugly and seams to verbose.