Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 14 Jun 2017 11:15:12 +0200
From:      Koop Mast <kwm@FreeBSD.org>
To:        Tijl Coosemans <tijl@FreeBSD.org>, ports-committers@freebsd.org,  svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r443499 - head/Mk/Uses
Message-ID:  <1497431712.62091.0.camel@FreeBSD.org>
In-Reply-To: <201706131328.v5DDSnfs097665@repo.freebsd.org>
References:  <201706131328.v5DDSnfs097665@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 2017-06-13 at 13:28 +0000, Tijl Coosemans wrote:
> Author: tijl
> Date: Tue Jun 13 13:28:49 2017
> New Revision: 443499
> URL: https://svnweb.freebsd.org/changeset/ports/443499
> 
> Log:
>   Remove check for unsupported versions of FreeBSD and add some
> comments.
>   
>   Submitted by:	John Hein <john.hein@microsemi.com>
> 
> Modified:
>   head/Mk/Uses/iconv.mk
> 
> Modified: head/Mk/Uses/iconv.mk
> =====================================================================
> =========
> --- head/Mk/Uses/iconv.mk	Tue Jun 13 10:44:52 2017	(r44
> 3498)
> +++ head/Mk/Uses/iconv.mk	Tue Jun 13 13:28:49 2017	(r44
> 3499)
> @@ -41,13 +41,12 @@ ICONV_CONFIGURE_BASE=
>  ICONV_INCLUDE_PATH=	/usr/include
>  ICONV_LIB_PATH=		/usr/lib/libc.so
>  
> -.if (${OPSYS} == FreeBSD && \
> - (${OSVERSION} >= 1100000 && ${OSVERSION} < 1100069)) \
> - || exists(${LOCALBASE}/include/iconv.h)
> +.if exists(${LOCALBASE}/include/iconv.h)
> +# Check that libiconv iconv.h is recent enough for LIBICONV_PLUG to
> work.
>  BUILD_DEPENDS+=	libiconv>=1.14_9:converters/libiconv
> -ICONV_INCLUDE_PATH=	${LOCALBASE}/include
>  .endif
>  
> +# LIBICONV_PLUG makes libiconv iconv.h act like libc iconv.h.
>  CPPFLAGS+=	-DLIBICONV_PLUG
>  CFLAGS+=	-DLIBICONV_PLUG
>  CXXFLAGS+=	-DLIBICONV_PLUG
> 

Hi,

Was it intended to remove the OSSYS check also? That probably going to
break dragonfly.

-Koop



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