[Reportbug-maint] Pre-approval for reportbug/3.45
dato at net.com.org.es
Sat Aug 16 13:04:09 UTC 2008
* Sandro Tosi [Fri, 15 Aug 2008 23:56:32 +0200]:
> > I'd like to receive you approval before upload reportbug/3.45 in
> > unstable to let it transit to Lenny after the right amount of time.
> > This upload fixes two bugs: password method is missing in urwid
> > interface, bugs list is not shown anymore due to BTS page changes (I
> > know I know, we got to remove that part of code, and move to SOAP, but
> > we are still planning that part and it would be a big change in code).
> > Please let me know, so that I can upload.
Yes; please mail back when it's uploaded.
Now if you don't mind a bit of perfectionism... (possibly follow-up to
-maint with CC to me).
> +def get_password(prompt=None):
> + return getpass.getpass(prompt)
Maybe you always pass a prompt when calling get_password(), but None
seems a bad default to me. Try getpass.getpass(None)...
if prompt is None:
> + except:
> + pass
Adeodato Simó dato at net.com.org.es
Debian Developer adeodato at debian.org
Listening to: Alanis Morissette - Front row
More information about the Reportbug-maint