On Wed, Feb 14, 2007 at 11:05:22PM +0900, Junichi Uekawa wrote:
> > > The same applies to --debuildopts IIRC, this is slightly related to 
> > > #229834 as
> > > well.
> > 
> >  I don't think so, I use spaces in debuildopts to pass multiple options,
> >  for example --debbuildopts "-vx.y -sa".
> > 
> >  Perhaps this bug is about having spacing support in --debootstrapopts?
> 
> Documentation would be nice.
> 
> Changing command-line option behavior is destructive to existing scripts and 
> should be avoided.

Currently using a space as a --debootstrapopts argument to separate its
arguments won't work anyway as debootstrap parses that as a non-option:

--debootstrapopts "--help -a=i386"

correctly gives

+ cdebootstrap --variant=buildd '--help -a=i386' sid . http://127.0.0.1/debian
cdebootstrap: unrecognized option `--help -a=i386'

and you are expected to pass cdebootstrap only options, as passing a valid
non-option will result in debootstrap having too much arguments:

--debootstrapopts testing

+ cdebootstrap --variant=buildd testing sid . http://127.0.0.1/debian
cdebootstrap: too much arguments

so yes, this bug is really about properly supporting spaces within
--debootstrapopts arguments

I'm not sure about the correct way to implement this, the most obvious is to
save IFS, use IFS=' ' during debootstrap invocation and restore IFS after that.

However, this is a proposed patch to the man page:

--- /tmp/Q7UxfBjLcP-pbuilder.8  2007-02-14 17:04:09.940839524 +0100
+++ /tmp/y6gOWXELkB-pbuilder.8  2007-02-14 17:04:09.948839524 +0100
@@ -482,8 +482,9 @@

 .TP
 .BI "\-\-debootstrapopts " "\-\-variant=buildd"
-Add extra command-line options to debootstrap.
+Add extra command-line option to debootstrap in the form of --option=value.

+This option can be used multiple times to pass debootstrap multiple options.
 .TP
 .BI "\-\-debootstrap " "debootstrap"
 Use specified debootstrap implementation as debootstrap.


Although I prefer properly supporting arguments with spaces, as it is not going
to break anything. And mentioning "you can't use spaces" in the man page just
looks silly :)

my two cents,
filippo
--
Filippo Giunchedi - http://esaurito.net
PGP key: 0x6B79D401
random quote follows:

If there is any better use for being famous and respected than using
that status to question orthodoxy, I haven't found it yet.
-- Eric S. Raymond

Attachment: signature.asc
Description: Digital signature

Reply via email to