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
retroarch: 1.8.1 -> 1.8.5 #82633
retroarch: 1.8.1 -> 1.8.5 #82633
Conversation
I don't understand the check failure here. |
@@ -50,7 +50,7 @@ in stdenv.mkDerivation rec { | |||
|
|||
enableParallelBuilding = true; | |||
|
|||
configureFlags = if stdenv.isLinux then [ "--enable-kms" ] else ""; | |||
configureFlags = stdenv.lib.optionals stdenv.isLinux [ "--enable-kms" "--enable-egl" ]; |
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.
Can you confirm that "egl" isn't detected by default? I thought in the past the configure script could guess it.
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.
I thought that it didn't pick it up when I tested it on aarch64. Maybe it does though. I'm still afraid to change anything until I understand why the check is failing, if it matters.
This reverts commit e68a409.
- Avoid using overrides unless necessary - Set platform and ARCH by default - Don’t set dontConfigure unless absolutely necessary - Use preBuild instead of overriding entire configurePhase
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.
Looks good.
I have a few changes related to retroarch cores.nix that I have pushed. Mostly have nothing to do with your changes but make things a little less crazy with override logic.
I'll merge given this passes CI. |
@GrahamcOfBorg build libretro |
@GrahamcOfBorg eval |
This is badly out of date so requires extra work to get working. Anyway, you can still use it without the menu, with no problem.
This should be zlib the license, not zlib the package.
Thanks for fixing this up for me. Although this broke picodrive on aarch64. I still don't understand the check failure from before. Might it have something to do with setting gw license to just zlib? Other than that I'm clueless. |
Yeah - the license was being set to zlib the package, not zlib the license. Needed to add stdenv.lib.licenses to make it clear. |
Motivation for this change
Package is out of date, so are the optional core libraries.
Things done
Updated retroarch to 1.8.4
Updated all libretro cores.
Fixed broken libretro cores.
Fixed building on aarch64-linux ( Assumes desktop OpenGL ).
Added several missing libretro cores.
All cores should now build on x86_64-linux and aarch64-linux.
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)