Skip to content

Conversation

@mazunki
Copy link
Contributor

@mazunki mazunki commented Oct 29, 2025

Requested change for #2325.

This purely introduces consistency changes and formatting.

@MagnusS
Copy link
Member

MagnusS commented Oct 29, 2025

Thanks! Why is it better to use let..in here?

@mazunki
Copy link
Contributor Author

mazunki commented Oct 29, 2025

It's not necessarily better per se, but variables such as ${out} and friends don't exist until after the derivation is created. This will be useful when we start exposing include paths and so forth.

I think it will also allow us to look at source files after patches are applied, but I haven't tested this yet.

@mazunki
Copy link
Contributor Author

mazunki commented Oct 30, 2025

After these formatting changes, I will be able to use said ${out} and similar variables which don't exist before the derivation is complete the grant easy attribute access to include, lib and source paths of the dependencies for usage in other scripts.

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.

2 participants