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

Revert "nixos-rebuild: switch to tmpDir during rebuilds" #150975

Merged
merged 1 commit into from
Dec 16, 2021

Conversation

thiagokokada
Copy link
Contributor

Reverts #150065

@github-actions github-actions bot added the 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS label Dec 16, 2021
@thiagokokada
Copy link
Contributor Author

CC @TredwellGit.

@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 labels Dec 16, 2021
@sternenseemann
Copy link
Member

Can you give a reason in the commit message?

This seems to break
`boot.kernelPackages = config.boot.zfs.package.latestCompatibleLinuxPackages`
causing it to use `linuxPackages`.
@thiagokokada thiagokokada force-pushed the revert-150065-workaround-issue-144811 branch from 3b09aca to e6ca3fc Compare December 16, 2021 17:49
@thiagokokada
Copy link
Contributor Author

Can you give a reason in the commit message?

Done.

@Ma27 Ma27 merged commit fb0091f into master Dec 16, 2021
@Ma27 Ma27 deleted the revert-150065-workaround-issue-144811 branch December 16, 2021 22:15
@github-actions
Copy link
Contributor

Successfully created backport PR #151026 for release-21.05.

@github-actions
Copy link
Contributor

Successfully created backport PR #151027 for release-21.11.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants