GNOME Bugzilla – Bug 552122
Add PLD groups and services
Last modified: 2009-08-16 17:35:50 UTC
Attached are current patches. This bug is part of pushing PLD patches upstream.
Created attachment 118659 [details] [review] The current patch we use for groups
Created attachment 118660 [details] [review] The current patch we use for services
*ping*
Carlos: ping
Ah. g-s-t is unmaintained: http://blogs.gnome.org/carlosg/2008/02/01/two-new-children-in-cookieville/
Sorry for the delay, but could you elaborate a little more about that patch? I perfectly see how this could be useful, but are other distributions supposed to follow those group policies too? e.g. in Ubuntu the usb and cdwrite groups don't exist. Same for services. If we add these, we need to make all downstream know that they should use it - else we add options that will mislead users. Thanks!
I thought the point of having that list was being able to detect interesting groups among those present in the system, not creating all of them on each and every distro.
Good point! I'm not very familiar yet with all the internals of the g-s-t. If it works just like boot services are detected, so we can and should upstream that patch. BTW, if you have other patches that should go the same way, please file bugs! I've prepared a commit with your patch updated, but I'd like a precision about alphabetical order: why is NetworkManager at the top, and not with the 'n'? should really alsasound go after alsa-utils (I don't know how the - is sorted)? Since you tested it, I guess you know better than I do. Thanks!
Ping?
About ordering: Not sure about the order, I'm used to ASCII ordering in source code. In the GUI we should let GTK+ order items by translation anyway.
OK, I've reordered them using the 'sort' command, that should be more correct. Pushed as 6743464cb8e1aed196afe5b07d876bfe50990eb7. Thanks again!