Hi, On Tuesday 19 February 2008, Grégory Oestreicher wrote: > Le jeudi 24 janvier 2008 16:49, Frans Pop a écrit : > > Attaching patch and especially the example recipes to ensure they > > remain available. Recipes will be useful as documentation. > > Here is an updated patch that correct a bug when > partman-auto-lvm/extra_devices is preseeded but none of them exist. This > patch is against the SVN repository as of today (2008/02/19) and replaces > the previous one.
Now that Beta1 is out I plan to look at your patch in detail soon. I noticed the following change in the updated patch: +++ b/packages/partman/partman-auto-lvm/lib/auto-lvm.sh [...] -+ if [ -d $tmpdevdir ]; then ++ if [ -d "$tmpdevdir" ]; then ++ echo "$tmpdevdir" >> /tmp/lvtest What is the purpose of this last line? Or is that just something left over from your testing? Cheers, FJP
