According to your suggestion I replaced the call of the 'mtp-detect'
command with the following code:

    if command_available("mtp-detect"):
        report['MTPDetect'] = command_output("mtp-detect")
    else:
        ui.information("Please install the package 'mtp-tools' so we can gather 
"\
                        "more detailed debugging information. Afterwards, rerun 
" \
                        "the command 'ubuntu-bug libmtp' or add more 
information "\
                        "to an existing bug report by running the command "\
                        "'apport-collect -p libmtp '<bugnumber>', replacing "\
                        "<bugnumber> with the number of the bug report.")

Do you agree with my extension of the message? I though it could be
useful until a proper GUI has been written for 'apport-collect' and
'ubuntu-bug', but maybe it only adds confusion.

What about localisation? Shouldn't we call gettext for the strings
displayed in the information dialogue?

** Attachment added: "source_libmtp.py using command_available()"
   http://launchpadlibrarian.net/36402455/source_libmtp.py

-- 
add Apport hook to libmtp
https://bugs.launchpad.net/bugs/430261
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to