[Gross] Makefile cleanup for installing grossd.conf

Eino Tuominen eino at utu.fi
Tue Apr 15 15:01:18 EEST 2008


Nuutti Kotivuori wrote:
> 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.

Can you give me an example, please. I have thought that overwriting 
existing config would make things even more confusing, when upgrading.

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

Thats a bug, thanks for pointing it out! I'll fix it shortly.

-- 
   Eino Tuominen




More information about the Gross mailing list