create rationalize_options() and use it (bugzilla #3525)

more checks should be done in this function, and used at more places (but where?)

1 files changed, 15 insertions, 2 deletions

diff --git a/perl-install/fs.pm b/perl-install/fs.pm
index 148f6cf23..06a864f25 100644
--- a/perl-install/fs.pm
+++ b/perl-install/fs.pm
@@ -483,6 +483,19 @@ user,exec,dev,suid )."),
     );
 }
 
+
+sub rationalize_options {
+    my ($part) = @_;
+
+    my ($options, $unknown) = mount_options_unpack($part);
+
+    if (!isThisFs('reiserfs', $part)) {
+	$options->{notail} = 0;
+    }
+
+    mount_options_pack($part, $options, $unknown);
+}
+
 sub set_default_options {
     my ($part, %opts) = @_;
     #- opts are: useSupermount security iocharset codepage
@@ -540,8 +553,6 @@ sub set_default_options {
     }
     if (isThisFs('reiserfs', $part)) {
 	$options->{notail} = 1;
-    } else {
-	$options->{notail} = 0;
     }
     if (isLoopback($part) && !isSwap($part)) { #- no need for loop option for swap files
 	$options->{loop} = 1;
@@ -561,6 +572,8 @@ sub set_default_options {
     }
 
     mount_options_pack($part, $options, $unknown);
+
+    rationalize_options($part);
 }
 
 sub set_all_default_options {