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

build(nix): fix naersk build broken when using non-default features #521

Merged
merged 1 commit into from
Apr 1, 2024

Conversation

JakeStanger
Copy link
Owner

Fixes #520

@yavko
Copy link
Contributor

yavko commented Mar 31, 2024

Sorry about this mistake 😔

@JakeStanger
Copy link
Owner Author

Mistakes happen, it's cool. I've got Nix setup on my machine and just about enough grasp to be able to check things myself in future, but please do test things before PR-ing them.

@yavko
Copy link
Contributor

yavko commented Apr 1, 2024

Mistakes happen, it's cool. I've got Nix setup on my machine and just about enough grasp to be able to check things myself in future, but please do test things before PR-ing them.

I think I tested only the crane builder tbh, but it's been so long I don't even remember if I tested crane.

@tukanoidd
Copy link

just checked, it works, can be merged :)

@JakeStanger JakeStanger merged commit e052a99 into master Apr 1, 2024
8 checks passed
@JakeStanger JakeStanger deleted the build/nix/naersk branch April 9, 2024 10:01
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.

When setting "features" through HomeManager config in Nix, cargo throws an error
3 participants