Re: [PATCH] nfs-utils: Substitute sbindir in systemd unit files.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2015-07-14 at 10:32 -0400, Steve Dickson wrote:
> Hello Calivn,
> 
> I'm finally getting back to take a look at this...
> 
> I see what your are trying to do... Make the systemd scripts
> use @sbindir@ instead of a hard coded '/usr/sbin/' which 
> is fine. To do this you are renaming all the systemd 
> scripts to .in systemd scripts. 

Not all of them, there's a few I left untouched because they didn't
need any directory substitutions.

> But what I'm missing is what actually generates the 
> systemd scripts with the correct path names?

At the bottom of the systemd/Makefile.am file, a rule is added to
generate a ".service" file from a ".service.in" file, which gets called
automatically by make to generate the files listed in the unit_DATA
variable.

> Since you took out the install-data-hook: rule from
> the Makefile.am file, when I do a make install none
> of the scripts get installed.... 

Hmm. I took a look at this; the files don't get installed unless you
pass "--with-systemd=/usr/lib/systemd/system" (or the equivalent path
on your system). In particular, using "--with-systemd" with no
parameter won't work, since they'll get installed to the directory
"yes" (i.e. the value of the --with-systemd parameter).

This is the same behaviour as before my patch, but it could be improved
by reworking the configure script a bit (in particular, using pkg
-config to ask for the system default systemd unit directory).

Do you still want the patches split up between the Makefile changes and
each individual modified unit file?

-- 
Calvin Walton <calvin.walton@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux