Launchpad has imported 3 comments from the remote bug at https://bugzilla.redhat.com/show_bug.cgi?id=702457.
If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. ------------------------------------------------------------------------ On 2011-05-05T17:31:43+00:00 Andrew wrote: Created attachment 497172 Patch to fix the bug Description of problem: I found the bug in Ubuntu Natty: https://bugs.launchpad.net/ubuntu/+source/urlgrabber/+bug/776555 The progress_obj is documented as: po.start(filename, url, basename, length, text) But it is called as: po.start(filename, url, basename, size=..., text=...) Regression (worked fine in Ubuntu Maverick) Version-Release number of selected component (if applicable): Ubuntu version of python-urlgrabber: 3.9.1-4 How reproducible: 100% reproducible Steps to Reproduce: 1. Pass a progress_obj to the grabber 2. Implement the documented API 3. Code fails when called incorrectly by grabber.py Actual results: Code fails with a KeyboardInterrupt Expected results: Should work as documented, and as it used to work Additional info: See details in ubuntu bug: https://bugs.launchpad.net/ubuntu/+source/urlgrabber/+bug/776555 Reply at: https://bugs.launchpad.net/ubuntu/+source/urlgrabber/+bug/776555/comments/5 ------------------------------------------------------------------------ On 2011-05-19T20:15:31+00:00 James wrote: This code has been like this pretty much forever (2005 the text param. was added), I can see how you might be confused if you just looked at the documentation though. Alas. we can't just remove the param name from the call, as the API has other params: def start(self, filename=None, url=None, basename=None, size=None, now=None, text=None): ...so our only sane option is to treat it as a documentation bug, and fix that. Sorry, if that doesn't help you much. Reply at: https://bugs.launchpad.net/ubuntu/+source/urlgrabber/+bug/776555/comments/7 ------------------------------------------------------------------------ On 2013-03-14T19:34:50+00:00 James wrote: The documentation was fixed upstream. This request was evaluated by Red Hat Engineering for inclusion in a Red Hat Enterprise Linux maintenance release. Red Hat does not currently plan to provide this change in a Red Hat Enterprise Linux update release for currently deployed products. With the goal of minimizing risk of change for deployed systems, and in response to customer and partner requirements, Red Hat takes a conservative approach when evaluating enhancements for inclusion in maintenance updates for currently deployed products. The primary objectives of update releases are to enable new hardware platform support and to resolve critical defects. Reply at: https://bugs.launchpad.net/ubuntu/+source/urlgrabber/+bug/776555/comments/9 ** Changed in: urlgrabber (Fedora) Status: Unknown => Won't Fix ** Changed in: urlgrabber (Fedora) Importance: Unknown => Undecided -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/776555 Title: KeyboardInterrupt thrown by urlgrab To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/urlgrabber/+bug/776555/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
