-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix build failure #1
Fix build failure #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks 👍 I changed base to my branch but in case you want that in master let me know
Thats fine :) |
Since I merged to the branch of that PR it should already be there |
Ah I guess I misunderstood :) yeah go for it, thanks |
Additional question, I would like to confirm, that this is broken on the nix-community upstream, any Idea on how to do that? |
Do you mean to confirm that manix doesn't provide home-manager options? |
No, i only got this build failure when (for example) doing nix run on this fork, if I do it on the nix-community/manix i dont get a failure. So I cant confirm my commit actually fixes a bug in the upstream. |
Interesting, I don't know. The only change I think could make a difference is
In upstream they have 23.05. my only guess is try changing only this in upstream and rebuild then |
Just tried that, even when committing this doesn´t break the build. |
Nope, but I did this change quite some time ago, didn't try to build this since then |
Seems to be related to this issue nix-community/naersk#323