[Gross] Makefile cleanup for installing grossd.conf

Nuutti Kotivuori naked at iki.fi
Tue Apr 15 14:26:19 EEST 2008


Hi,

I found this bit in the Makefile.am:

  install-data-local:
          if test ! -f $(sysconfdir)/grossd.conf; then $(INSTALL_DATA) $(srcdir)/doc/examples/grossd.conf $(sysconfdir); fi

As a minor point, I dislike the test for existence of grossd.conf.
Sure, that's handy when testing and developing, but those sorts of
things in the Makefile make things hard for the package maintainers -
and can confuse some users.

But the real problem is that the commandline doesn't take in to
account $(DESTDIR).

It should be:

  install-data-local:
          if test ! -f $(DESTDIR)$(sysconfdir)/grossd.conf; then $(INSTALL_DATA) $(srcdir)/doc/examples/grossd.conf $(DESTDIR)$(sysconfdir); fi

Or just the same without the test for existence before.

Without this fix, installing with DESTDIR causes probably a permission
error from the system install location - and in any case the
configuration file lands up outside the directory specified.

-- Naked



More information about the Gross mailing list