From ae0d8fbfc6f2c3b5659c82e7a5be6eb7de4c82ee Mon Sep 17 00:00:00 2001 From: Pascal Terjan Date: Thu, 13 May 2010 17:15:26 +0000 Subject: enable acl in fstab not on the fs as we currently do not handle options on the fs in an unified way and have no way to check if they are set in the apps --- perl-install/fs/format.pm | 12 ++++-------- perl-install/fs/mount_options.pm | 3 +++ 2 files changed, 7 insertions(+), 8 deletions(-) (limited to 'perl-install/fs') diff --git a/perl-install/fs/format.pm b/perl-install/fs/format.pm index 3c1c9d346..9caa4f45f 100644 --- a/perl-install/fs/format.pm +++ b/perl-install/fs/format.pm @@ -271,14 +271,10 @@ sub formatMount_part { } #- setting user_xattr on /home (or "/" if no /home) - #- setting acl everywhere - if (!$part->{isMounted} && member($part->{fs_type}, qw(ext2 ext3 ext4))) { - my $default_options = 'acl'; - if ($part->{mntpoint} eq '/home' || - (!fs::get::has_mntpoint('/home', $all_hds) && $part->{mntpoint} eq '/')) { - $default_options .= ',user_xattr'; - } - run_program::run('tune2fs', '-o', $default_options, devices::make($part->{real_device} || $part->{device})); + if (!$part->{isMounted} && member($part->{fs_type}, qw(ext3 ext4)) + && ($part->{mntpoint} eq '/home' || + !fs::get::has_mntpoint('/home', $all_hds) && $part->{mntpoint} eq '/')) { + run_program::run('tune2fs', '-o', 'user_xattr', devices::make($part->{real_device} || $part->{device})); } fs::mount::part($part, 0, $wait_message); diff --git a/perl-install/fs/mount_options.pm b/perl-install/fs/mount_options.pm index 899356daa..ee8099b93 100644 --- a/perl-install/fs/mount_options.pm +++ b/perl-install/fs/mount_options.pm @@ -239,6 +239,9 @@ sub set_default { $options->{notail} = 1; $options->{user_xattr} = 1; } + if (member($part->{fs_type}, qw(ext2 ext3 ext4)) { + $options->{acl} = 1; + } if (isLoopback($part) && !isSwap($part)) { #- no need for loop option for swap files $options->{loop} = 1; } -- cgit v1.2.1