-
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
Import HLSL shader presets #10
Conversation
This imports only the used files from https://github.com/libretro/common-shaders.git at commit c2b31d5ac59796aa70b3d0900416258c018010ec.
This updates the used shader preset files to commit 86cfa146a8dfddf6377ddb5dbcff552feae2e5bf.
Rebased on #9. Lets see how CI does. EDIT: win32 and win64 are good! |
Good to merge? |
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.
Sorry, yeah lgtm.
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 as well. Can we do the same for glsl shaders too?
This PR replaces VelocityRa/xbmc#24. Did I ever do a similar PR for glsl shaders? If not, you'll have to do the same for glsl what I did here. |
You didn't. I'll check if the presets aren't updated regularly and if so I'll make a PR similar to this one. |
Description
This PR replaces the git submodules with the actual files, and only the ones we use.
Git submodules aren't handled by Kodi's build system, and even if they were, we don't want to ship 10s of megabytes of unused shaders.
However, the HLSL shaders aren't really being developed these days, so we can just copy/paste the files directly.
I checked and, in the last 6 years, only a few files have changed. I left the changes as a separate commit in case they need to be inspected.
Related PRs
Replaces VelocityRa/xbmc#24
How has this been tested?
Tested with the following commit: 9408b746c1d64c8774abfed9d8ef983257d14781
Test builds with this change, and removing the shaders from the xbmc repo, are in progress.