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

Bug#638822: smb.conf: acl related options need mount flags in fstab (acl, user_xattr)



I tested a modified version of patch, and it did not work as expected.
I fixed the typo in the recipe, but only the user_xattr option showed
up in the fstab.  the acl option was missing.  I have no idea what can
be done to fix this.

The patch I tried is attached.

-- 
Happy hacking
Petter Reinholdtsen


 * Add acl and user_attr options to the home0 and the backup partitions,
   to allow samba to use ACLs (Closes: #638822).

diff --git a/lib/partman/common/90edumain b/lib/partman/common/90edumain
index 7ffa878..d899671 100644
--- a/lib/partman/common/90edumain
+++ b/lib/partman/common/90edumain
@@ -62,6 +62,8 @@ debian-edu-install/text/partman-debian-edu-main ::
         format{ }
         use_filesystem{ }
         filesystem{ ext4 }
+        options/acl{ acl }
+        options/user_xattr{ user_xattr }
         mountpoint{ /skole/backup } .
 
 
@@ -71,6 +73,8 @@ debian-edu-install/text/partman-debian-edu-main ::
 	format{ }
 	use_filesystem{ }
 	filesystem{ ext4 }
+	options/acl{ acl }
+	options/user_xattr{ user_xattr }
 	mountpoint{ /skole/tjener/home0 } .
 
 32 1000 -1 ext4
diff --git a/lib/partman/common/91edumain+ltsp b/lib/partman/common/91edumain+ltsp
index 5767bc8..92eef6f 100644
--- a/lib/partman/common/91edumain+ltsp
+++ b/lib/partman/common/91edumain+ltsp
@@ -70,6 +70,8 @@ debian-edu-install/text/partman-debian-edu-main+ltsp ::
 	format{ }
 	use_filesystem{ }
 	filesystem{ ext4 }
+	options/acl{ acl }
+	options/user_xattr{ user_xattr }
 	mountpoint{ /skole/tjener/home0 } .
 
 64 1000 15360 ext4
@@ -86,6 +88,8 @@ debian-edu-install/text/partman-debian-edu-main+ltsp ::
         format{ }
         use_filesystem{ }
         filesystem{ ext4 }
+        options/acl{ acl }
+        options/user_xattr{ user_xattr }
         mountpoint{ /skole/backup } .
 
 32 1000 -1 ext4

Reply to: