[Reportbug-maint] Emdebian version strings and the BTS

[Neil Williams]

On Fri, 20 Feb 2009 22:02:30 +0100
Sandro Tosi <morph at debian.org> wrote:

> Ah not problem, given the small size of the proposed patch; I was fear
> of a very impacting patch, that's why I wanted to "shout" asap :)
> 
> > This is my initial stab at a patch:
> >
> > $ diff -u reportbug-3.41/reportbug reportbug.new/reportbug
> > --- reportbug-3.41/reportbug    2008-06-04 16:20:36.000000000 +0100
> > +++ reportbug.new/reportbug     2009-02-06 18:41:59.000000000 +0000
> > @@ -1193,6 +1193,10 @@
> >             (pkgversion, pkgavail, depends, recommends, conffiles, maintainer,
> >              installed, origin, vendor, reportinfo, priority, desc, src_name,
> >              fulldesc, state) = status
> > +            emdebian = re.search('em[0-9]$', pkgversion, re.M | re.I)
> > +            if emdebian:
> > +                ewrite('%s (%s) is an Emdebian package, redirecting to buildd.emdebian.org.\n',package, pkgversion)
> > +                package="buildd.emdebian.org"
> >
> >         buginfo = '/usr/share/bug/' + package
> >         bugexec = submitas = submitto = presubj = None
> >
> > Hastily prepared on the Eurostar on the way to FOSDEM and the first
> > time I've ever had to actually patch python. ;-)
> >
> > It worked in basic testing but I'm sure you'll have ideas on how it can
> > be improved.
> 
> It does the job, that's fine. We (maints) should think about a way to
> easily reassign bugs given a set of rules. Since nothing was done so
> far in this direction, your patch is fine (please, rebase your work on
> the reportbug experimental version, that will soon be uploaded in
> sid).

After initial confusion that reportbug in 3.99 is now a directory and
the file I patched is now in bin/, I managed to refresh the patch.

Test output:
bsdmainutils (6.1.10em1) is an Emdebian package, redirecting to buildd.emdebian.org.

email body would have contained:
Subject: buildd.emdebian.org: test bug
Package: buildd.emdebian.org
Version: 6.1.10em1

Hmm, that Version: could be dropped but it doesn't do any harm in this
specific case. I don't know enough python to know if setting pkgversion
to null would be correct. It is worth outputting pkgversion in the
ewrite, but whether it can be cleared after that point I will leave to
you.

Patch:

--- reportbug.old/bin/reportbug	2009-02-26 23:05:58.000000000 +0000
+++ reportbug-3.99.1/bin/reportbug	2009-02-26 23:06:48.000000000 +0000
@@ -1220,6 +1220,11 @@
              installed, origin, vendor, reportinfo, priority, desc, src_name,
              fulldesc, state, suggests) = status
 
+            emdebian = re.search('em[0-9]$', pkgversion, re.M | re.I)
+            if emdebian:
+                ewrite('%s (%s) is an Emdebian package, redirecting to buildd.emdebian.org.\n',package, pkgversion)
+                package="buildd.emdebian.org"
+
         buginfo = '/usr/share/bug/' + package
         bugexec = submitas = submitto = presubj = None
         reportwith = []



-- 


Neil Williams
=============
http://www.data-freedom.org/
http://www.linux.codehelp.co.uk/
http://e-mail.is-not-s.ms/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/reportbug-maint/attachments/20090226/a985df27/attachment.pgp