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

Lars Wirzenius liw at liw.fi
Sun May 3 16:45:02 UTC 2015


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.

-- 
Happiness is a tested backup.



More information about the Vmdebootstrap-devel mailing list