On Wed, Dec 21, 2011 at 10:49:15AM -0000, TJ wrote: > Your patch looks good - although you could drop the use of $found to > track detection and use "break 2" which would avoid those extra 4 lines > of code.
Not at all; 'break 2' has the effect that no other lines from /etc/crypttab will be processed, which is definitely not what we want. -- Steve Langasek Give me a lever long enough and a Free OS Debian Developer to set it on, and I can move the world. Ubuntu Developer http://www.debian.org/ [email protected] [email protected] -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/874774 Title: could not mount /dev/mapper/cryptswap1 To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/cryptsetup/+bug/874774/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
