Hi Sandro,
Le lundi 27 juillet 2009 à 15:19 +0200, Sandro Tosi a écrit :
> Hi Carl,
> thanks a lot for the patch (as usual)!
>
> On Thu, Jul 23, 2009 at 23:44, Carl Chenet<cha...@ohmytux.com> wrote:
> > Package: reportbug
> > Version: 4.5
> > Severity: normal
> > Tags: patch
> >
> > Hi,
> >
> > Here is a patch to allow "querybts --buglist package" to provide a list of
> > the bugs for the given package.
>
> just one clarification, in :
>
> @@ -142,6 +147,9 @@ def main():
> sys.exit(1)
>
> if use_browser:
> + if use_browser and buglist:
> + print "--mbox and --web won't work together."
> + sys.exit(1)
> package = args[0]
> m = re.match('^#?(\d+)$', package)
> if m:
> @@ -153,6 +161,9 @@ def main():
> return
>
This hunk above is to ensure the buglist option and the use_browser
option won't be used together, because there is no point about it. But I
don't see why I told "--mbox and --web won't work together.", it should
be "--web and --buglist won't work together."
> if mbox:
> + if mbox and buglist:
> + print "--mbox and --buglist won't work together."
> + sys.exit(1)
> for bugnum in args:
> m = re.match('^#?(\d+)$', bugnum)
> if not m:
>
> the first hunk is unrelated to the introduction of --buglist, and you
> added it because it needs to be fixed anyhow, right?
>
> Cheers,
Thanks for double-checking.
Bye,
Carl Chenet
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
