[Reportbug-maint] Emdebian version strings and the BTS

[Sandro Tosi]

Hi Neil,
sorry for not replying until now

On Sun, Feb 15, 2009 at 12:33, Neil Williams <codehelp at debian.org> wrote:
> On Sun, 15 Feb 2009 12:09:02 +0100
> Sandro Tosi <morph at debian.org> wrote:
>
>> On Sun, Feb 15, 2009 at 11:34, Neil Williams <codehelp at debian.org> wrote:
>> > A patch for reportbug is being tested. A large proportion of the
>> > development work to actually get Emdebian to the point of two releases
>> > only happened after the lenny freeze began. (Not the recent deep
>> > freeze, the initial toolchain freeze before DebConf8). (reportbug
>> > itself is not currently part of Emdebian for a variety of reasons.)
>>
>> I think involve us in this might be of some help (at least because we
>> know what's going on around reportbug that should be merged in later
>> on).
>
> I only had a quick chance to look at the reportbug issue during FOSDEM
> so this is the first time I've had a chance to discuss the idea of how
> to patch reportbug with anyone - my next email was to file the bug with
> the initial patch and get feedback from the reportbug maintainers.
>
> I should probably have done that the other way around, sorry.

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).

Cheers,
-- 
Sandro Tosi (aka morph, morpheus, matrixhasu)
My website: http://matrixhasu.altervista.org/
Me at Debian: http://wiki.debian.org/SandroTosi