Bug 130229

Summary:	/dev/ttyIx devices are owned by group root instead of uucp
Product:	[openSUSE] SUSE LINUX 10.0	Reporter:	Per Jessen <per>
Component:	Hotplug	Assignee:	Kay Sievers <kasievers>
Status:	RESOLVED FIXED	QA Contact:	E-mail List <qa-bugs>
Severity:	Normal
Priority:	P5 - None
Version:	RC 1
Target Milestone:	---
Hardware:	i586
OS:	Other
Whiteboard:
Found By:	Other	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---

Description Per Jessen 2005-10-24 08:46:15 UTC

In previous releases such as 9.0, the /dev/ttyIx devices were owned by
root.uucp.  In 10.0RC1 they are owned by root.root.  I suspect this is an
unintentional change.

Comment 1 Karsten Keil 2005-10-26 19:52:47 UTC

Yes, we should set uucp group in the udev rules too.

Comment 2 Kay Sievers 2005-10-26 20:02:09 UTC

I will change the static_devices entries with the next release.
Karten, will we still need the static entries for 10.1?

Comment 3 Karsten Keil 2005-10-26 20:18:08 UTC

I will try to add some more udev/sysfs support to the old i4l core for 10.1, so maybe most of them are not longer needed then.
I have some more updates for the isdn udev rules, but need some more testing
(solving the hotplug event issue #116462 and the Auerswald #129938).

Comment 4 Kay Sievers 2005-11-09 03:30:41 UTC

The current udev has the right permissions for the nodes.

They are created by the rpm and placed in /lib/udev/devices/*, which content is
copied over to /dev at every startup. Karsten, can you take the static
nodes over to the isdn rpm, so we don't install them unconditionally on
every system?

Comment 5 Karsten Keil 2005-11-12 16:31:07 UTC

Since isdn is in the default selection, it doesn't change it, but I agree they should be in the i4l-base.rpm for future products.
What would be the best way to do this ?
1.) create the nodes on build time in /BUILD_ROOT/lib/udev/devices/ and package them into the rpm ?
2.) Do this in the post install script ?

What must be called to copy the nodes to /dev/... (in post install of the rpm)?

Comment 6 Kay Sievers 2005-11-12 19:10:03 UTC

> 1.) create the nodes on build time
Just use the rpm native dev macro:
  %attr(600,root,uucp) %dev(c,45,128)     /lib/udev/devices/ippp0

You can copy the whole section from the current udev.spec file.
Can you take over the isdn rules too?

> 2.) Do this in the post install script ?
RPM keeps track of this.

> What must be called to copy the nodes to /dev/
The udev boot script will do this, no need for the isdn package to
do anything here.

Comment 7 Karsten Keil 2005-11-14 12:41:48 UTC

No that is not enough, think about that somebody install i4l-base later in a running system, this have to be done without a reboot cycle.

Comment 8 Kay Sievers 2005-11-14 14:09:59 UTC

Ok, right. In that case, you may just copy the whole /lib/udev/devices/ directory
with cp --update ... to /dev. That should copy all missing nodes over without
overwriting the ones who may have changed its permissions after boot. Sounds ok?

Comment 9 Kay Sievers 2005-12-26 04:55:13 UTC

This is fixed in the current development release.