On Fri, Oct 06 2017, Peter Hessler <phess...@openbsd.org> wrote: > On 2017 Oct 06 (Fri) at 13:09:02 +0200 (+0200), Christian Weisgerber wrote: > :Port build failures on amd64 -current after the clang 5.0 update: > : > :devel/valgrind > : > :valgrind errors out in configure because it doesn't recognize the > :compiler version: > :configure: error: please use gcc >= 3.0 or clang >= 2.9 > > Only slightly tested diff for valgrind, OK?
Well, if it builds, please ship it. No need for a REVISION bump. I see you added clang-6.*, I think you could just as well make it clang-*... > > Index: devel/valgrind/Makefile > =================================================================== > RCS file: /cvs/ports/devel/valgrind/Makefile,v > retrieving revision 1.16 > diff -u -p -u -p -r1.16 Makefile > --- devel/valgrind/Makefile 26 May 2017 22:07:16 -0000 1.16 > +++ devel/valgrind/Makefile 6 Oct 2017 12:14:50 -0000 > @@ -9,7 +9,7 @@ CATEGORIES = devel > > V = 3.10.1 > PV = 20160331 > -REVISION = 11 > +REVISION = 12 > DISTNAME = valgrind-${V} > EXTRACT_SUFX = .tar.bz2 > > Index: devel/valgrind/patches/patch-configure_ac > =================================================================== > RCS file: devel/valgrind/patches/patch-configure_ac > diff -N devel/valgrind/patches/patch-configure_ac > --- /dev/null 1 Jan 1970 00:00:00 -0000 > +++ devel/valgrind/patches/patch-configure_ac 6 Oct 2017 12:14:50 -0000 > @@ -0,0 +1,14 @@ > +$OpenBSD$ > + > +Index: configure.ac > +--- configure.ac.orig > ++++ configure.ac > +@@ -145,7 +145,7 @@ case "${is_clang}-${gcc_version}" in > + notclang-5.*) > + AC_MSG_RESULT([ok (${gcc_version})]) > + ;; > +- clang-2.9|clang-3.*|clang-4.*) > ++ clang-2.9|clang-3.*|clang-4.*|clang-5.*|clang-6.*) > + AC_MSG_RESULT([ok (clang-${gcc_version})]) > + ;; > + *) -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE