That change looks wrong to me. tthis is to create a symlink for config files at $(templatedir) to $(configdir). that change just makes symlinks for $(CONF_LINKS) to $(DESTDIR)$(templatedir). it won't ends up to $(templatedir). that's why it changed current directory outside of the loop first. I guess they may use DESTDIR for another purpose? Anyway, it breaks the installation for generic packaging system. On Thu, Jul 24, 2014 at 12:57 AM, Behdad Esfahbod <behdad@xxxxxxxxxxxxxxxxxxxxxx> wrote: > conf.d/Makefile.am | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > New commits: > commit fd5667b42c253da9c4c5502f53b5c0fb7e0f589e > Author: Behdad Esfahbod <behdad@xxxxxxxxxx> > Date: Wed Jul 23 11:57:31 2014 -0400 > > Symlinks fix for DESTDIR > > From: > https://github.com/Alexpux/MINGW-packages/blob/master/mingw-w64-fontconfig/fontconfig-2.11.0-symlinks-fix.patch > > diff --git a/conf.d/Makefile.am b/conf.d/Makefile.am > index d61756b..2b2c85f 100644 > --- a/conf.d/Makefile.am > +++ b/conf.d/Makefile.am > @@ -88,9 +88,9 @@ install-data-hook: > @(echo cd $(DESTDIR)$(configdir); \ > cd $(DESTDIR)$(configdir); \ > for i in $(CONF_LINKS); do \ > - echo $(RM) $$i";" ln -s $(templatedir)/$$i .; \ > + echo $(RM) $$i";" ln -s $(DESTDIR)$(templatedir)/$$i .;\ > $(RM) $$i; \ > - ln -s $(templatedir)/$$i .; \ > + ln -s $(DESTDIR)$(templatedir)/$$i .; \ > done) > uninstall-local: > @(echo cd $(DESTDIR)$(configdir); \ > _______________________________________________ > Fontconfig mailing list > Fontconfig@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/fontconfig -- Akira TAGOH _______________________________________________ Fontconfig mailing list Fontconfig@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/fontconfig
