On Wednesday 03 June 2009 18:30:24 Daniel Baumann wrote:
> Kel Modderman wrote:
> > How about something like:
> > + /filesystem*|/live*)
> > + # used by live-initramfs
> > + continue
> > + ;;
>
> looks good.
>
> >> in the second replace, we also add '/' so that $REG_MTPTS is never
> >> empty. it would be nice if you can make the whole unmounting of local
> >> filesystems conditional, so that the script doesn't try to do anything
> >> when there's nothing to unmount.
> >
> > I do not know why adding / to REG_MTPTS helps. What is the rationale for it?
>
> if $REG_MTPTS is empty, the script fails with an error.
>
Hmm, there is already a check to see if $REG_MTPTS is empty:
---
if [ "$REG_MTPTS" ]
then
REG_MTPTS="$(pioodl $REG_MTPTS)"
...
---
Could it be that the error may have been introduced by piping pioodl function
output through `grep -v /live' as in your current hack?
Thanks, Kel.
