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

add missing libretro addons #3262

Merged
merged 3 commits into from Jan 27, 2019
Merged

add missing libretro addons #3262

merged 3 commits into from Jan 27, 2019

Conversation

CvH
Copy link
Member

@CvH CvH commented Jan 26, 2019

  • this is just buildtested (at generic) because I have nothing to runtime test it
  • mame2016 is building, game.libretro.mame2016 fails - ideas welcome how to fix it
buildbot:~/LE$ PROJECT=Generic ARCH=x86_64 scripts/create_addon game.libretro.mame2016
CREATE ADDON game.libretro.mame2016 (Generic/x86_64)
     BUILD      game.libretro.mame2016 (target)
         TOOLCHAIN      cmake (auto-detect)
Executing (target): cmake -GNinja -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DCMAKE_TOOLCHAIN_FILE=/builddir/toolchain/etc/cmake-x86_64-libreelec-linux-gnu.conf -DCMAKE_INSTALL_PREFIX=/usr -DCMAKE_BUILD_TYPE=MinSizeRel /builddir/game.libretro.mame2016-2581a76552da518cb82f8940e3dd1c37d07e640d
-- MAME2016_VERSION=0.174.0.1
-- Configuring done
-- Generating done
-- Build files have been written to: /builddir/game.libretro.mame2016-2581a76552da518cb82f8940e3dd1c37d07e640d/.x86_64-libreelec-linux-gnu
Executing (target): ninja 
ninja: no work to do.
[0/1] Install the project...
-- Install configuration: "MinSizeRel"
CMake Error at cmake_install.cmake:49 (file):
  file INSTALL cannot find
  "/builddir/game.libretro.mame2016-2581a76552da518cb82f8940e3dd1c37d07e640d/MAME2016_LIB-NOTFOUND".


FAILED: CMakeFiles/install.util 
cd /builddir/game.libretro.mame2016-2581a76552da518cb82f8940e3dd1c37d07e640d/.x86_64-libreelec-linux-gnu && /builddir/toolchain/bin/cmake -P cmake_install.cmake
ninja: build stopped: subcommand failed.

@CvH
Copy link
Member Author

CvH commented Jan 26, 2019

@garbear , @5schatten pointed out that fbalpha2012 is completely useless - any idea why this was added at game.kodi at all ?

@garbear
Copy link
Contributor

garbear commented Jan 26, 2019

why is it useless?

@CvH
Copy link
Member Author

CvH commented Jan 26, 2019

@garbear the latest version supports everything even at RPi0 - why ship an 6year old one ?

@garbear
Copy link
Contributor

garbear commented Jan 26, 2019

Sometimes older versions can offer improved compatibility and performance on low-end devices. If that's not the case, then FBA2012 can be dropped.

@vpeter4
Copy link
Contributor

vpeter4 commented Jan 27, 2019

* mame2016 is building, game.libretro.mame2016 fails - ideas welcome how to fix it

I think you are setting wrong shared library name:
mamearcade2016_libretro.so -> mame2016_libretro.so

@CvH
Copy link
Member Author

CvH commented Jan 27, 2019

@vpeter4
tx fixed, hmm strange for some reason upstream is not using the real name

@CvH
Copy link
Member Author

CvH commented Jan 27, 2019

Removed fbalpha2012 from pr, there is no point in supporting some random old version if everything works at the latest version - same situation for fbalpha2018.

As long nobody has some proper reason to include it (besides that libretro/kodi-game has it) we can easily skip it. Also we don't increase the mess due the huge number of emulator cores with no proper way to distinguish the functions.

@HiassofT HiassofT merged commit 8a67396 into LibreELEC:master Jan 27, 2019
@CvH CvH deleted the 9.0/game-addons branch January 27, 2019 13:54
@garbear
Copy link
Contributor

garbear commented Jan 27, 2019

I support your decision to drop FBA2012. Thanks for keeping libretro add-ons up to date, there's so many of them and I know how much effort it takes to keep everything updated.

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.

None yet

4 participants