On Mon, Apr 23 2018, <[email protected]> wrote:
> Picking up on your last comment, I restarted the php build on arm with the 
> intent of looking for why gcc is required.  So far I have come across one 
> package that pulls in gcc and it is specific to arm.
>
> Cmake pulls in libuv whose Makefile has:
>
> # for atomic builtins
> MODULES =       gcc4
> MODGCC4_ARCHS = arm
>
> This is what starts off the gcc build.  Can and should these lines be 
> replaced with a clang equivalent?

Probably,

> To keep the build going I took out these two items and the cmake build 
> completed without  gcc.

so the diff below is a candidate for a commit.  If clang didn't provide
those atomics, linking cmake should have failed because of unresolved
references.  I won't have access to my arm box until tomorrow, but this
looks safe.

ok?


Index: Makefile
===================================================================
RCS file: /cvs/ports/devel/libuv/Makefile,v
retrieving revision 1.4
diff -u -p -p -u -r1.4 Makefile
--- Makefile    11 Feb 2018 18:31:35 -0000      1.4
+++ Makefile    23 Apr 2018 18:16:37 -0000
@@ -5,6 +5,8 @@ COMMENT =       multi-platform library for asy
 VER =          1.19.1
 DISTNAME =     libuv-v${VER}
 PKGNAME =      libuv-${VER}
+REVISION =     0
+
 CATEGORIES =   devel
 
 SHARED_LIBS =  uv 1.0  # 1.0
@@ -17,10 +19,6 @@ MASTER_SITES =       http://dist.libuv.org/dis
 PERMIT_PACKAGE_CDROM = Yes
 
 WANTLIB += pthread
-
-# for atomic builtins
-MODULES =      gcc4
-MODGCC4_ARCHS =        arm
 
 BUILD_DEPENDS =        devel/libtool \
                ${MODGNU_AUTOCONF_DEPENDS} \


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

Reply via email to