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

Add commented-out test for remote building with fixed output derivations #3940

Merged

Conversation

Ericson2314
Copy link
Member

@Ericson2314 Ericson2314 commented Aug 16, 2020

Since there is a preexisting bug, this serves as a reminder to fix it.

@Ericson2314
Copy link
Member Author

Great! This confirms it's an existing bug.

@Ericson2314 Ericson2314 changed the title Test remote building with fixed output derivations WIP: Test remote building with fixed output derivations Aug 17, 2020
@Ericson2314 Ericson2314 changed the title WIP: Test remote building with fixed output derivations WIP: Add commented out test for remote building with fixed output derivations Aug 17, 2020
@Ericson2314 Ericson2314 changed the title WIP: Add commented out test for remote building with fixed output derivations Add commented out test for remote building with fixed output derivations Aug 17, 2020
@Ericson2314 Ericson2314 changed the title Add commented out test for remote building with fixed output derivations Add commented-out test for remote building with fixed output derivations Aug 17, 2020
@edolstra
Copy link
Member

What's the bug exactly? Are remote fixed-output derivations broken?

@edolstra edolstra merged commit 0c9365c into NixOS:master Aug 18, 2020
@Ericson2314 Ericson2314 deleted the fixed-output-remote-builder-test branch August 18, 2020 15:20
@Ericson2314
Copy link
Member Author

Ericson2314 commented Aug 21, 2020

@edolstra Yes, it would appear that they are indeed broken in that they cannot be built remotely.

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

3 participants