Skip to content
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 game addons... for Christmas! #9581

Merged
merged 4 commits into from
Dec 18, 2024

Conversation

garbear
Copy link
Contributor

@garbear garbear commented Dec 16, 2024

Description

I had some time over the holiday break to do a round of game add-on updates. Besides rebasing everything on their master branches, not much has changed, except that the "Settings" button in the Advanced Settings is now using Kodi's localized "General" label: kodi-game/kodi-game-scripting#132

How has this been tested?

My build machine is in pieces on the floor, so I'm not able to compile-test this PR. But all add-ons have been run through Kodi's CI.

@heitbaum
Copy link
Contributor

A quick build test of the addons (Generic)

UNPACK      libretro-stella
    APPLY PATCH (common)      packages/emulation/libretro-stella/patches/stella-01-do_not-use_static_libgcc_libstdc++.patch
patching file src/os/libretro/Makefile
Hunk #1 FAILED at 57.
1 out of 1 hunk FAILED -- saving rejects to file src/os/libretro/Makefile.rej

Need to revert b7fc8f5 for game.libretro.scummvm

Unable to build test game.libretro.tyrquake - my dev environment is gcc-15 at the moment (and game.libretro.tyrquake doesn’t build with gcc-15.)

@garbear
Copy link
Contributor Author

garbear commented Dec 16, 2024

I'll look into stella. I know that for Kodi's CI, it currently only builds on Windows, due to needing C++20.

Lets drop the scummvm patch, I'm still working on that: #9083

I'll also look into tyrquake.

@heitbaum
Copy link
Contributor

the addition of these 2 commits allow all to build
heitbaum@fc94126
heitbaum@08c1c23

@garbear
Copy link
Contributor Author

garbear commented Dec 18, 2024

I've confirmed that all add-ons build against LE master now after adding your patches, including Tyrquake (LE master using gcc version 14.2.0)

@chewitt chewitt merged commit aecbc8b into LibreELEC:master Dec 18, 2024
@garbear garbear deleted the update-game-addons branch December 19, 2024 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants