[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#88413: PATCH]: modconf broken with ash



> > Index: modconf
> > ===================================================================
> > RCS file: /cvs/debian-boot/modconf/modconf,v
> > retrieving revision 1.7
> > diff -r1.7 modconf
> > 115c115
> > <       tdir=${directory//\//_}
> > ---
> >>       tdir=`eval echo \\${directory//\//_}`
> 
> It still wouldn't work with ash.  In fact, those two lines are exactly
> the same in any POSIX shell.

I fail to see why the BASH_VERSION conditional around line 115 was there anyway.
The fix that I uploaded was to just remove the bash'ism entirely so ash didn't
have to parse it, though I didn't commit to CVS.  Randolph commited the above to
cvs, I believe. 

If Randolph's fix still won't work with ash I'll just commit what I have, cause
it works and I don't see the reason behind adding the conditional bash'ism.

David
 
 




Reply to: