Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 25 Mar 2014 13:20:54 +0000 (UTC)
From:      Tijl Coosemans <tijl@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r349139 - head/net/rsync
Message-ID:  <201403251320.s2PDKsEM000982@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: tijl
Date: Tue Mar 25 13:20:54 2014
New Revision: 349139
URL: http://svnweb.freebsd.org/changeset/ports/349139
QAT: https://qat.redports.org/buildarchive/r349139/

Log:
  - Bump PORTREVISION after the latest change to converters/libiconv because
    rsync uses locale_charset when the ICONV option is enabled. [1]
  - Replace CFLAGS+=-I... with CPPFLAGS+=-I... to fix warnings during
    configure about headers being accepted by the compiler but not the
    preprocessor.
  - Replace CONFIGUREDCFLAGS (guard against CFLAGS damage by MAKE_ENV)
    with CPPFLAGS.  The dist makefile contains CFLAGS=@CFLAGS@ so CFLAGS
    in MAKE_ENV isn't a problem.
  - Replace ac_cv_header_iconv_h=no with --disable-iconv-open.  The first
    disables HAVE_ICONV_H while the latter disables USE_ICONV_OPEN but both
    are checked in rsync.h:
    #if defined USE_ICONV_OPEN && defined HAVE_ICONV_H
  - Set ac_cv_search_libiconv_open=no when empty(ICONV_LIB).  This prevents
    detection of libiconv when the ICONV option is disabled, but it is also
    needed on FreeBSD 10 with libiconv installed to prevent linking with
    -liconv.
  - New LIB_DEPENDS syntax.
  - Move pre-configure to post-patch.
  - Remove malloc.h patching.
  - Replace MAKE with MAKE_CMD.  MAKE is the currently running make and
    MAKE_CMD is the make used to build the port.
  - Unmute installation commands.
  
  Reported by:	"Kenta S." <kentas@hush.com> [1]
  Approved by:	ehaupt

Modified:
  head/net/rsync/Makefile

Modified: head/net/rsync/Makefile
==============================================================================
--- head/net/rsync/Makefile	Tue Mar 25 13:10:12 2014	(r349138)
+++ head/net/rsync/Makefile	Tue Mar 25 13:20:54 2014	(r349139)
@@ -3,7 +3,7 @@
 
 PORTNAME=	rsync
 PORTVERSION=	3.1.0
-PORTREVISION=	1
+PORTREVISION=	2
 CATEGORIES=	net ipv6
 MASTER_SITES=	http://rsync.samba.org/ftp/%SUBDIR%/ \
 		ftp://ftp.samba.org/pub/%SUBDIR%/ \
@@ -71,18 +71,14 @@ EXTRA_PATCHES+=	${WRKSRC}/patches/atimes
 
 .if ${PORT_OPTIONS:MICONV} || ${PORT_OPTIONS:MFLAGS} || ${PORT_OPTIONS:MACL} || make(makesum) || ${PORT_OPTIONS:MTIMELIMIT} || ${PORT_OPTIONS:MATIMES} || ${PORT_OPTIONS:MRENAMED} || ${PORT_OPTIONS:MRENFL} 
 DISTFILES+=	${PORTNAME}-patches-${PORTVERSION}${EXTRACT_SUFX}
-CFLAGS+=	-I${LOCALBASE}/include
 .endif
 
 .if ${PORT_OPTIONS:MICONV}
 USES+=		iconv
-CONFIGUREDCFLAGS+=	-I${LOCALBASE}/include
-LDFLAGS+=		-L${LOCALBASE}/lib
-CONFIGURE_ENV+=	CONFIGUREDCFLAGS="${CONFIGUREDCFLAGS}"
+CPPFLAGS+=	-I${LOCALBASE}/include
+LDFLAGS+=	-L${LOCALBASE}/lib
 .else
-CONFIGURE_ARGS+=	--disable-iconv
-CONFIGURE_ENV+=		ac_cv_search_libiconv_open=no
-CONFIGURE_ENV+=		ac_cv_header_iconv_h=no
+CONFIGURE_ARGS+=--disable-iconv --disable-iconv-open
 .endif
 
 .if ${PORT_OPTIONS:MFLAGS}
@@ -111,9 +107,9 @@ EXTRA_PATCHES+=	${WRKSRC}/patches/acls.d
 .endif
 
 .if ${PORT_OPTIONS:MPOPT_PORT}
-LIB_DEPENDS+=	popt:${PORTSDIR}/devel/popt
-CONFIGURE_ENV+=	LIBS="-L${LOCALBASE}/lib"
-CFLAGS+=	-I${LOCALBASE}/include
+LIB_DEPENDS+=	libpopt.so:${PORTSDIR}/devel/popt
+CPPFLAGS+=	-I${LOCALBASE}/include
+LDFLAGS+=	-L${LOCALBASE}/lib
 .else
 CONFIGURE_ARGS+=	--with-included-popt
 .endif
@@ -124,25 +120,24 @@ CONFIGURE_ARGS+=	--with-rsh=ssh
 CONFIGURE_ARGS+=	--with-rsh=rsh
 .endif
 
-# guard against CFLAGS damage by MAKE_ENV
-post-patch:
-	@${REINPLACE_CMD} -e 's|CFLAGS|CONFIGUREDCFLAGS|' \
-		-e 's|perl.*|${DO_NADA}|' \
-			${WRKSRC}/Makefile.in
+.include <bsd.port.pre.mk>
+
+.if empty(ICONV_LIB)
+CONFIGURE_ARGS+=ac_cv_search_libiconv_open=no
+.endif
 
-pre-configure:
+post-patch:
+	@${REINPLACE_CMD} -e 's|perl.*|${DO_NADA}|' ${WRKSRC}/Makefile.in
 	@${REINPLACE_CMD} -e 's:/etc/:${PREFIX}/etc/:g'  \
 		${WRKSRC}/rsync.h ${WRKSRC}/rsync.1 ${WRKSRC}/rsyncd.conf.5
-	@${REINPLACE_CMD} -e 's|malloc.h|stdlib.h|g' \
-		${WRKSRC}/rsync.h ${WRKSRC}/zlib/zutil.h
 
 post-install:
 	@${STRIP_CMD} ${STAGEDIR}${PREFIX}/bin/rsync
-	@${INSTALL_DATA} ${FILESDIR}/rsyncd.conf.sample ${STAGEDIR}${PREFIX}/etc/
+	${INSTALL_DATA} ${FILESDIR}/rsyncd.conf.sample ${STAGEDIR}${PREFIX}/etc/
 	@${MKDIR} ${STAGEDIR}${DOCSDIR}
-	@${INSTALL_DATA} ${PORTDOCS:S,^,${WRKSRC}/,} ${STAGEDIR}${DOCSDIR}
+	${INSTALL_DATA} ${PORTDOCS:S,^,${WRKSRC}/,} ${STAGEDIR}${DOCSDIR}
 
 test: build
-	@cd ${WRKSRC} && ${MAKE} check
+	@(cd ${WRKSRC} && ${MAKE_CMD} check)
 
-.include <bsd.port.mk>
+.include <bsd.port.post.mk>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201403251320.s2PDKsEM000982>