[Reportbug-maint] Pre-approval for reportbug/3.45
Sandro Tosi
matrixhasu at gmail.com
Sat Aug 16 15:40:47 UTC 2008
Hi,
On Sat, Aug 16, 2008 at 15:04, Adeodato Simó <dato at net.com.org.es> wrote:
> * 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.
reportbug v3.45 has been 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)...
>
> Maybe this?:
>
> def get_password(prompt=None):
> if prompt is None:
> return getpass.getpass()
> else:
> return getpass.getpass(prompt)
As agreed on IRC, I left as in the diff proposed, because that's the
same code used in text ui.
>> + except:
>> + pass
>
> Eeek.
Thanks because I'd like to avoid corner cases where "('Select bugs',
[])" is not there, so I don't care if remove fails.
Thanks,
Sandro
--
Sandro Tosi (aka morph, Morpheus, matrixhasu)
My website: http://matrixhasu.altervista.org/
Me at Debian: http://wiki.debian.org/SandroTosi
More information about the Reportbug-maint
mailing list