Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

flake.nix: add workaround for ipetkov/crane#385 #44

Merged
merged 1 commit into from
Oct 2, 2023
Merged

Conversation

bryango
Copy link
Contributor

@bryango bryango commented Oct 2, 2023

system-manager currently fails to build on nixpkgs master with the following error:

error: builder for '/nix/store/whz63ya63ha11bzqrwjcqfkivxrg52q9-system-manager-0.1.0.drv' failed with exit code 101;
       last 10 log lines:
       > will append /build/source/.cargo-home/config.toml with contents of /nix/store/1mqzrm4lka0w8r2sxqcnlyq4pdbr4m9z-vendor-cargo-deps/config.toml
       > default configurePhase, nothing to do
       > building
       > ++ command cargo --version
       > cargo 1.72.0 (103a7ff2e 2023-08-15)
       > ++ command cargo build --release --message-format json-render-diagnostics --locked
       >    Compiling libdbus-sys v0.2.5
       > error: output file /build/source/target/release/deps/liblibdbus_sys-6348d0f22c139ac6.rmeta is not writeable -- check its permissions
       >
       > error: could not compile `libdbus-sys` (lib) due to previous error
       For full logs, run 'nix log /nix/store/whz63ya63ha11bzqrwjcqfkivxrg52q9-system-manager-0.1.0.drv'.
error: 1 dependencies of derivation '/nix/store/qbz0zwl83amd3c4kckq8qh65aflg0av3-system-manager.drv' failed to build

The issue is tracked in ipetkov/crane#385. This PR includes a crane option doNotLinkInheritedArtifacts to fix the build. This workaround is borrowed from linyinfeng/oranc@c632217.

@r-vdp
Copy link
Member

r-vdp commented Oct 2, 2023

Awesome, thanks! Just waiting for the checks to finish.

If this fixes the build, then we just have the nix issue with systemd-tmpfiles left to be able to bump the flake again. I need to find the time to finish the open PR for this.

@r-vdp r-vdp merged commit 549bc38 into numtide:main Oct 2, 2023
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants