* Martin Michlmayr <[EMAIL PROTECTED]> [2005-11-03 20:02]:
> * Martin Michlmayr <[EMAIL PROTECTED]> [2004-05-19 02:14]:
> > partman doesn't recognize ext2 revision 0 as ext2r0 but shows it as
> > ext2.  Ideally, what it should do is this: if it finds an ext2 file
> > system, it should run "tune2fs -l" over the partition.
> 
> Is the patch below acceptable?
> 
> Comments:
>  - I'm not if testing for a file belonging to partman-ext2r0 is the
>    best way to check if it is available.  Maybe valid_filesystems/ext2r0
>    should be called?  But I don't know.

Anton, do you have a more elegant way or is it okay to apply the patch
I proposed?

> Index: partman-base/update.d/detected_filesystem
> ===================================================================
> --- partman-base/update.d/detected_filesystem (revision 31824)
> +++ partman-base/update.d/detected_filesystem (working copy)
> @@ -28,6 +28,11 @@
>       rm -f $id/detected_filesystem
>      else
>       [ -d $id ] || mkdir $id
> +     if [ "$filesystem" = "ext2" -a -f /lib/partman/parted_names/ext2r0 ]; 
> then
> +             if $(tune2fs -l $path | grep -q "^Filesystem revision #: \+0 
> (original)$"); then
> +                     filesystem=ext2r0
> +             fi
> +     fi
>       echo "$filesystem" >$id/detected_filesystem
>      fi  
>  fi

-- 
Martin Michlmayr
http://www.cyrius.com/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to