[Merge] lp:~jibel/ubuntu/trusty/ubuntu-release-upgrader/dont_fail_if_no_apport into lp:ubuntu/ubuntu-release-upgrader

Brian Murray brian at ubuntu.com
Tue Mar 4 16:52:06 UTC 2014


Review: Needs Fixing

Thanks for working on this, I think instead of logging the error regarding apport-bug failing we should instead change the error message (e.g. lines 819-822) to not mention that a bug reporting process will be started and instead tell them how to report a bug.  So something like:

if os.path.exists('/usr/bin/apport-bug'):
   self._view.error(existing message)
   subproces.Popen(bug reporting)
else:
   self._view.error("Install apport to report a bug...")
   logging.error("Could not automatically report a bug")

If that change makes sense do you want to work on it? or shall I?
-- 
https://code.launchpad.net/~jibel/ubuntu/trusty/ubuntu-release-upgrader/dont_fail_if_no_apport/+merge/208550
Your team Ubuntu branches is subscribed to branch lp:ubuntu/ubuntu-release-upgrader.



More information about the Ubuntu-reviews mailing list