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 
the files outside of this directory we'll have to hard-code the path to 
the script, so it will be a little trickier.

> I also wonder if these files should be installed by default. Just because I
> don't use udev to kick them off doesn't mean I don't want this.

Well, I'm not sure how useful they are without udev.  They depend on 
certain environment variables being set ($ACTION, $INTERFACE, at least) so 
if someone wanted to use them without udev, they'd have to manually set 
those.

------------------------------------------------------------------------------
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

Reply via email to