garbear / xbmc

XBMC Main Repository
http://xbmc.org
Other
132 stars 53 forks source link

Game add-ons trying to build uae4arm on amd64? #108

Open csdougliss opened 5 years ago

csdougliss commented 5 years ago

Hi,

Trying to build beta2 game.* add-ons on amd64 (Ubuntu 18.04).

Fresh checkout, following a script here:

#!/bin/bash

#ADDONS=game.libretro*
ADDONS=game.*
BRANCH=retroplayer-18beta2-20180914
WORKSPACE=/home/kodi/repos/kodibuild

#sudo rm -rfv $WORKSPACE
# uncomment if you want to remove automatically
#rm -rfv $WORKSPACE
git clone -b $BRANCH https://github.com/garbear/xbmc $WORKSPACE

mkdir $WORKSPACE/build-addons
cd $WORKSPACE/build-addons
cmake -DADDONS_TO_BUILD=$ADDONS \
      -DCMAKE_BUILD_TYPE=Debug \
      -DCMAKE_INSTALL_PREFIX=$WORKSPACE/addons \
      -DPACKAGE_ZIP=1 \
      $WORKSPACE/cmake/addons
make -j2

Bombs out here:

src/audio.cpp: In function ‘void audio_reset()’:
src/audio.cpp:944:28: warning: overflow in implicit constant conversion [-Woverflow]
      cdp->per = PERIOD_MAX - 1;
./src/md-retro/maccess.h: Assembler messages:
./src/md-retro/maccess.h:23: Error: no such instruction: `ldrh %ax,[%rax]'
./src/md-retro/maccess.h:24: Error: no such instruction: `rev16 %ax,%ax'
Makefile:194: recipe for target 'src/audio.o' failed
make[6]: *** [src/audio.o] Error 1
CMakeFiles/uae4arm.dir/build.make:110: recipe for target 'uae4arm/src/uae4arm-stamp/uae4arm-build' failed
make[5]: *** [uae4arm/src/uae4arm-stamp/uae4arm-build] Error 2
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/uae4arm.dir/all' failed
make[4]: *** [CMakeFiles/uae4arm.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make[3]: *** [all] Error 2
CMakeFiles/uae4arm.dir/build.make:110: recipe for target 'build/uae4arm/src/uae4arm-stamp/uae4arm-build' failed
make[2]: *** [build/uae4arm/src/uae4arm-stamp/uae4arm-build] Error 2
CMakeFiles/Makefile2:687: recipe for target 'CMakeFiles/uae4arm.dir/all' failed
make[1]: *** [CMakeFiles/uae4arm.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2
garbear commented 5 years ago

Can you open this issue upstream?

csdougliss commented 5 years ago

@garbear where? in xbmc?

garbear commented 5 years ago

Oh sorry, I wasn't paying attention. uae4arm shouldn't be built for amd64, but we don't have a way to disable per-achitecture. I'll add a note to https://github.com/xbmc/xbmc/pull/13600.

garbear commented 5 years ago

@craigcarnell We're still working on architecture support for our binary add-ons repo (see PR). After that's in we'll see about disabling per-architecture, but it's not a priority. Try make -j2 -k to allow make to continue even if one process fails.