[buildd-tools-devel] Bug#538083: sbuild: Does not distinguish between failed/attempted builds on failure

[Roger Leigh]

Package: sbuild
Version: 0.58.6-1
Severity: important

15:57 < zobel> phil: can we have a sbuild which actually distinguishs between
               failed and attempted builds (missing build-deps and FTBFS)?
15:57 < zobel> like the sbuild we had with neuro
16:25 < phil> rleigh: ^^
21:21 < rleigh> zobel/phil: Distinguishing between failed/attempted builds is
                certainly possible.  I just haven't had that patch; it wasn't
                IIRC part of any sources I've seen myself.  If you have a copy
                of neuro's code, I'll be happy to merge it.  I don't think it
                was in his SVN repo.
21:22 < rleigh> phil: Regarding the failure to purge, I'll take a look.  Could
                you file a bug?
21:41 < rleigh> phil: Could you try http://paste.debian.net:80/42431/ and let
                me know if that fixed the cleanup?
21:47 < zobel> rleigh: rleigh at d.o?
21:55 < rleigh> zobel: Yes.
22:05 < rleigh> zobel: At this point in time, a diff against the point at which
                is was branched would be much more useful than a drop of the
                code.
22:07 < rleigh> zobel: Though we could potentially diff it against the SVN (if
                still available).
22:18 < rleigh> zobel: Thanks for the email.  The diff against SVN (rev 107) is
                pretty messy http://paste.debian.net:80/42434/
22:21 < rleigh> zobel: I think we have most of these changes in sbuild.git, but
                there's obviously a small bit to do with the wanna-build
                giveback missing.
22:21 < rleigh> zobel: This looks like the auto_giveback in build() which is
                missing.
22:22 < rleigh> (--built rather than --give-back)
22:23 < rleigh> We're also missing --attempted in analyze_fail_stage.
22:26 < rleigh> I don't have time to look into this right now, but sbuild can
                be converted to use Buildd::wannabuild_command in all three
                places.  This is a case where the wanna-build usage between
                buildd* and sbuild* needs unifying.
22:49 -!- blueness [~hnsctq40 at cpe-74-77-139-157.buffalo.res.rr.com] has quit
          [Quit: Ex-Chat]
22:51 < zobel> rleigh: i just said, i am missing that feature before switching
               to the new version of the sbuild we are currently running on
               phil's and luk's buildds.
22:51 < zobel> and i want to keep that feature..
22:55 < rleigh> zobel: Completely understood.  If you can confirm that it's
                just the missing wb --attempted in analyze_fail_stage, that's
                trivial to add.  I'll add it once I'm finished with the schroot
                work I'm currently doing, though a patch would speed things
                along if you have time to do that.


-- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (550, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.30-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages sbuild depends on:
ii  adduser                       3.110      add and remove users and groups
ii  libsbuild-perl                0.58.6-1   Tool for building Debian binary pa
ii  perl                          5.10.0-24  Larry Wall's Practical Extraction 
ii  perl-modules                  5.10.0-24  Core Perl modules

Versions of packages sbuild recommends:
ii  debootstrap                   1.0.13     Bootstrap a basic Debian system
ii  fakeroot                      1.12.4     Gives a fake root environment

Versions of packages sbuild suggests:
ii  deborphan                     1.7.28     program that can find unused packa
ii  wget                          1.11.4-2   retrieves files from the web

-- no debconf information