Brian Callahan writes:
>
> On 04/15/18 15:37, Klemens Nanni wrote:
> > On Sun, Apr 15, 2018 at 02:56:41PM -0400, Brian Callahan wrote:
> >> Index: Makefile
> >> ===================================================================
> >> RCS file: /cvs/ports/games/ja2-stracciatella/Makefile,v
> >> retrieving revision 1.3
> >> diff -u -p -r1.3 Makefile
> >> --- Makefile 26 Jul 2017 22:45:21 -0000 1.3
> >> +++ Makefile 15 Apr 2018 18:50:40 -0000
> >> @@ -27,7 +27,7 @@ RUN_DEPENDS = x11/gtk+3,-guic \
> >>
> >> USE_GMAKE = Yes
> >> MAKE_FLAGS = Q= WITH_UNITTESTS=0 WITH_DEBUGINFO=1 \
> >> - CXX=c++
> >> + CXX="${CXX}"
> >>
> >> NO_TEST = Yes
> >>
> >> Index: patches/patch-Makefile
> >> ===================================================================
> >> RCS file: /cvs/ports/games/ja2-stracciatella/patches/patch-Makefile,v
> >> retrieving revision 1.1.1.1
> >> diff -u -p -r1.1.1.1 patch-Makefile
> >> --- patches/patch-Makefile 13 Jun 2017 09:24:23 -0000 1.1.1.1
> >> +++ patches/patch-Makefile 15 Apr 2018 18:50:40 -0000
> >> @@ -1,6 +1,7 @@
> >> $OpenBSD: patch-Makefile,v 1.1.1.1 2017/06/13 09:24:23 kirby Exp $
> >> ---- Makefile.orig Sun Sep 11 12:13:27 2016
> >> -+++ Makefile Wed Sep 21 14:30:42 2016
> >> +Index: Makefile
> >> +--- Makefile.orig
> >> ++++ Makefile
> >> @@ -3,15 +3,11 @@
> >> # Please update COMPILATION.txt if necessary after changing this file.
> >> #
> >> @@ -38,6 +39,15 @@ $OpenBSD: patch-Makefile,v 1.1.1.1 2017/
> >> CFLAGS += -I _build/lib-rapidjson
> >> CFLAGS += -I _build/lib-slog
> >> CFLAGS += -I _build/lib-smacker/libsmacker
> >> +@@ -155,7 +151,7 @@ CCFLAGS += -Werror-implicit-function-declaration
> >> + CCFLAGS += -Wimplicit-int
> >> + CCFLAGS += -Wmissing-prototypes
> >> +
> >> +-CXXFLAGS += $(CFLAGS)
> >> ++CXXFLAGS += $(CFLAGS) -std=gnu++98
> > Why not passing it through CFLAGS in MAKE_FLAGS as done with CXX?
>
> Because it ends up obliterating all the other CXXFLAGS, and compilation
> fails immediately.
I agree with the CXX fix. David Carlier already sent a diff that uses
MAKE_ENV to fix clang6. I would prefer that to setting CXXFLAGS, except
with -Wno-narrowing instead of -std.
MAKE_ENV = CXXFLAGS="${CXXFLAGS} -Wno-narrowing"
