Re: [clang6] fix: net/nam

2018-05-17 Thread Stuart Cassoff
Thanks for a lot fixing this and tclcl!
Not sure if I could've fixed it myself; I don't really know C++.
Apologies; I haven't had much time lately for ports, hoping that'll change soon.

Stu


> -- Original Message --
> From: Ingo Schwarze 
> Date: April 16, 2018 at 7:19 PM
> 
> 
> Hi,
> 
> i already committed a fix that doesn't require
> forcing an old language standard.
> 
> Yours,
>   Ingo
> 
> 
> Giovanni Bechis wrote on Tue, Apr 17, 2018 at 12:36:12AM +0200:
> 
> > clang6 fix for net/nam, still a lot of warnings but at least it builds.
> 
> > Index: Makefile
> > ===
> > RCS file: /var/cvs/ports/net/nam/Makefile,v
> > retrieving revision 1.40
> > diff -u -p -r1.40 Makefile
> > --- Makefile26 Jul 2017 22:45:28 -  1.40
> > +++ Makefile16 Apr 2018 22:30:01 -
> > @@ -35,6 +35,16 @@ CONFIGURE_ARGS=  --with-tcl-ver=${MODTCL_
> > --x-includes=${X11BASE}/include \
> > --x-libraries=${X11BASE}/lib
> >  
> > +MAKE_ENV=  CC="${CC}" \
> > +   CFLAGS="${CFLAGS}" \
> > +   CPPFLAGS="${CPPFLAGS}" \
> > +   CXXFLAGS="${CXXFLAGS}"
> > +
> > +.include 
> > +.if ${PROPERTIES:Mclang}
> > +CXXFLAGS +=-std=c++03
> > +.endif
> > +
> >  ALL_TARGET=
> >  
> >  NO_TEST=   Yes
> > Index: patches/patch-Makefile_in
> > ===
> > RCS file: patches/patch-Makefile_in
> > diff -N patches/patch-Makefile_in
> > --- /dev/null   1 Jan 1970 00:00:00 -
> > +++ patches/patch-Makefile_in   16 Apr 2018 22:30:01 -
> > @@ -0,0 +1,14 @@
> > +$OpenBSD$
> > +
> > +Index: Makefile.in
> > +--- Makefile.in.orig
> >  Makefile.in
> > +@@ -83,7 +83,7 @@ RM   = rm -f
> > + .SUFFIXES: .cc $(.SUFFIXES)
> > + 
> > + .cc.o:
> > +-  rm -f $@; $(CPP) -o $@ -c $(CFLAGS) $*.cc
> > ++  rm -f $@; $(CPP) -o $@ -c $(CXXFLAGS) $(CFLAGS) $*.cc
> > + 
> > + .c.o:
> > +   rm -f $@; $(CC) -o $@ -c $(CFLAGS) $*.c
>



Re: [clang6] fix: net/nam

2018-04-16 Thread Ingo Schwarze
Hi,

i already committed a fix that doesn't require
forcing an old language standard.

Yours,
  Ingo


Giovanni Bechis wrote on Tue, Apr 17, 2018 at 12:36:12AM +0200:

> clang6 fix for net/nam, still a lot of warnings but at least it builds.

> Index: Makefile
> ===
> RCS file: /var/cvs/ports/net/nam/Makefile,v
> retrieving revision 1.40
> diff -u -p -r1.40 Makefile
> --- Makefile  26 Jul 2017 22:45:28 -  1.40
> +++ Makefile  16 Apr 2018 22:30:01 -
> @@ -35,6 +35,16 @@ CONFIGURE_ARGS=--with-tcl-ver=${MODTCL_
>   --x-includes=${X11BASE}/include \
>   --x-libraries=${X11BASE}/lib
>  
> +MAKE_ENV=CC="${CC}" \
> + CFLAGS="${CFLAGS}" \
> + CPPFLAGS="${CPPFLAGS}" \
> + CXXFLAGS="${CXXFLAGS}"
> +
> +.include 
> +.if ${PROPERTIES:Mclang}
> +CXXFLAGS +=  -std=c++03
> +.endif
> +
>  ALL_TARGET=
>  
>  NO_TEST= Yes
> Index: patches/patch-Makefile_in
> ===
> RCS file: patches/patch-Makefile_in
> diff -N patches/patch-Makefile_in
> --- /dev/null 1 Jan 1970 00:00:00 -
> +++ patches/patch-Makefile_in 16 Apr 2018 22:30:01 -
> @@ -0,0 +1,14 @@
> +$OpenBSD$
> +
> +Index: Makefile.in
> +--- Makefile.in.orig
>  Makefile.in
> +@@ -83,7 +83,7 @@ RM = rm -f
> + .SUFFIXES: .cc $(.SUFFIXES)
> + 
> + .cc.o:
> +-rm -f $@; $(CPP) -o $@ -c $(CFLAGS) $*.cc
> ++rm -f $@; $(CPP) -o $@ -c $(CXXFLAGS) $(CFLAGS) $*.cc
> + 
> + .c.o:
> + rm -f $@; $(CC) -o $@ -c $(CFLAGS) $*.c