[apport] bugnumbers crashed with IOError in open_local_file()

[Emilio Pozuelo Monfort]

Binary package hint: bughelper

Doing a "bugnumbers -l launchpad.net/~pochu/+bugs" or a "bugnumbers -l bugs.launchpad.net/~pochu" crashes bugnumbers.

It should handle urls without "https://"

ProblemType: Crash
Architecture: i386
Date: Tue Apr 3 19:54:51 2007
DistroRelease: Ubuntu 7.04
ExecutablePath: /usr/bin/bugnumbers
InterpreterPath: /usr/bin/python2.5
Package: bughelper 0.1.7
PackageArchitecture: all
ProcCmdline: /usr/bin/python /usr/bin/bugnumbers -l launchpad.net/~pochu/+bugs
ProcCwd: /home/emilio/bzr/bughelper.0.1
ProcEnviron:
 SHELL=/bin/bash
 PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
 LANG=en_US.UTF-8
PythonArgs: ['/usr/bin/bugnumbers', '-l', 'launchpad.net/~pochu/+bugs']
SourcePackage: bughelper
Uname: Linux kiko 2.6.20-13-generic #2 SMP Sun Mar 25 00:21:25 UTC 2007 i686 GNU/Linux

[Emilio Pozuelo Monfort (pochu)]

This patch against bughelper.main fixes it.

[Emilio Pozuelo Monfort (pochu)]

BTW, this patch seems more intelligent :)

[Markus Korn (thekorn)]

Emilio,
nice patch, works fine for me.
My personal opinion is: we should inform the user that we changed the url (maybe the user is running bughelper in a script, so we should tell him that he has a systematic mistake in his url).
That's why I added a print statement to your patch.

What do you think?

Markus

[Emilio Pozuelo Monfort (pochu)]

Hi Markus

Sounds reasonably, and it won't hurt ;)

[Markus Korn (thekorn)]

------------------------------------------------------------
revno: 132
committer: Markus Korn <email address hidden>
branch nick: bughelper.main
timestamp: Thu 2007-04-05 13:53:25 +0200
message:
  commited Emilio's patch to fix bug 102480
------------------------------------------------------------

[Markus Korn (thekorn)]

Ah, we need a workaround for the bughelper.0.1 release

[Emilio Pozuelo Monfort (pochu)]

What about this patch? (similar to the other one)

[Emilio Pozuelo Monfort (pochu)]

So should this be fixed in 0.1, or there is no need? Note that there is already a patch against 0.1 ;)

[Markus Korn (thekorn)]

Well, I'm in a kind of twisted situation:
On the one hand I think we could commit that 0.1 patch but on the other hand I think: "We don't have any url-error-handling in 0.1, so why to add that patch?"
The problem is: in 0.1 you get a crash when you run "./bughelper -l launch.net" even with your patch but in .main we are handling that crash with an error-message.

Markus

[Emilio Pozuelo Monfort (pochu)]

Yeah Markus, that makes a lot of sense!

So then I'm changing the target to 0.2, and let's concentrate in fixing milestone bugs to release 0.2! :)

[Emilio Pozuelo Monfort (pochu)]

Fixed with 0.2~169