Comment 20 for bug 874774

Revision history for this message
Steve Langasek (vorlon) wrote : Re: [Bug 874774] Re: could not mount /dev/mapper/cryptswap1

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 address hidden> <email address hidden>