Page MenuHomePhabricator

build: switch to using LLVM 17.x for macOS builds
ClosedPublic

Authored by Fabien on Fri, Nov 29, 15:29.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABCc1938796c7b2: build: switch to using LLVM 17.x for macOS builds
Summary
We need LLVM 17.x so that lld has -fixup_chains.

Backport of core#28880.

Depends on D17254.

Test Plan

Run the osx guix build.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR28880
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 31417
Build 62332: Build Diff
Build 62331: arc lint + arc unit

Event Timeline

Fabien requested review of this revision.Fri, Nov 29, 15:29
Fabien planned changes to this revision.Fri, Nov 29, 15:29

@bot guix-osx

Tail of the build log:

checking for library containing opendir... none required
checking execinfo.h usability... yes
checking execinfo.h presence... yes
checking for execinfo.h... yes
checking sys/select.h usability... yes
checking sys/select.h presence... yes
checking for sys/select.h... yes
checking sys/socket.h usability... yes
checking sys/socket.h presence... yes
checking for sys/socket.h... yes
checking sys/time.h usability... yes
checking sys/time.h presence... yes
checking for sys/time.h... yes
checking for struct stat.st_blksize... yes
checking for inttypes.h... (cached) yes
checking for stdint.h... yes
checking if stdint.h can be used by C++... yes
checking stddef.h usability... yes
checking stddef.h presence... yes
checking for stddef.h... yes
checking for unistd.h... (cached) yes
checking size of char... 1
checking size of unsigned char... 1
checking size of short... 2
checking size of unsigned short... 2
checking size of int... 4
checking size of unsigned int... 4
checking size of long... 8
checking size of unsigned long... 8
checking size of long long... 8
checking size of unsigned long long... 8
checking size of char *... 8
checking for u_char... yes
checking for u_short... yes
checking for u_int... yes
checking for u_long... yes
checking for u_int8_t... yes
checking for u_int16_t... yes
checking for int16_t... yes
checking for u_int32_t... yes
checking for int32_t... yes
checking for u_int64_t... yes
checking for int64_t... yes
checking for FILE *... yes
checking for off_t... yes
checking for pid_t... yes
checking for size_t... yes
checking for time_t... yes
checking size of size_t... 8
checking for ssize_t... yes
checking for uintmax_t... yes
checking for uintptr_t... yes
checking for socklen_t... yes
checking for ANSI C exit success/failure values... yes
checking for getopt optreset variable... yes
checking for mutexes... UNIX/fcntl
configure: error: Support for FCNTL mutexes was removed in BDB 4.8.
make: *** [funcs.mk:297: /bitcoin/depends/work/build/x86_64-apple-darwin/bdb/5.3.28-381c107f69c/build_unix/.stamp_configured] Error 1
make: Leaving directory '/bitcoin/depends'
Build guix-osx failed with exit code 2
Fabien planned changes to this revision.Fri, Nov 29, 19:21
This revision is now accepted and ready to land.Fri, Nov 29, 20:45