Bug#249755: doesn't detect ext2 revision 0 file system

[Martin Michlmayr <tbm@cyrius.com>]

* Martin Michlmayr <tbm@cyrius.com> [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.


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 $dev | grep -q "^Filesystem revision #: \+0 (original)$"); then
+			filesystem=ext2r0
+		fi
+	fi
 	echo "$filesystem" >$id/detected_filesystem
     fi  
 fi

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