62 Commits (fafd1a0f648fcd602166d01d47da417b0eb380df)

Author	SHA1	Message	Date
Hennadii Stepanov	91a799247d	depends: Add host-specific `cmake_system_version` variables	3 months ago
Hennadii Stepanov	9b31209b4c	depends: Rename `cmake_system` -> `cmake_system_name`	3 months ago
fanquake	bda537f7c4	depends: remove ENV unsetting for darwin ... Now that we use the native compiler, and have fixed Qt, and these vars are unset it Guix, we can remove the unsetting from our compiler command here. Fixes #21552.	4 months ago
fanquake	d042230f7a	depends: swap mmacosx-version-min for mmacos-version-min ... Whilst these remain aliases for each other, the later is preferred, and I assume the former will be removed at some point in the future; see: https://github.com/llvm/llvm-project/pull/95374.	5 months ago
fanquake	7cbfd7a7ce	refactor: rename (macho) ld64 to lld ... Change some references to the macho ld64 to lld, which is now what is used.	5 months ago
fanquake	fb74fd66cb	depends: remove no-longer used llvm_* vars from macOS build	5 months ago
fanquake	9043f12425	depends: no-longer pass -B to clang in macOS cross-compile	5 months ago
fanquake	e9a44faf14	depends: remove FORCE_USE_SYSTEM_CLANG	5 months ago
fanquake	f836f7e9b3	depends: remove cctools & libtapi	6 months ago
fanquake	4a0536c5d9	build: switch to using lld for macOS builds ... Adjust the security check for: ld64.lld: warning: Option `-allow_stack_execute' is not yet implemented. ld64.lld: error: -fixup_chains is incompatible with -no_pie and to account for the embedding of LLVMs version number.	6 months ago
fanquake	437e908ebd	depends: swap cctools-x for llvm-x ... Only build ld64 from cctools. Disable adhoc codesigning, to avoid non-determinism.	6 months ago
fanquake	7f5ac4520d	build: swap otool for (llvm-)objdump ... Similar to libtool, (llvm-)otool only exists with a version suffix on some systems (Ubuntu), which makes it annoying to use/find. Avoid this, by switching to objdump. Which is a drop-in replacement. This is related to #21778, and the switchover to using vanilla LLVM for macOS.	6 months ago
fanquake	3bee51427a	build: don't use install_name_tool for macOS deploy when cross-compiling ... This is only needed when compiling on macOS. This means we can also better scope the usage of `-headerpad_max_install_names`.	7 months ago
fanquake	eef51afc6a	depends: add -g to DEBUG=1 flags	7 months ago
Cory Fields	5b9d5bf866	depends: remove (darwin) libtool now that it's no longer used ... Note that this is completely unrelated to gnu usage of libtool.	10 months ago
fanquake	cbc9bf11fe	build: move -mlinker-version to *FLAGS ... This doesn't need to exist in C & CXX.	10 months ago
fanquake	42b2283765	depends: deduplicate use of mmacosx-version-min in macOS build	10 months ago
fanquake	2d1b1c7dae	build: remove --enable-lto ... This has outlived its usefulness, doesn't gel well with newer compilers & `-flto` related options, i.e thin vs full, or `=auto`, and having `-flto` as the only option means that sometimes this just needs to be worked around, i.e in oss-fuzz: https://github.com/google/oss-fuzz/blob/master/projects/bitcoin-core/build.sh. While it was convenient when `-flto` was newer, support for `-flto` is now in all compilers we use, and there's also no-longer any real need for us to treat `-flto` different to any other optimization option. Remove it, to remove build complexity, and so there's no need to port a similar option to CMake. Note that the LTO option remains in depends, because we still a way to build packages that have LTO specific patches/options. If we decide to merge this, I'll follow up downstream in oss-fuzz first, to make sure we don't break the build.	10 months ago
fanquake	8ea45e626e	build: use macOS 14 SDK (Xcode 15.0)	11 months ago
fanquake	423949a13b	depends: add -platform_version to macOS build flags ... ```bash -platform_version platform min_version sdk_version This is set to indicate the platform, oldest supported version of that platform that output is to be used on, and the SDK that the output was built against. ```	11 months ago
fanquake	cefbf0bc20	depends: update LD64_VERSION to 711 ... I forgot to do this in 7d5815293e.	1 year ago
Cory Fields	fb61bc0c02	depends: Bump MacOS minimum runtime requirement to 11.0 ... This is necessary as the new fixup_chains linker behavior is only valid when the runtime target is >=11.0.	1 year ago
Cory Fields	cbee1d7091	depends: modernize clang flags ... Fixes builds with llvm >= 11 in guix by working around the problem. As a bonus, this is much cleaner and more maintainable than what we had before.	1 year ago
Cory Fields	4fe5f3c467	depends: remove redundant stdlib option ... Use of -stdlib++-isystem gets rid of any system c++ header include paths and negates the need for this option. In newer versions of clangs the combo produces a warning.	1 year ago
Cory Fields	87afcb0029	depends: fix osx build with clang 16 ... For some reason the previous syntax worked with clang 15 and below, but clang 16 requires that the option and value are properly separated.	2 years ago
fanquake	9b60690b94	cctools: fixup building with LTO ... Use lto.h from clang+llvm not libtapi. The later is older, and comes bundled with the libtapi repo. Copy libLTO.so when building with FORCE_USE_SYSTEM_CLANG.	2 years ago
fanquake	f7595f1354	build: add and use CXX_STANDARD in depends	2 years ago
fanquake	7e7b3e42fa	build: add and use C_STANDARD in depends	2 years ago
fanquake	094772656d	build: support LTO in depends ... No Qt for now.	2 years ago
Hennadii Stepanov	f3af4f7a18	build: Let the depends build system define a path to `dsymutil` tool	3 years ago
fanquake	446e73cc0b	build: use macOS 11 SDK (Xcode 12.2) ... This should be sufficient to support building for Apple ARM when cross-compiling.	3 years ago
fanquake	a43b8e9555	build: set OSX_MIN_VERSION to 10.15 ... This is required to use std::filesystem on macOS as support for it only landed in the libc++ dylib shipped with 10.15. See also: https://developer.apple.com/documentation/xcode-release-notes/xcode-11-release-notes Clang now supports the C++17 <filesystem> library for iOS 13, macOS 10.15, watchOS 6, and tvOS 13.	3 years ago
fanquake	a5550f877a	build: use -stdlib++-isystem with Clang 10	4 years ago
fanquake	c29cba44b3	build: Xcode 12.1, macOS SDK 10.15.6	4 years ago
fanquake	9ed2f19d38	build: native cctools 973.0.1, ld64 609	4 years ago
fanquake	cf971c9ea0	build: use -isysroot over --sysroot on macOS	4 years ago
fanquake	765e0be534	build: split native_cctools	4 years ago
Carl Dong	949c480e52	depends: Fully determine path for darwin cctools ... See previous commit for description.	4 years ago
Carl Dong	880660acfa	depends: Fully determine path for darwin_{CC,CXX} ... Instead of doing the awkward /bin path prepending at config.site creation time, set darwin_{CC,CXX} in a way that fully determines the program's path (clang/clang++) similar to how AC_PATH_{TOOL,PROG} would do. Also see the added comment block in depends/Makefile for more context on determining $PATH for our config.site.	4 years ago
Carl Dong	77b1ef89a0	depends: Remove -fuse-ld line ... clang warns when a command line option is unused, and some of our tests use Werror, so unfortunately we cannot use this flag to pin our linker for now. Leaving this commit in for future reference, as it would be great if there's more granularity to Werror and we can be explicit about what linker we want to use.	4 years ago
Carl Dong	3007339218	depends: Pin clang search paths for darwin host	4 years ago
fanquake	a52ecc936a	build: set minimum supported macOS to 10.14	4 years ago
Carl Dong	60c55b1b9b	depends: Add justifications for macOS clang flags	4 years ago
Cory Fields	6b8e497eea	depends: specify libc++ header location for darwin ... For depends builds this was fixed by fbcfcf69, which deleted the conflicting headers. When we no longer control the clang installation, we need to ensure that the SDK's libc++ headers are used rather than the ones shipped with clang. We can do that by turning off the default include path and hard-coding our own. This hard-coded path is ok because we control (via SDK packaging) where these headers end-up. Side-note: Now that this path is hard-coded in depends, we can potentially package the SDK differently, as the c++ folder can live wherever is most convenient for us.	4 years ago
Carl Dong	c9c572a367	depends: Allow building with system clang	4 years ago
Carl Dong	e6e5c8d6ca	depends: Decouple toolchain + binutils ... For now they remain the same, but in the next commit, we will assign them differently according to wether or not we're using system clang.	4 years ago
fanquake	a8d39b8840	doc: explain why passing -mlinker-version is required	4 years ago
Cory Fields	adf543d714	darwin: pass mlinker-version so that clang enables new features ... Without this clang fails to add any newly-added linker features. Removing this in ca5055a5aa was likely a regression. See https://github.com/bitcoin/bitcoin/pull/19240#issuecomment-647764049 for more discussion.	4 years ago
Cory Fields	2418f739f7	macos: Bump to xcode 11.3.1 and 10.15 SDK ... This gets us a newer SDK with c++17 support and retains 10.12 back-compat. Co-authored-by: Carl Dong <contact@carldong.me>	4 years ago
Carl Dong	3381e4a189	Adapt rest of tooling to new SDK naming scheme	4 years ago