On Wed, 2012-03-21 at 09:02 -0700, Phil Dibowitz wrote: > On 03/21/2012 08:30 AM, Adam Williamson wrote: > > On Wed, 2012-03-21 at 09:23 -0400, Scott Talbert wrote: > >> On Tue, 20 Mar 2012, Phil Dibowitz wrote: > >> > >>> + $(install_sh_DATA) start_concordance_dhcpd.sh \ > >>> + $(DESTDIR)$(sysconfdir)/udev/ > >>> + $(install_sh_DATA) start_concordance_dhcpd_wrapper.sh \ > >>> + $(DESTDIR)$(sysconfdir)/udev/ > >>> > >>> These shouldn't go in /etc/udev - they're executable scripts. These > >>> should go > >>> in $sbindir. You'll of course need to update the udev rule itself as well > >>> when > >>> you move these. > >> > >> The reason for putting these in $(sysconfdir)/udev was so that udev could > >> find them without having to hard-code the path to them. This directory > >> seems to be the place where udev-related scripts get stored. If we move > > > > On my up-to-the-minute F17, where presumably stuff is in the location > > udev authors intend, such scripts seem to be in /usr/lib/udev (note: > > _not_ $libdir/udev - they're in /usr/lib/udev on 32-bit and 64-bit). > > Scott, good point. > > Adam, I think that's because Fedora has unified / and /usr... For me, they're > in /lib/udev which should be true for a wider variety of users. And Fedora > should have a symlink from /lib to /usr/lib.
Ah yeah, good catch. I just checked a pre-/usrmove machine and they are indeed in /lib/udev there. I definitely think sbindir is wrong, though, as they're not meant to be used interactively, they're helper scripts for a system component. /etc also seems wrong as they're not configuration. So /lib/udev seems like a good candidate to me. I think we had a big bikeshedding session about this on fedora-devel-list recently, where the use of /usr/libexec was universally excoriated as a fedora-ism but there was lots of exciting debate on where else this kind of thing should go. -- Adam Williamson Fedora QA Community Monkey IRC: adamw | Twitter: AdamW_Fedora | identi.ca: adamwfedora http://www.happyassassin.net ------------------------------------------------------------------------------ This SF email is sponsosred by: Try Windows Azure free for 90 days Click Here http://p.sf.net/sfu/sfd2d-msazure _______________________________________________ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
