Created attachment 188212 [details] i386 and FreeBSD 10 build fix This patch removes the hardcoded x64 gyp build paths and replaces them with architecture specific ones. It also mandates clang 4.0 on FreeBSD 10.
I'm testing builds with latest diff in Phabricator right now.
Created attachment 188357 [details] Cleaned up patch
(In reply to Nikolai Lifanov from comment #1) Thank you! I've just uploaded a reviewed patch.
Created attachment 188359 [details] build failure on 10.4-RELEASE i386 This still doesn't build for me. Please see the attached log.
(In reply to Nikolai Lifanov from comment #4) Does the build only fail on i386? That might require us to ONLY_FOR_ARCHS it to amd64.
Created attachment 188360 [details] i386 build fix Could you please try this one a spin? Thanks.
Can you take out the patches you are not changing from the diff? I'm going to do this manually and retest.
When you initially submitted the port, you listed galu@packetdam.com as the MAINTAINER. I'll need to add a maintainer to "Approved by:" line when committing the change and it's easy to do when Bugzilla email address matches the MAINTAINER address. This is you, right?
Created attachment 188363 [details] build failure on 10.4-RELEASE i386 The latest patch still fails for me. Please see the updated build log.
(In reply to Nikolai Lifanov from comment #8) Yes, that's me, thanks.
OK, awesome! But the last patch still doesn't build for me. I attached the last build log.
Created attachment 188366 [details] Restrict the build to amd64
(In reply to Nikolai Lifanov from comment #11) On it, thanks for bearing with me. Could you please try the latest patch? I've tried a build in a 32bit jail but there is quite some amount of work to get it to build cleanly.
Created attachment 188367 [details] Really restrict the build to amd64
Created attachment 188368 [details] Clean patch
Hi Nikolai, would you mind giving the latest patch a try? I think that should fix the failing builds. Thanks!
Hi! I'm already trying it. I'm waiting for the builds to finish.
A commit references this bug: Author: lifanov Date: Fri Dec 1 16:29:36 UTC 2017 New revision: 455300 URL: https://svnweb.freebsd.org/changeset/ports/455300 Log: fix databases/arangodb32 on FreeBSD 10.x o Fix build on FreeBSD 10.x by requiring recent LLVM o Only build on amd64 - many hardcoded directories PR: 223815 Submitted by: galu@packetdam.com (maintainer, same as dev@dudu.ro) Differential Revision: https://reviews.freebsd.org/D13282 Changes: head/databases/arangodb32/Makefile
Committed, thanks!