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

Allow StorePathWithOutputs in the cli #4494

Closed

Conversation

thufschmitt
Copy link
Member

Allow things like nix build /nix/store/...-foo.drv!dev.

This is especially useful for ca derivations (because this is the
canonical way of refering to the outputs of ca derivations), but is also
generally useful for consitency − for example it means that
nix build $(nix-instantiate ./foobar.nix) will always work − barring
any GC interaction.

Depends on #4372

Allow things like `nix build /nix/store/...-foo.drv!dev`.

This is especially useful for ca derivations (because this is the
canonical way of refering to the outputs of ca derivations), but is also
generally useful for consitency − for example it means that
`nix build $(nix-instantiate ./foobar.nix)` will always work − barring
any GC interaction.
@thufschmitt thufschmitt added this to the ca-derivations-mvp milestone Jan 29, 2021
@thufschmitt thufschmitt added ca-derivations Derivations with content addressed outputs feature Feature request or proposal labels Jan 29, 2021
@dpulls
Copy link

dpulls bot commented Feb 5, 2021

🎉 All dependencies have been resolved !

@edolstra
Copy link
Member

edolstra commented Feb 5, 2021

We should document this syntax somewhere. Currently I don't think it is (since it was mostly a hack to get multiple outputs to work between nix-instantiate and nix-build).

@edolstra
Copy link
Member

edolstra commented Feb 5, 2021

@regnat There is a merge conflict now, can you look into that?

@thufschmitt
Copy link
Member Author

thufschmitt commented Feb 23, 2021

Superseded by #4541 #4543

@Ericson2314
Copy link
Member

I think you meant #4543, but either way, thanks.

@thufschmitt
Copy link
Member Author

I think you meant #4543, but either way, thanks.

Yes thanks. Edited the comment to rectify

@thufschmitt thufschmitt deleted the ca/allow-pathWithOutputs-in-cli branch March 1, 2021 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ca-derivations Derivations with content addressed outputs feature Feature request or proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants