On Tue, Apr 10 2018, Edd Barrett <e...@theunixzoo.co.uk> wrote:
> On Sun, Apr 08, 2018 at 11:21:51PM +0200, Jeremie Courreges-Anglas wrote:
>> -std=c++03 isn't supported by g++-4.2.1; do you need the workaround on
>> clang archs only?
>
> I think you are right. How about this:

While it would in most cases in practice, I think the test should be
driven by CHOSEN_COMPILER.  But since many other ports seem to use this
idiom anyway, please go ahead with your diff.   ok jca@

> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/emulators/fs-uae/Makefile,v
> retrieving revision 1.23
> diff -u -p -r1.23 Makefile
> --- Makefile  21 Nov 2017 00:13:00 -0000      1.23
> +++ Makefile  10 Apr 2018 19:41:54 -0000
> @@ -9,7 +9,7 @@ V =                   2.4.0
>  MODPY_EGG_VERSION =  ${V}
>  DISTNAME =           fs-uae-$V
>  CATEGORIES =         emulators
> -REVISION =           4
> +REVISION =           5
>  
>  HOMEPAGE =           http://fengestad.no/fs-uae/
>  MAINTAINER =         Edd Barrett <e...@openbsd.org>
> @@ -17,7 +17,7 @@ MAINTAINER =                Edd Barrett <edd@openbsd.o
>  # GPLv2, LGPLv2.1
>  PERMIT_PACKAGE_CDROM =       Yes
>  
> -WANTLIB += GL GLU SDL X11 c freetype glib-2.0 gthread-2.0 iconv intl m
> +WANTLIB += GL GLU SDL X11 c freetype glib-2.0 gthread-2.0 intl m
>  WANTLIB += openal png pthread ${COMPILER_LIBCXX} z
>  
>  MASTER_SITES =               ${HOMEPAGE}stable/$V/
> @@ -31,6 +31,11 @@ RUN_DEPENDS =              devel/desktop-file-utils 
>                       x11/gtk+3,-guic
>  
>  USE_GMAKE =          Yes
> +
> +.include <bsd.port.arch.mk>
> +.if ${PROPERTIES:Mclang}
> +CXXFLAGS =           -std=c++03
> +.endif
>  
>  MAKE_FLAGS +=                prefix=${PREFIX} CXXFLAGS="${CXXFLAGS}"
>  MAKE_FLAGS +=                CFLAGS="${CFLAGS}" CXX="${CXX}"

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Reply via email to