Merged
Conversation
3bfc34e to
3f8515d
Compare
- Tidy up LLVM-based stdenv, include LLVM bintools - Add separate depends devShell for building from depends - Update nixpkgs to unstable - Pin Qt6 to specific version for depends compatibility - Modernize CI: update GitHub Actions runners and add per-platform core counts - Fix depends workflow to use new devShell and remove custom cmake config - Set CMAKE_EXPORT_COMPILE_COMMANDS for native LSP support - Use mold linker on Linux for faster builds - Add include-what-you-use and other tools - Format nix files with nixfmt-tree The depends devShell sets build_CC/build_CXX to clang to avoid issues with depends packages that hardcode CXX=g++.
3f8515d to
62a50c6
Compare
The flake previously used llvmPackages.bintools on macOS, which brings in LLVM's lld linker. Apple's toolchain expects its native linker (ld64) with flags like -arch, -platform_version, -syslibroot, etc., which lld doesn't support.
Contributor
Author
|
@josibake @ismaelsadeeq are you (or anyone else) using this flake still? I'd quite like to merge this into master at some point, so if you're using pls test 😘 If not then just let me know, and I can merge even easier :) |
|
I am using the bitcoin flake in the devserver, https://github.com/bitcoin-dev-tools/nix-dev-server/blob/master/home-manager/bitcoin/flake.nix Perhaps @rustaceanrob uses this? |
|
I also use the one in the dev server. LGTM we can give it a spin sometime |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The depends devShell sets build_CC/build_CXX to clang to avoid issues with depends packages that hardcode CXX=g++.