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

Bug#413183: Patch



Here's the patch to make sure that superfluous flags are not set.

--
David Härdeman

Index: debian/changelog
===================================================================
--- debian/changelog	(revision 45633)
+++ debian/changelog	(working copy)
@@ -1,3 +1,10 @@
+partman-lvm (52) UNRELEASED; urgency=low
+
+  * Make sure that the lvm, raid and swap flags are used in a mutually
+    exclusive manner.
+
+ -- David Härdeman <david@hardeman.nu>  Sat,  3 Mar 2007 02:55:38 +0100
+
 partman-lvm (51) unstable; urgency=low
 
   [ Updated translations ]
Index: update.d/lvm_sync_flag
===================================================================
--- update.d/lvm_sync_flag	(revision 45633)
+++ update.d/lvm_sync_flag	(working copy)
@@ -51,13 +51,28 @@
 done
 close_dialog
 
+# Some flags make no sense in combination with lvm
+cleanflags=''
+for $flag in $flags; do
+	if [ "$flag" = raid ]; then
+		continue
+	elif [ "$flag" = swap ]; then
+		continue
+	elif [ -n "$cleanflags" ]; then
+		cleanflags="$cleanflags
+$flag"
+	else
+		cleanflags="$flag"
+	fi
+done
+
 if [ "$method" = '' ] && [ "$has_lvm" = yes ]; then
 	echo lvm >$dev/$id/method
 	rm -f $dev/$id/use_filesystem
 	rm -f $dev/$id/format
 elif [ "$method" = lvm ] && [ "$has_lvm" = no ]; then
 	open_dialog SET_FLAGS $id
-	write_line "$flags"
+	write_line "$cleanflags"
 	write_line lvm
 	write_line NO_MORE
 	close_dialog


Reply to: