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

Reply via email to