Re: Wrong log file displayed in status/package.php

[Andreas Barth <aba@not.so.argh.org>]

* Andreas Barth (aba@not.so.argh.org) [100305 23:39]:
> * Joachim Breitner (nomeata@debian.org) [100305 18:24]:
> > Hi,
> > 
> > Am Freitag, den 05.03.2010, 12:22 +0100 schrieb Joachim Breitner:
> > > on
> > > https://buildd.debian.org/status/package.php?suite=unstable&p=haskell-opengl&a=armel
> > > you can see that the build, version 2.2.3.0-2, failed "Build-Attempted"
> > > but below, a successful built is shown.
> > > 
> > > https://buildd.debian.org/build.cgi?pkg=haskell-opengl;arch=armel shows
> > > that 2.2.3.0-2 worked, but the binNMU 2.2.3.0-2+b1 failed.
> > > 
> > > I’d expect package.php to still list 2.2.3.0-2+b1 and display its
> > > version number.
> > > 
> > > Any idea how to fix this?
> > 
> > ok, the problem lies within mkstats, which imports the wb data into
> > status’ own format. There were changes to the format (probably aba’s)
> > that made the parser fail.
> > 
> > A patch which fixes this is available here:
> > http://git.debian.org/?p=users/nomeata/buildd-status.git;a=commitdiff;h=a2e723effa830b7777d91226d374567a6f804ef4
> > 
> > Please merge that change if mkstats is still to be used.
> > 
> > But I see that aba is re-implementing mkstats as mkstats.aba. Andreas,
> > is your mkstats.aba already used? If so, can you make sure that it works
> > for binNMUed packages as well?
> 
> It's already used, and it's not reimplemented, but rather shortend.
> 
> I'll take a look at it tomorrow, unless there is a patch available
> earlier.

I commited this change:

--- a/mkstats.aba
+++ b/mkstats.aba
@@ -39,7 +39,7 @@ tie %previous_db, 'DB_File', "$TARGET/db.$suite", O_RDONLY, 0, $DB_BTREE

 for my $arch (@archs) {
        open ALL, "-|", "/org/wanna-build/bin/wanna-build",
-                        "--format=#%S %p %v %u%n%D%p#%a%n%v^%S^%E^%s^%u^%X^%c^%P",
+                        "--format=#%S %p %v %u%n%D%p#%a%n%v%{+b}B%B^%S^%E^%s^%u^%X^%c^%P",
                        "-d", $suite, "-A", "$arch",
                        "--list", "all"
                or die("Not saving: Couldn't open $arch: $!");

(and I should probably define %V as an short way to specify %v%{+b}B%B in wanna-build)

It seems to work for me now. Can you please check if you still notice
an bug somewhere?

Also, please be on #d-buildd in IRC while you play with scripts on cimarosa. :)



Cheers,
Andi