On 05/03/2015 06:45 PM, Lars Wirzenius wrote:
On Sun, May 03, 2015 at 02:55:58PM +0200, [email protected] 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.
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
