-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
ryujinx: init at 1.0.5160 #94329
ryujinx: init at 1.0.5160 #94329
Conversation
7858c4d
to
66b39d6
Compare
runHook postBuild | ||
''; | ||
|
||
installPhase = '' |
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.
getting wrapped 3 times
installPhase = '' | |
dontWrapGApps = true; | |
installPhase = '' |
66b39d6
to
cd73c6a
Compare
With all of those suggestions added, the app fails to open the file picker:
Not using |
cd73c6a
to
872ffac
Compare
oh, may need to have |
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.
LGTM
Result of nixpkgs-review pr 94329 1
18 packages marked as broken and skipped:
- linuxPackages_hardkernel_4_14.prl-tools - linuxPackages_hardkernel_4_14.r8125 - linuxPackages_hardkernel_4_14.r8168 - linuxPackages_hardkernel_4_14.rtl8814au - linuxPackages_hardkernel_4_14.rtl8821au - linuxPackages_hardkernel_4_14.rtl8821ce - linuxPackages_hardkernel_4_14.rtl88x2bu - linuxPackages_hardkernel_4_14.rtl88xxau-aircrack - linuxPackages_hardkernel_4_14.rtlwifi_new - php72Packages.phpmd - php74Extensions.zmq - python27Packages.habanero - python27Packages.handout - python27Packages.hass-nabucasa - python37Packages.notify - python37Packages.pyblock - python38Packages.notify - qes1 package built:
- ryujinx
Thanks to both of you. |
Motivation for this change
Adds Ryujinx, a Nintendo Switch emulator written in C#.
This is my first time doing any dotnet packaging, so I've mainly based this off of osu-lazer.
The update script for that uses some checks to see if it already has the latest dependencies (for the
deps.nix
), however i could not get that to work here as there are no stable builds.Upstream does not have stable releases, however dotnet fails when using the regular
unstable-${date}
version convention for git revs. The current versioning is based off of the official appveyor builds.Ryujinx normally tries to write logs next to the executable, so to the nix store. I've added a patch which just puts those in the directory you're launching it from.
Edit: Patch now writes logs to
~/.config/Ryujinx/Logs/
on Linux, and the appropriate directory on other platforms as well.Closes: #85684
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)