]> git.ipfire.org Git - thirdparty/xz.git/commitdiff
Build: Fix the combination of --disable-xzdec --enable-lzmadec.
authorLasse Collin <lasse.collin@tukaani.org>
Fri, 25 Apr 2014 14:53:42 +0000 (17:53 +0300)
committerLasse Collin <lasse.collin@tukaani.org>
Sat, 26 Apr 2014 05:47:36 +0000 (08:47 +0300)
In this case "make install" could fail if the man page directory
didn't already exist at the destination. If it did exist, a
dangling symlink was created there. Now the link is omitted
instead. This isn't the best fix but it's better than the old
behavior.

src/xzdec/Makefile.am

index 2348e4e6ba6e2669b817c1663e3a7f8088ca925d..70746c0c34552bf27d193360ef7d0925a38ad6cf 100644 (file)
@@ -59,7 +59,14 @@ endif
 if COND_LZMADEC
 bin_PROGRAMS += lzmadec
 
-# FIXME: If xzdec is disabled, this will create a dangling symlink.
+# Create the symlink lzmadec.1->xzdec.1 only if xzdec.1 was installed.
+# This is better than creating a dangling symlink, especially
+# because creating the link may fail due to the directory being missing.
+#
+# FIXME: The correct solution would be to install xzdec.1 as lzmadec.1
+# but I don't know what is the sane way to do it and since this is a bit
+# unusual situation anyway, it's not that important.
+if COND_XZDEC
 install-data-hook:
        cd $(DESTDIR)$(mandir)/man1 && \
        target=`echo xzdec | sed '$(transform)'` && \
@@ -72,3 +79,4 @@ uninstall-hook:
        link=`echo lzmadec | sed '$(transform)'` && \
        rm -f $$link.1
 endif
+endif