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

[PATCH] no_mount_point resume-partitioning bug



Dante Díaz reported a navigation bug in partman's "no mount point" error
messages (https://launchpad.net/bugs/256459). Nicolas Valcárcel provided
a fix for partman-ext3, which I've generalised to the other broken
partman packages.

OK to commit despite the freeze?

Index: partman-ext2r0/check.d/nomountpoint_ext2r0
===================================================================
--- partman-ext2r0/check.d/nomountpoint_ext2r0	(revision 56254)
+++ partman-ext2r0/check.d/nomountpoint_ext2r0	(working copy)
@@ -28,7 +28,7 @@
 			db_input critical partman-ext2r0/no_mount_point || true
 			db_go || exit 1
 			db_get partman-ext2r0/no_mount_point
-			if [ "$RET" = 'Resume partitioning' ]; then
+			if [ "$RET" = true ]; then
 				exit 1
 			fi
 			;;
Index: partman-ext3/check.d/nomountpoint_ext3
===================================================================
--- partman-ext3/check.d/nomountpoint_ext3	(revision 56253)
+++ partman-ext3/check.d/nomountpoint_ext3	(working copy)
@@ -28,7 +28,7 @@
 			db_input critical partman-ext3/no_mount_point || true
 			db_go || exit 1
 			db_get partman-ext3/no_mount_point
-			if [ "$RET" = 'Resume partitioning' ]; then
+			if [ "$RET" = true ]; then
 				exit 1
 			fi
 			;;
Index: partman-jfs/check.d/nomountpoint_jfs
===================================================================
--- partman-jfs/check.d/nomountpoint_jfs	(revision 56254)
+++ partman-jfs/check.d/nomountpoint_jfs	(working copy)
@@ -28,7 +28,7 @@
 			db_input critical partman-jfs/no_mount_point || true
 			db_go || exit 1
 			db_get partman-jfs/no_mount_point
-			if [ "$RET" = "Resume partitioning" ]; then
+			if [ "$RET" = true ]; then
 				exit 1
 			fi
 			;;
Index: partman-reiserfs/check.d/nomountpoint_reiserfs
===================================================================
--- partman-reiserfs/check.d/nomountpoint_reiserfs	(revision 56254)
+++ partman-reiserfs/check.d/nomountpoint_reiserfs	(working copy)
@@ -28,7 +28,7 @@
 			db_input critical partman-reiserfs/no_mount_point || true
 			db_go || exit 1
 			db_get partman-reiserfs/no_mount_point
-			if [ "$RET" = 'Resume partitioning' ]; then
+			if [ "$RET" = true ]; then
 				exit 1
 			fi
 			;;
Index: partman-xfs/check.d/nomountpoint_xfs
===================================================================
--- partman-xfs/check.d/nomountpoint_xfs	(revision 56254)
+++ partman-xfs/check.d/nomountpoint_xfs	(working copy)
@@ -28,7 +28,7 @@
 			db_input critical partman-xfs/no_mount_point || true
 			db_go || exit 1
 			db_get partman-xfs/no_mount_point
-			if [ "$RET" = 'Resume partitioning' ]; then
+			if [ "$RET" = true ]; then
 				exit 1
 			fi
 			;;

Thanks,

-- 
Colin Watson                                       [cjwatson@ubuntu.com]


Reply to: