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
yuzu: init at 2019-05-03 #58312
yuzu: init at 2019-05-03 #58312
Conversation
Is there any blockers preventing this from being merged? |
@microsofttakeover This needs to use |
I'll try to spend some time on it tonight (CEST), it's just fallen off my radar recently. Sorry! |
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.
Just some tiny things. Thanks for packaging this 👍
Odd... this failed to build for me locally:
|
That's odd, ofborg seems to have had no issues with it. I'm going to guess there's some hermeticity issues with keeping .git, but not keeping it breaks yuzu build. I think we're going to have to go with some custom patching for version detection :-( |
@delroth Any update on this? |
I don't have any updates. My assessment is that we'd need some amount of patching and/or upstream contributions to get this package to build reproducibly. I don't have the bandwidth to work on this at the moment or likely any time in the near future. I'll close this PR to clarify the status. |
Motivation for this change
Requested in #58176.
Yuzu doesn't have stable releases yet, so this is based on the current latest git revision.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)