-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update ares to v143 #53
Conversation
Started test build 181719 |
Build 181719 successful
|
Since the ares Flatpak v142 has some broken functionality (everything that uses the database folder doesn't work, including NES and SNES games) along with shaders not working, it'd be good to either temporarily revert the Flatpak to 141 or review and get this PR merged. |
* Updated buildsystem: * Added cmake-ninja to build ares * Added module to build librashader * Added module to download shaders from libretro * Added module to install the metainfo file * Added README with instructions on how to update the `cargo-sources.json` file for Librashader * Added `ares-paths.patch` to temporarily fix Flatpak not locating the database or shaders folders * Working with upstream ares developers for a better resolution, until then use this patch file
c58231e
to
660a0f9
Compare
Started test build 181729 |
Build 181729 successful
|
Started test build 181731 |
Build 181731 successful
|
Started test build 181734 |
Build 181734 successful
|
Started test build 181735 |
Build 181735 successful
|
Started test build 181759 |
Build 181759 successful
|
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.
The diff issue on the manifest file needs fixing to finalize the review.
This branch also needs rebasing. |
Thanks for looking into it and giving detailed explanations of what's going on. Very much appreciated. |
No problem! Should be good to go now. |
Started test build 181855 |
Build 181855 successful
|
For this flatpak I suggest to pick also this commit ares-emulator/ares@54c898f that fixes a serious bug in v143 affecting game saves. |
Thank you for the heads up! I updated the manifest to point to that commit instead. |
Started test build 181904 |
Build 181904 successful
|
I would like this PR to stick to what's required to update to v143. What @Mastergatto suggested should eventually go to a separate PR following this one, and only include the patch corresponding to the commit. In general, it is bad practice to package git snapshots since other features in development may still be in flux and not considered stable yet. So either we wait for v144 or include the patch corresponding to 54c898f. That being said if this fix is that serious, it should be a good reason to convince upstream to release a new stable version. |
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.
Other than reverting the source for ares to stable archives, LGTM.
Started test build 182088 |
Build 182088 successful
|
cargo-sources.json
file for Librashaderares-paths.patch
to temporarily fix Flatpak not locating the database or shaders foldersSupercedes #43, #51 and #55
Closes #36, #49, #50, and #52
EDIT: