Bug#312121: [Pbuilder-maint] Bug#312121: pbuilder: The attached patch should handle all cases

Junichi Uekawa dancer at netfort.gr.jp
Thu Oct 20 14:19:28 UTC 2005


Hi,

> > Hmm.. is this patch removing -P or am I missing something?
> > I'd expect it to be something like
> > -    echo "dpkg-buildpackage -S -us -uc -r${BUILDSOURCEROOTCMD}
> > $DEBBUILDOPTS" | perl -pe 's/(^|\s)-P(\s|$)/$1$2/g' | /bin/bash
> > +    echo "dpkg-buildpackage -S -us -uc -r${BUILDSOURCEROOTCMD}
> > $DEBBUILDOPTS" | perl -pe 's/(^|\s)-B(\s|$)/$1$2/g' | /bin/bash
> > 
> 
> Hmm - no, you are right.
> 
> Then you probably need to double-check that it actually works as
> intended (really weird - I am sure my problems went away after
> applying that hack, but that obviously can't be...).


I'm worried if people are going to ignore files named '-B', 
since you can probably ignore such files with passing a 
command-line argument '-i -B' to dpkg-buildpackage.

I can't really think of a sane way to beat this corner 
case without breaking existing behavior.


regards,
	junichi




More information about the Pbuilder-maint mailing list