From 4f3907899811992460202f96427ddc9d4c0f76b5 Mon Sep 17 00:00:00 2001 From: nirenjan Date: Fri, 7 Aug 2020 22:57:58 -0700 Subject: [PATCH] Fix install error on macOS when doxygen is found The `install` command on macOS does not support the `-D` flag. It is not required anyway, since the directory components are created by the previous line. This commit also updates the options to rsync to not copy in 'archive' mode. Because it is used in `make install`, it is likely to be run as root, and therefore, the root user should own the files in the destination directory. --- Makefile.am | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index 32383cb..2d85255 100644 --- a/Makefile.am +++ b/Makefile.am @@ -43,10 +43,10 @@ clean-local: # Install Doxygen generated HTML documentation and manpages install-data-local: $(INSTALL) -d $(DESTDIR)$(docdir) - rsync -a $(top_builddir)/docs/html $(DESTDIR)$(docdir) + rsync -rlp $(top_builddir)/docs/html $(DESTDIR)$(docdir) $(INSTALL) -d $(DESTDIR)$(mandir)/man1 - $(INSTALL) -D docs/man/man1/x52cli.1 $(DESTDIR)$(mandir)/man1 + $(INSTALL) docs/man/man1/x52cli.1 $(DESTDIR)$(mandir)/man1 uninstall-local: rm -rf $(DESTDIR)$(docdir)