Bug 599639 - udev rules broken for rawhide
Summary: udev rules broken for rawhide
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: libmtp
Version: 14
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Linus Walleij
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 642435
TreeView+ depends on / blocked
 
Reported: 2010-06-03 16:47 UTC by Bastien Nocera
Modified: 2010-12-04 12:47 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 642435 (view as bug list)
Environment:
Last Closed: 2010-12-04 12:47:55 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Bastien Nocera 2010-06-03 16:47:07 UTC
In the latest spec file:

# Remove permissions from udev rules, but register for ACL management
examples/hotplug -a"SYMLINK+=\"libmtp-%k\", ENV{ACL_MANAGE}=\"1\"" > libmtp.rules

Apparently ACL_MANAGE shouldn't be used in udev rules anymore, and broke gnome-bluetooth (see bug 588660 and the links near the end).

Comment 1 Linus Walleij 2010-06-05 09:36:29 UTC
OK edited the Fedora extenstion to remove this, built to
RawHide, is it working out?

Comment 2 Kevin Kofler 2010-06-08 23:00:43 UTC
Are you sure that just removing the ACL_MANAGE is sufficient? I think you'll have to set the TAG+="udev-acl" instead, as e.g. http://svn.debian.org/wsvn/pkg-gnome/desktop/unstable/gnome-bluetooth/debian/61-gnome-bluetooth-rfkill.rules or http://cvs.fedoraproject.org/viewvc/rpms/gnome-bluetooth/devel/61-gnome-bluetooth-rfkill.rules?revision=1.2&view=markup does.

Comment 3 Bastien Nocera 2010-06-09 00:47:22 UTC
Just removing ACL_MANAGE isn't enough, and apparently adding udev-acl is frowned upon. You'll need to ask Harald or upstream udev what to do in this case.

Comment 4 Linus Walleij 2010-06-09 23:30:44 UTC
OK now it says ENV{ACL_MANAGE}="0" TAG+="udev-acl"
But if adding udev-acl is frowned upon, then it's not good for
RFKill either I guess?

This is more than a bit idiomatic, is there some post I can
read up on the general idea of these strings and how it's
supposed to work these days...

Comment 5 Kevin Kofler 2010-06-09 23:34:48 UTC
ENV{ACL_MANAGE}="0" is incorrect. ENV{ACL_MANAGE}!="0" as an added condition would make sense.

Comment 6 Kevin Kofler 2010-06-09 23:36:13 UTC
(ENV{ACL_MANAGE}="0" is an assignment, not a condition, ENV{ACL_MANAGE}=="0" would be a condition, but the wrong one: if you check the gnome-bluetooth rules carefully, it SKIPS the block if ENV{ACL_MANAGE}=="0".)

Comment 7 Bug Zapper 2010-07-30 11:48:23 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 14 development cycle.
Changing version to '14'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

Comment 8 Linus Walleij 2010-10-19 20:57:43 UTC
Well I don't know how to proceed with this because I just
don't know what to do. I obviously know too little about
udev to make use of the advice given here, please help.
A patch is very good help, for example...

Comment 9 Kevin Kofler 2010-10-31 22:38:39 UTC
With the current udev in F14, ACL_MANAGE is not used anymore at all, so just TAG+="udev-acl" is likely to work.

But as far as I understand, a proper long-term solution would be for the udev authors to add some ENV{ID_*} variable for each library which needs one so that libraries don't have to set that udev-acl tag themselves.

Comment 10 Linus Walleij 2010-12-04 12:47:55 UTC
This is fixed in rawhide now, closing.


Note You need to log in before you can comment on or make changes to this bug.