FS#13842 - [udev] clean some files/rules

Attached to Project: Arch Linux
Opened by Gerardo Exequiel Pozzi (djgera) - Wednesday, 18 March 2009, 06:25 GMT
Last edited by Tobias Powalowski (tpowa) - Saturday, 28 March 2009, 12:13 GMT
Task Type Feature Request
Category Packages: Testing
Status Closed
Assigned To Tobias Powalowski (tpowa)
Aaron Griffin (phrakture)
Thomas Bächler (brain0)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description: The udev package ships some files that aren't used in Arch Linux. So for keep it simply and stupid...

/* can be deleted */
40-ia64.rules: only for IA64/MIPS (not used in i686 or x86_64) (for kernel option CONFIG_SGI_SN)
40-ppc.rules: Maybe for ArchLinuxPPC project (for kernel options CONFIG*ISERIES*)
40-s390.rules: (not used in i686 x86_64) (for kernel option CONFIG_ZCRYPT on s390)
40-zaptel.rules: (not in kernel-vanilla/arch modules, there are a pkgbuild in unsupported that make this module and rules that are needed)
40-infiniband.rules: (not used, arch-kernel not support CONFIG_INFINIBAND)
/* end of can be deleted */

If you prefer in don't delete all/some these files, maybe rename to "{name}.rules.notused" or "{name}.rules.example" so udev don't look on these files.

/* need to edit */
40-isdn.rules: there are a reference to group "dialout" (inexistent in UID/GID database for arch), i see that these group in other files are changed from "dialout" to "uucp", so maybe this need to include in the "arch-udev-rules.patch"
/* end of need to edit */

Move readme-udev-arch.txt to /usr/share/doc/udev/readme-udev-arch.txt this more apropiate location.

Additional info:
udev-140-1 (and below)
This task depends upon

Closed by  Tobias Powalowski (tpowa)
Saturday, 28 March 2009, 12:13 GMT
Reason for closing:  Implemented
Comment by Tobias Powalowski (tpowa) - Wednesday, 18 March 2009, 21:31 GMT
We ship packages as they should, so removing standard rules is not imho not needed here, some might need them.
Changing dialout to uucp is needed in every rule file, will do this in next release which comes soon.
Comment by Aaron Griffin (phrakture) - Wednesday, 18 March 2009, 22:06 GMT
Actually, udev ships these rules in a directory that is not normally installed.

We take a extra step in the PKGBUILD to copy rules/packages/* to the package.

What's suggested here is simply to NOT copy all of them because they're not used.

I moved the dialout sed line to the end of the build function to cover the ISDN case

Also, regarding the readme - I STRONGLY think we should delete it. udev has very good man pages, and we do nothing Arch-specific that is not covered by the normal udev docs. If anything is missing, the additional documentation should be submitted upstream for inclusion (just like all our other packages)
Comment by Tobias Powalowski (tpowa) - Saturday, 28 March 2009, 12:13 GMT
40-zaptel.rules: (not in kernel-vanilla/arch modules, there are a pkgbuild in unsupported that make this module and rules that are needed)
40-infiniband.rules: (not used, arch-kernel not support CONFIG_INFINIBAND)
those 2 left if someone needs it in a custom kernel.
Rest removed as suggested.

Loading...