Re: adjustments for arm clang switch

On Sat, Jan 20 2018, Jonathan Gray <jsg@jsg.id.au> wrote:
> With armv7 switching to clang as the base compiler atomic builtins and
> -mfpu=neon are now available on arm.
>
> Other BROKEN markers are suspect as well
>
> devel/codeblocks/Makefile:BROKEN-arm= wxwidgets va_list c++ mangling requires gcc < 4.4
> devel/arm-none-eabi/gcc-linaro/Makefile:BROKEN-armv7= error during libgcc autoconf "checking for suffix of object files", probably similar to i386
> devel/reposurgeon/Makefile:BROKEN-arm= out of memory compiling cyreposurgeon.c
> games/crimson/Makefile:BROKEN-arm= mktileset buggy, loops at high CPU use
> games/stone-soup/Makefile:BROKEN-arm= tilegen.elf loops burning cpu
> games/fifengine/Makefile:BROKEN-arm = out of memory when compiling fifePYTHON_wrap.cxx
> lang/seed7/Makefile:BROKEN-arm = gmake: *** [makefile:279: ../bin/make7] Bus error (core dumped)
> security/john-jumbo/Makefile:BROKEN-arm = uaf_encode_plug.c:61: error: stray '$' in program
>
> Index: audio/audiality2/Makefile
> ===================================================================
> RCS file: /cvs/ports/audio/audiality2/Makefile,v
> retrieving revision 1.2
> diff -u -p -r1.2 Makefile
> --- audio/audiality2/Makefile 20 Dec 2017 20:26:33 -0000 1.2
> +++ audio/audiality2/Makefile 20 Jan 2018 00:48:15 -0000
> @@ -25,8 +25,6 @@ WANTLIB += c SDL2 m
>
> COMPILER = base-clang ports-gcc base-gcc
> COMPILER_LANGS = c
> -# __sync_bool_compare_and_swap_4
> -MODGCC4_ARCHS = arm

The intent here was to require ports-gcc on arm only. Please delete all
of COMPILER/COMPILER_LANGS/MODGCC4_ARCHS.

Except for racket-minimal which looks suspicious (see below), the rest
look good, ok jca@

> Index: lang/racket-minimal/Makefile
> ===================================================================
> RCS file: /cvs/ports/lang/racket-minimal/Makefile,v
> retrieving revision 1.43
> diff -u -p -r1.43 Makefile
> --- lang/racket-minimal/Makefile 2 Dec 2017 21:39:49 -0000 1.43
> +++ lang/racket-minimal/Makefile 20 Jan 2018 00:52:42 -0000
> @@ -38,7 +38,6 @@ EXTRACT_SUFX = .tgz
> # "places" and "futures" require TLS.
> COMPILER = base-clang ports-gcc
> COMPILER_LANGS = c
> -MODGCC4_ARCHS = arm

If you remove this line only, then mips64el, powerpc and sparc64 will
start using ports-gcc. I'm not sure what was intended. cc'ing
maintainer.

(The port uses ONLY_FOR_ARCHS:
ONLY_FOR_ARCHS = arm amd64 i386 mips64el powerpc sparc64)

>
> LIB_DEPENDS = converters/libiconv \
> databases/sqlite3 \

--
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE