Bug#514358: pdebuild host --debbuildopts inherited by target pbuild

Junichi Uekawa dancer at netfort.gr.jp
Sun Feb 8 23:03:06 UTC 2009


At Sun, 08 Feb 2009 15:49:58 +0000,
TJ wrote:
> 
> On Mon, 2009-02-09 at 00:02 +0900, Junichi Uekawa wrote:
> > Hmm....
> > 
> > IMO, That's a feature, not a bug. If this is really biting people, I
> > might consider adding that...
> 
> A feature to break the expected functionality? I would agree with you
> *if* dpkg-source hadn't changed the semantics of -I in versions >=
> v1.14.7.

So you are saying it's better to break pdebuild parameter option
semantics just for backports, which isn't nice.
At least, a documentation that says:

pdebuild --debbuildopts 

no longer works, but you need to specify:

pdebuild --debbuildopts 'for outside chroot' -- --debbuildopts 'for
inside chroot'

I'm fine with documentation fix accompanying the patch; and a warning.

It doesn't give any benefit for sid on sid users.

> But, the differences are there and using the parameter format for -I
> that is acceptable on the host will cause pre v1.14.7 to break the
> build.
> 
> The alternative would be to ignore the default -I behaviour in >=
> v1.14.7 and always pass a file with the exclusion list regardless of
> version. That requires the file be added to the package though (so it
> can be picked up inside the pbuild).

No, -I takes a list of file patterns, it won't require extra file, so
it shoud not be much of a problem to require explicitly passing -I.git
-ICVS etc.
 
> The other alternative of removing all VCS files and folders from the
> host source directory before a pdebuild so the -I option can be dropped
> (and therefore the build succeed) isn't viable.


regards,
	junichi
-- 
dancer@{netfort.gr.jp,debian.org}





More information about the Pbuilder-maint mailing list