<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" content="1" /><style type="text/css">p, li { white-space: pre-wrap; }</style></head><body style=" font-family:'DejaVu Sans Mono'; font-size:9pt; font-weight:400; font-style:normal;">On Wednesday 03 June 2009 18:30:24 Daniel Baumann wrote:<br>
> Kel Modderman wrote:<br>
> > How about something like:<br>
> > + /filesystem*|/live*)<br>
> > + # used by live-initramfs<br>
> > + continue<br>
> > + ;;<br>
> <br>
> looks good.<br>
> <br>
> >> in the second replace, we also add '/' so that $REG_MTPTS is never<br>
> >> empty. it would be nice if you can make the whole unmounting of local<br>
> >> filesystems conditional, so that the script doesn't try to do anything<br>
> >> when there's nothing to unmount.<br>
> > <br>
> > I do not know why adding / to REG_MTPTS helps. What is the rationale for it?<br>
> <br>
> if $REG_MTPTS is empty, the script fails with an error.<br>
> <br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>Hmm, there is already a check to see if $REG_MTPTS is empty:<br>
---<br>
if [ "$REG_MTPTS" ]<br>
then<br>
REG_MTPTS="$(pioodl $REG_MTPTS)"<br>
...<br>
---<br>
Could it be that the error may have been introduced by piping pioodl function<br>
output through `grep -v /live' as in your current hack?<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>Thanks, Kel.</p></body></html>