Ubuntu
mountall package

Merge lp:~mdz/ubuntu/lucid/mountall/bug-557177 into lp:ubuntu/lucid/mountall

  1. Lucid (10.04)
  2. bug-557177
  3. Merge into lucid
Proposed by Matt Zimmerman
Status: Rejected
Rejected by: James Westby
Proposed branch: lp:~mdz/ubuntu/lucid/mountall/bug-557177
Merge into: lp:ubuntu/lucid/mountall
Diff against target: 12 lines (+1/-1)
1 file modified
conf/mounted-tmp.conf (+1/-1)
To merge this branch: bzr merge lp:~mdz/ubuntu/lucid/mountall/bug-557177
Reviewer Review Type Date Requested Status
Scott James Remnant (Canonical) (community) Disapprove
Review via email: mp+23002@code.launchpad.net

Description of the change

One-liner to make mountall-tmp.conf fail safely

To post a comment you must log in.
Revision history for this message
Scott James Remnant (Canonical) (canonical-scott) wrote :

I already committed a much better fix for this yesterday.

review: Disapprove
Revision history for this message
Matt Zimmerman (mdz) wrote :

On Thu, Apr 08, 2010 at 10:01:35AM -0000, Scott James Remnant wrote:
> Review: Disapprove
> I already committed a much better fix for this yesterday.

Oh, good. For some reason, this didn't end up linked to the bug, so I
didn't see it.

--
 - mdz

Unmerged revisions

297. By Matt Zimmerman

Make mounted-tmp.conf fail safe if MOUNTPOINT is unspecified (LP: #557177)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'conf/mounted-tmp.conf'
--- conf/mounted-tmp.conf 2009-12-14 08:35:37 +0000
+++ conf/mounted-tmp.conf 2010-04-08 08:33:34 +0000
@@ -12,7 +12,7 @@
12script12script
13 . /etc/default/rcS13 . /etc/default/rcS
1414
15 cd "${MOUNTPOINT}"15 cd "${MOUNTPOINT:?mount point not specified}"
16 rm -f .X*-lock16 rm -f .X*-lock
1717
18 case "${TMPTIME}" in18 case "${TMPTIME}" in

Subscribers

People subscribed via source and target branches

to all changes: