[buildd-tools-devel] Bug#551311: Bug#551311: Updated patches for debuild like functionality in sbuild.
Roger Leigh
rleigh at codelibre.net
Sun Dec 13 12:18:49 UTC 2009
On Sat, Dec 12, 2009 at 09:04:29PM -0500, Andres Mejia wrote:
> On Saturday 12 December 2009 20:22:11 Roger Leigh wrote:
> > On Sat, Dec 12, 2009 at 06:18:43PM -0500, Andres Mejia wrote:
> > > On Saturday 12 December 2009 05:52:12 Roger Leigh wrote:
> > > > On Sat, Dec 12, 2009 at 12:42:10AM -0500, Andres Mejia wrote:
> > > > > On Friday 11 December 2009 07:17:04 Roger Leigh wrote:
> > > > > > On Thu, Dec 10, 2009 at 09:38:08PM -0500, Andres Mejia wrote:
> > > > > > > First I would like to say, thank you for reviewing my patches. :)
> > > > > > >
> > > > > > > Next I would like to say, please don't attach any patches inline
> > > > > > > anymore. I spent way too much time ripping out the patch from my
> > > > > > > email client and then had to properly format the patch just to
> > > > > > > get it to apply.
> > > > > >
> > > > > > Thanks for the updated patch. I've attached what I currently have,
> > > > > > but I've not yet merged all of your changes into it yet.
> > > > > >
> > > > > > This applies after my first patch (it replaces the second patch).
> > > > > >
> > > > > >
> > > > > > Regards,
> > > > > > Roger
> > > > >
> > > > > It didn't apply cleanly for me. I noticed the first hunk from the two
> > > > > patches adds "use Sbuild::ChrootRoot;", so I figure there's already
> > > > > some problem with the patch. I'll just wait until it's applied to the
> > > > > git tree rather than attempt to fix the patch again. I'll report
> > > > > problems then.
> > > >
> > > > I've put all the current changes at
> > > > git://git.debian.org/users/rleigh/sbuild.git
> > > > branch "debuild-am"
> > > >
> > > > It's working for me so far.
> > > >
> > > > > One thing I found wrong was the pipe_command() call for
> > > > > dpkg-parsechangelog. The hash needs to have 'DIR => getcwd()', else
> > > > > it will fail.
> > > >
> > > > I think this was caused by something else, since getcwd() is the
> > > > default. It's working for me without this.
> > >
> > > I don't know why, but for some reason, it wants to default to using "/"
> > > as DIR in my system. I've even tried this on a pure checkout of the
> > > debuild-am branch. I am running this through Debian sid, if that helps.
> > > I've attached the patch that fixes this for me.
> >
> > Are you using the libs in lib/ when testing (PERL5LIB=lib ./bin/sbuild
> > ...)? Otherwise you might be using a copy of the system modules.
>
> I'm using perl directly with appropriate options. As in,
>
> $ perl -Isbuild/lib sbuild/bin/sbuild sbuild
>
> > DIR does default to "/" in Chroot.pm, new() line 65. However, in Build.pm,
> > this is overridden in run() line 253:
> >
> > my $chroot_defaults = $session->get('Defaults');
> > $chroot_defaults->{'DIR'} =
> > $session->strip_chroot_path($session->get('Build Location'));
>
> Yes, this is the problem. You sure you pushed all your changes? I don't see
> this line in the debuild-am branch and I just verified this is the case via
> http://git.debian.org/?p=users/rleigh/sbuild.git as well.
I pushed all of those changes.
It's definitely there, line 345 of
http://git.debian.org/?p=users/rleigh/sbuild.git;a=blob;f=lib/Sbuild/Build.pm;h=888d291901c6c9ca1baa0383ba7f8483ee87f30e;hb=0f9e497679774569473e451e854f93d19cddeed7
This isn't a recent change. It's been there since the 11th of April
this year.
Regards,
Roger
--
.''`. Roger Leigh
: :' : Debian GNU/Linux http://people.debian.org/~rleigh/
`. `' Printing on GNU/Linux? http://gutenprint.sourceforge.net/
`- GPG Public Key: 0x25BFB848 Please GPG sign your mail.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/buildd-tools-devel/attachments/20091213/b748dba7/attachment.pgp>
More information about the Buildd-tools-devel
mailing list