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

nix/eval-machine-info.nix: fix evaluation with nixpkgs master #1047

Merged
merged 1 commit into from Nov 16, 2018

Conversation

Mic92
Copy link
Member

@Mic92 Mic92 commented Nov 16, 2018

The evaluation broken with NixOS/nixpkgs@6c68fbd

I have no idea why this file was imported in the first place:

bac59ad#diff-6f15e27c3e048b98302b636c2b8f983bR7

I just removed it and it is still working for my none backend.
However since this is file is sufficiently complex I cannot
tell what the general consequence of removing is.

The evaluation broken with NixOS/nixpkgs@6c68fbd

I have no idea why this file was imported in the first place:

NixOS@bac59ad#diff-6f15e27c3e048b98302b636c2b8f983bR7

I just removed it and it is still working for my `none` backend.
However since this is file is sufficiently complex I cannot
tell what the general consequence of removing is.
@Mic92
Copy link
Member Author

Mic92 commented Nov 16, 2018

cc @edolstra if you still can tell why testing was imported
cc @Ekleog

relevant nixpkgs issue: NixOS/nixpkgs#50419

@Mic92
Copy link
Member Author

Mic92 commented Nov 16, 2018

Was runCommand historically in this file?

@domenkozar domenkozar merged commit d5ad09c into NixOS:master Nov 16, 2018
@Mic92 Mic92 deleted the eval-fix branch November 16, 2018 10:46
@Mic92 Mic92 mentioned this pull request Nov 17, 2018
infinisil added a commit to infinisil/system that referenced this pull request Nov 25, 2018
@zimbatm
Copy link
Member

zimbatm commented Nov 28, 2018

@AmineChikhaoui can we get a new release?

@Mic92
Copy link
Member Author

Mic92 commented Nov 28, 2018

I had prepared a release here: #1050

@Mic92
Copy link
Member Author

Mic92 commented Nov 28, 2018

nixopsUnstable also carries the fix and was backported to stable.

@AmineChikhaoui
Copy link
Member

@zimbatm it's actually @rbvermaa who's the release manager. I can help with running the tests though to make it faster.

@WhittlesJr
Copy link

When does this get released into the wild?

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.

None yet

6 participants