man install procedure fix

View: New views
2 Messages — Rating Filter:   Alert me  

man install procedure fix

by Ciprian Dorin Craciun :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

    Hello all!

    I found a little bug in the rsync v3.0.2 install, that is if you
separate the build (output) folder from the source folder, the man
files fail to install because they are generated in the output folder,
but the install target searches them in the source folder.
    I attach a small patch that solves this problem.

    Thanks all,
    Ciprian Craciun.

    P.S.: I'm not on the mailing list, so please keep me in CC.

[man-install.patch]

diff --git a/Makefile.in b/Makefile.in
index b91b3e9..4235eef 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -66,8 +66,8 @@ install: all
  ${INSTALLCMD} ${INSTALL_STRIP} -m 755 rsync$(EXEEXT) ${DESTDIR}${bindir}
  -mkdir -p ${DESTDIR}${mandir}/man1
  -mkdir -p ${DESTDIR}${mandir}/man5
- if test -f $(srcdir)/rsync.1; then ${INSTALLMAN} -m 644 $(srcdir)/rsync.1 ${DESTDIR}${mandir}/man1; fi
- if test -f $(srcdir)/rsyncd.conf.5; then ${INSTALLMAN} -m 644 $(srcdir)/rsyncd.conf.5 ${DESTDIR}${mandir}/man5; fi
+ if test -f rsync.1; then ${INSTALLMAN} -m 644 rsync.1 ${DESTDIR}${mandir}/man1; fi
+ if test -f rsyncd.conf.5; then ${INSTALLMAN} -m 644 rsyncd.conf.5 ${DESTDIR}${mandir}/man5; fi
 
 install-strip:
  $(MAKE) INSTALL_STRIP='-s' install


--
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html

Re: man install procedure fix

by Wayne Davison-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Sun, Jun 22, 2008 at 10:35:07AM +0300, Ciprian Dorin Craciun wrote:
>     I found a little bug in the rsync v3.0.2 install, that is if you
> separate the build (output) folder from the source folder, the man
> files fail to install [...]

Thanks.  The latest source has this fixed.

..wayne..
--
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html
LightInTheBox - Buy quality products at wholesale price