Merge lp:~ronj/apport/betterErrorBox into lp:~apport-hackers/apport/trunk

[Ronan Jouchet]

#559345: Give a more explicit message when ubuntu-bug decides a package is "not a genuine Ubuntu package"

I just refactored the condition, separated the two cases, and made the UnreportableReason a bit more explicit.

I'm kinda new, please check the flow is correct and ensure I didn't break anything about localization.

[lp:~ronj/apport/betterErrorBox]

1761. By Ronan Jouchet on 2010-04-09

syntax fix

Unmerged revisions

=== modified file 'apport/ui.py'
--- apport/ui.py	2010-03-31 14:19:05 +0000
+++ apport/ui.py	2010-04-09 15:47:19 +0000
@@ -80,12 +80,15 @@
     # check package origin; we do that after adding hooks, so that hooks have
     # the chance to set a third-party CrashDB.
     try:
-        if ('Package' not in report or \
-              not apport.packaging.is_distro_package(report['Package'].split()[0])) \
-              and 'CrashDB' not in report:
-            #TRANS: %s is the name of the operating system
-            report['UnreportableReason'] = _('This is not a genuine %s package') % \
-                report['DistroRelease'].split()[0]
+        if 'CrashDB' not in report:
+            if 'Package' not in report
+                report['UnreportableReason'] = _('This package does not seem to be installed correctly') % \
+                    report['DistroRelease'].split()[0]
+            elif not apport.packaging.is_distro_package(report['Package'].split()[0]) 
+                #TRANS: %s is the name of the operating system
+                report['UnreportableReason'] = _('This package does not come from %s \
+official repositories. Please remove any PPA and try again.') % \
+                    report['DistroRelease'].split()[0]
     except ValueError:
         # this happens if we are collecting information on an uninstalled
         # package