| Summary: | sys-fs/udisks:2: Volumes fail to mount with gnome-base/gvfs-1.12.0 due to enforced acl (setfacl) usage | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Martin Wegner <gentoo-bugs> |
| Component: | [OLD] GNOME | Assignee: | Freedesktop bugs <freedesktop-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | rafalklys |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| See Also: |
http://bugs.freedesktop.org/show_bug.cgi?id=48842 https://bugs.gentoo.org/show_bug.cgi?id=458568 https://bugs.freedesktop.org/show_bug.cgi?id=61162 |
||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
|
Description
Martin Wegner
2012-04-17 17:12:55 UTC
For one, it's not a gvfs requirement, it's inherited for udisks2. Also, it's not that unreasonable. Reported to upstream here: http://bugs.freedesktop.org/show_bug.cgi?id=48842 And missing sys-apps/acl RDEPEND committed to Portage: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-fs/udisks/udisks-1.94.0.ebuild?r1=1.1&r2=1.2 Does it still fail after installing sys-apps/acl (even if kernel has the mentioned options disabled) ? Yes, it still fails with another error message, setfacl reports missing ACL support in the filesystem. I will provide the exact error message as soon as I'm home again. Same problem here. ACLs for USB sticks reasonable? No way. Downgrading to gnome-base/gvfs-1.10.1 is a good workaround. I had acl installed prior and the acl support isn't working yet. A user will get the error message of "no such file or directory" because the devices are being mounted in "/run/media/$user". The user directories aren't being created automatically and so I have had to manually add needed user directories before mounting works for udisks. (In reply to comment #5) > Same problem here. ACLs for USB sticks reasonable? No way. ACLs are for mount points, not for the content. (In reply to comment #6) Does /run exist ? If so, is that a permission problem ? What's the exact error ? The exact error is: Command-line `setfacl -m "u:brian:rx" "/run/media/brian"' didn't exit normally: setfacl: /run/media/brian: Operation not supported I got the problem resolved. ACL support needs to be enabled in the kernel for tmpfs. Quoting from NEWS of udisks-1.94.0:
------------
udisks 1.92.0
------------
[ .. ]
Changes since udisks 1.91.0:
[ .. ]
David Zeuthen (6):
Avoid using $XDG_RUNTIME_DIR/media for now
Use /run/media/$USER for mounting
So that would require us to check for:
CONFIG_TMPFS_POSIX_ACL=y
From udisks ebuild and warn the user if the option is missing.
(In reply to comment #10) > Quoting from NEWS of udisks-1.94.0: > > ------------ > udisks 1.92.0 > ------------ > > [ .. ] > > Changes since udisks 1.91.0: > > [ .. ] > > David Zeuthen (6): > Avoid using $XDG_RUNTIME_DIR/media for now > Use /run/media/$USER for mounting > > So that would require us to check for: > > CONFIG_TMPFS_POSIX_ACL=y > > From udisks ebuild and warn the user if the option is missing. Now the ebuild will warn if that option is missing: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-fs/udisks/udisks-1.94.0.ebuild?r1=1.2&r2=1.3 From ebuild point of view, this bug should now be done. If you wish for sys-apps/acl and CONFIG_TMPFS_POSIX_ACL to become optional in udisks, go to the upstream bug and make your case there. It's not me who you have to convince. http://bugs.freedesktop.org/show_bug.cgi?id=48842 Here is a bug in udisks bugzilla, about removing hard dependency on ACLs: https://bugs.freedesktop.org/show_bug.cgi?id=61162. It have patch attached, it looks like it's just a matter of doing fallback to behavior from previous udisks when acls are not present. (In reply to comment #12) > Here is a bug in udisks bugzilla, about removing hard dependency on ACLs: > https://bugs.freedesktop.org/show_bug.cgi?id=61162. It have patch attached, > it looks like it's just a matter of doing fallback to behavior from previous > udisks when acls are not present. See bug 458568. Of course we will support non-ACL kernels soon as upstream does too, but not before. Yes, I just wanted to leave here note for all those who will find this bug from udisks ebuild comment. Didn't spot the other bug, thanks. As udisks-2.1.x is lastest package in my ACL-less system, I send patch to mailing list, with --disable-acl build option, to avoid compiling with ACL support. |
