Merge lp:~brian-murray/apport/check-cdrom into lp:~apport-hackers/apport/trunk

[Brian Murray]

Looking at casper it seems that the root filesytem on Live Media, like a persistent USB stick, is mounted as /cdrom. I've updated generic.py to check /cdrom for low free space before reporting a bug.

[Martin Pitt (pitti)]

/cdrom is a read-only mount in the live system as well as installed systems, so it'll always be 100% full. I don't think it's interesting at all apport-wise.

On a live system the only interesting mount is /, as this has the combined read-write system partition.

review: Needs Fixing

[Martin Pitt (pitti)]

Most likely this is not necessary at all, but keeping open for the case that you intended to check for something different.

[Martin Pitt (pitti)]

No response for four months, and this looks unintended, closing. Please reopen if you still think that there's something wrong here. Thanks!

review: Disapprove

Unmerged revisions

1902. By Brian Murray on 2011-06-13

check for low space in root filesystem on persistent media

=== modified file 'data/general-hooks/generic.py'
--- data/general-hooks/generic.py	2011-06-12 19:07:37 +0000
+++ data/general-hooks/generic.py	2011-06-13 18:53:22 +0000
@@ -22,6 +22,7 @@
     # check for low space
     mounts = {'/': 'system',
               '/var': '/var',
+              '/cdrom': 'system'
              }
 
     home = os.getenv('HOME')