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

[Calvin Walton <calvin.walton@kepstin.ca>]

Hi Steve,

On Mon, 2015-11-02 at 07:53 -0500, Steve Dickson wrote:
> Hello, 
> 
> I'm finally getting around to looking at this and
> I'm seen a couple issues...

> > -unit_dir = /usr/lib/systemd/system
> > -
> > -if INSTALL_SYSTEMD
> > -install-data-hook: $(unit_files)
> But after making the patch apply cleanly the scripts
> no longer get installed and I believe its 
> because you took out the above rule. 

You had actually ran into this same issue when you reviewed the patch
earlier. This rule isn't needed because automake's built-in install
path functionality is used - but the files only get installed if
  --with-systemd=/some/path
is passed to configure. (If you only use --with-systemd, you might find
the units get installed to the directory "/yes")

The conclusion was that some patching of configure was needed - it
should enable installing the unit files by default if systemd is found,
and not require manually specifying the path where to install the units
(using e.g. pkg-config to find it if unspecified)

I should have done this earlier, but was busy with some other stuff for
work.

I'm ok with autoconf stuff, so I'll take a look at rolling an updated
patch series with the configure changes. Hopefully it'll get done
before several months have passed...

-- 
Calvin Walton <calvin.walton@kepstin.ca>