[clang6] fix: net/nam

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[clang6] fix: net/nam

Giovanni Bechis-7
Hi,
clang6 fix for net/nam, still a lot of warnings but at least it builds.
 ok ? Comments ?
  Cheers
   Giovanni

nam-1.15p1-clang6.diff (1K) Download Attachment
signature.asc (817 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [clang6] fix: net/nam

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 -0000 1.40
> +++ Makefile 16 Apr 2018 22:30:01 -0000
> @@ -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 <bsd.port.arch.mk>
> +.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 -0000
> +++ patches/patch-Makefile_in 16 Apr 2018 22:30:01 -0000
> @@ -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

Reply | Threaded
Open this post in threaded view
|

Re: [clang6] fix: net/nam

Stuart Cassoff-4
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 <[hidden email]>
> 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
> > --- Makefile 26 Jul 2017 22:45:28 -0000 1.40
> > +++ Makefile 16 Apr 2018 22:30:01 -0000
> > @@ -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 <bsd.port.arch.mk>
> > +.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 -0000
> > +++ patches/patch-Makefile_in 16 Apr 2018 22:30:01 -0000
> > @@ -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
>