Looking at the code I think we'd want the common hooks to run but not
the package hooks.  I've tested the following and it works, but the test
test_report.py fails because it passes a string for ui.

=== modified file 'apport/report.py'
--- apport/report.py    2013-09-27 03:56:13 +0000
+++ apport/report.py    2013-10-14 22:23:24 +0000
@@ -829,6 +829,10 @@
             if _run_hook(self, ui, hook):
                 return True
 
+        # LP: #1084979 - don't run package hooks if not sending to the crashdb
+        if not ui.ui.crashdb.accepts(self):
+            return False
+
         # binary package hook
         if package:
             for hook_dir in hook_dirs:

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

Title:
  Submitting error report asks confounding questions

To manage notifications about this bug go to:
https://bugs.launchpad.net/apport/+bug/1084979/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to