Skip to content

Conversation

Mic92
Copy link
Member

@Mic92 Mic92 commented Aug 27, 2025

With the migration to /nix/var/nix/builds we now have failing builds when the derivation name is too long.
This change removes the derivation name from the temporary build to have a predictable prefix length:

Also see: NixOS/infra#764
for context.

Motivation

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

With the migration to /nix/var/nix/builds we now have failing builds
when the derivation name is too long.
This change removes the derivation name from the temporary build to have
a predictable prefix length:

Also see: NixOS/infra#764
for context.
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Aug 27, 2025
@Mic92
Copy link
Member Author

Mic92 commented Aug 27, 2025

I would like to backport this to nix 2.30/2.31, so that we can also upgrade hydra.

@xokdvium
Copy link
Contributor

@Mic92, is this actually enough savings to make Darwin builds not fail? We can do a bit more drastic savings to err on the safe side. See Emily's patches to Lix for other options. I imagine we can also rename var/builds to var/b

@Mic92
Copy link
Member Author

Mic92 commented Aug 27, 2025

@Mic92, is this actually enough savings to make Darwin builds not fail? We can do a bit more drastic savings to err on the safe side. See Emily's patches to Lix for other options. I imagine we can also rename var/builds to var/b

I think it its. Names should be now shorter than previously with /tmp.

@Mic92
Copy link
Member Author

Mic92 commented Aug 27, 2025

Also just found out that lix is doing similar things now: https://git.lix.systems/lix-project/lix/commit/ce6dcf18d6e5b0a50bcc0059bd18867d8ac9f8da

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation with-tests Issues related to testing. PRs with tests have some priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants