[ldapvi] Patch for FreeBSD build

Ulrich Spoerlein uspoerlein at gmail.com
Sun Jul 15 16:42:52 CEST 2007


On Sun, 15.07.2007 at 15:54:31 +0200, Ulrich Spoerlein wrote:
> Hi, attached you'll find a patch to the install target required on
> FreeBSD systems.
> 
> INSTALL_PROGRAM is usually defined as 'install -s', which will strip
> binaries. This of course fails on manpages or documentation. Therefore
> use the appropriate INSTALL_DATA to install these files (this also takes
> care of permissions).
> 
> I hope this patch can make it into the official tree, as that would ease
> the porting effort. Thanks!

Here's the patch:


--- GNUmakefile.in.orig	2007-07-15 15:25:07.653278406 +0200
+++ GNUmakefile.in	2007-07-15 15:26:01.574449285 +0200
@@ -27,8 +27,8 @@
 install: ldapvi
 	mkdir -p @bindir@ @mandir@/man1/ @prefix@/share/doc/ldapvi
 	@INSTALL_PROGRAM@ ldapvi @bindir@
-	@INSTALL_PROGRAM@ -m 644 ldapvi.1 @mandir@/man1/
-	@INSTALL_PROGRAM@ -m 644 manual/manual.css manual/manual.xml manual/bg.png manual/html.xsl @prefix@/share/doc/ldapvi
+	@INSTALL_DATA@ ldapvi.1 @mandir@/man1/
+	@INSTALL_DATA@ manual/manual.css manual/manual.xml manual/bg.png manual/html.xsl @prefix@/share/doc/ldapvi
 
 distclean:
 	rm -rf configure GNUmakefile config.h config.log autom4te.cache config.status

Cheers,
Ulrich Spoerlein
-- 
"The trouble with the dictionary is you have to know how the word is
spelled before you can look it up to see how it is spelled."
-- Will Cuppy



More information about the ldapvi mailing list