Minor note related to the test case & implementation:
AFAIK apport.packaging.enabled() checks /etc/default/apport file contents for '^\s*enabled\s*=\s*0\s*$' regexp.
Meanwhile default contents of that file (on natty) are as follows:
# set this to 0 to disable apport, or to 1 to enable it
# you can temporarily override this with
# sudo service apport start force_start=1
enabled=0
Should that "temporarily override" part be removed altogether or changed to mention limits of such?
Or does that comment mean apport.packaging.enabled() should check such overrides as well?
Minor note related to the test case & implementation:
AFAIK apport. packaging. enabled( ) checks /etc/default/apport file contents for '^\s*enabled\ s*=\s*0\ s*$' regexp.
Meanwhile default contents of that file (on natty) are as follows:
# set this to 0 to disable apport, or to 1 to enable it
# you can temporarily override this with
# sudo service apport start force_start=1
enabled=0
Should that "temporarily override" part be removed altogether or changed to mention limits of such? packaging. enabled( ) should check such overrides as well?
Or does that comment mean apport.