On Tue, Mar 02, 2010 at 05:04:25PM +0300, Vitaly Kuznetsov wrote: > Iustin Pop <[email protected]> writes: > > > On Tue, Mar 02, 2010 at 04:43:26PM +0300, Vitaly Kuznetsov wrote: > >> Using absolute path for a privileged command is a bad idea as this path > >> may vary. > >> For example /usr/sbin/brctl in Debian and /sbin/brctl in ALTLinux. Using > >> $PATH is > >> a better idea. > > > > Thanks for the patch. What do you think about detecting the path to > > these commands at ./configure time and using that value, hardcoded? > > E.g. using AC_PATH_PROG/AC_CHECK_PROG. This would also allow checking > > of needed programs at configure time. > > I was thinking about this way and found it not very good: all these > programs must present in build environment (and they are redundant for > building).
Indeed, you're right. I git-am'd your patch and will push it shortly to the devel-2.1 branch. thanks! iustin
