Code review comment for lp://staging/~altair-ibn-la-ahad/ubuntu/precise/update-notifier/bug-887650

Revision history for this message
Sebastien Bacher (seb128) wrote :

If you want to get the fix SRUed to precise, some comments:
- could you use the version of the patch which was using in trunk
- you need to use "" around $mount_point to deal with paths with space in their name
- the bug should be made SRU compliant (https://wiki.ubuntu.com/StableReleaseUpdates)

Could you address those and resubmit the merge request?

review: Needs Fixing

« Back to merge proposal