Kyösti Mälkki (kyosti.malkki@gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2772
-gerrit
commit d362bbceedc33c5895840977149546703279f5aa Author: Kyösti Mälkki kyosti.malkki@gmail.com Date: Sat Mar 16 13:12:01 2013 +0200
Do not build without compatible LUA
The pre-built LUA library for 32-bit distro is most likely built with the type of LUA_INTEGER having essentially range of signed 32 bit integer. SerialICE requires range of 32bit unsigned, so do not even begin the build if that is not satisfied.
Change-Id: Icb93d0126aac5e6e6aaa49cb04a2ebb6077389f1 Signed-off-by: Kyösti Mälkki kyosti.malkki@gmail.com --- qemu-0.15.x/build.sh | 5 ++++- qemu-0.15.x/configure | 19 ++++++++++++++++++- 2 files changed, 22 insertions(+), 2 deletions(-)
diff --git a/qemu-0.15.x/build.sh b/qemu-0.15.x/build.sh index 34b564c..2d8ec98 100644 --- a/qemu-0.15.x/build.sh +++ b/qemu-0.15.x/build.sh @@ -2,4 +2,7 @@ ./configure --disable-kvm --disable-sdl --enable-serialice \ --target-list="x86_64-softmmu, i386-softmmu"
-make +if [ $? -eq 0 ]; then + make +fi + diff --git a/qemu-0.15.x/configure b/qemu-0.15.x/configure index 5eed581..5bb082f 100755 --- a/qemu-0.15.x/configure +++ b/qemu-0.15.x/configure @@ -1763,7 +1763,24 @@ EOF echo exit 1 fi - serialice=yes + +# Test if lua-5.2.x-intsize.patch has been applied. On a 32-bit host system +# LUA_INTEGER is a signed 32 bit integer and not enough for our needs. + cat > $TMPC << EOF +#include <lua.h> +int main(void) { LUA_INTEGER a = (unsigned long int)1<<31; if (a<0) return -1; else return 0; } +EOF + + if compile_prog "$lua_cflags" "$lua_ldflags" && $TMPE > /dev/null 2> /dev/null ; then + serialice=yes + else + echo + echo "Error Found LUA but it needs patch to handle 32 bit unsigned integers" + echo "correctly. Can't build with SerialICE support." + echo + exit 1 + fi + libs_softmmu="$lua_ldflags $libs_softmmu" fi # test "$serialice"