From: Ben Taylor <sol10x86@cox.net>
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] Patches
Date: Sat, 30 Jun 2007 11:13:18 -0400 [thread overview]
Message-ID: <23518129.1183216398890.JavaMail.root@eastrmwml11> (raw)
[-- Attachment #1: Type: text/plain, Size: 307 bytes --]
There are two patches included in the mail
1) qemu-sdl-error.diff - patch to capture output from the sdl compile test, and output it if there
is an error at reporting time (and delete the file when finished)
2) qemu-sdl-whitespace.diff - patch to fix the indentation for the sdl-config section
Ben
[-- Attachment #2: qemu-sdl-error.diff --]
[-- Type: text/x-patch, Size: 1177 bytes --]
--- qemu/configure 2007-06-23 12:03:35.000000000 -0400
+++ ../062707/qemu/configure 2007-06-28 17:55:52.348903000 -0400
@@ -553,7 +553,7 @@
int main( void ) { return SDL_Init (SDL_INIT_VIDEO); }
EOF
-if $cc -o $TMPE `$sdl_config --cflags 2> /dev/null` $TMPC `$sdl_config --libs 2> /dev/null` 2> /dev/null ; then
+if $cc -o $TMPE `$sdl_config --cflags 2> /dev/null` $TMPC `$sdl_config --libs 2> /dev/null` 2> /tmp/qemu-sdl-config.log ; then
_sdlversion=`$sdl_config --version | sed 's/[^0-9]//g'`
if test "$_sdlversion" -lt 121 ; then
sdl_too_old=yes
@@ -562,6 +562,9 @@
sdl=yes
fi
fi
+if [ -s /tmp/qemu-sdl-config.log ]; then
+ check_sdl_log=yes
+fi
# static link with sdl ?
if test "$sdl" = "yes" ; then
@@ -688,6 +691,11 @@
if test $sdl_too_old = "yes"; then
echo "-> Your SDL version is too old - please upgrade to have SDL support"
fi
+if test "$check_sdl_log" = "yes"; then
+ echo "The error log from compiling the libSDL test is: "
+ cat /tmp/qemu-sdl-config.log
+fi
+/usr/bin/rm -f /tmp/qemu-sdl-config.log
#if test "$sdl_static" = "no"; then
# echo "WARNING: cannot compile statically with SDL - qemu-fast won't have a graphical output"
#fi
[-- Attachment #3: qemu-sdl-whitespace.diff --]
[-- Type: text/x-patch, Size: 3204 bytes --]
--- configure.ORIG 2007-06-29 22:59:52.754312000 -0400
+++ configure 2007-06-29 23:11:11.638889000 -0400
@@ -537,60 +537,60 @@
if test -z "$sdl" ; then
-sdl_config="sdl-config"
-sdl=no
-sdl_static=no
-
-if test "$mingw32" = "yes" -a ! -z "$cross_prefix" ; then
-# win32 cross compilation case
- sdl_config="i386-mingw32msvc-sdl-config"
- sdl=yes
-else
-# normal SDL probe
+ sdl_config="sdl-config"
+ sdl=no
+ sdl_static=no
+
+ if test "$mingw32" = "yes" -a ! -z "$cross_prefix" ; then
+ # win32 cross compilation case
+ sdl_config="i386-mingw32msvc-sdl-config"
+ sdl=yes
+ else
+ # normal SDL probe
cat > $TMPC << EOF
#include <SDL.h>
#undef main /* We don't want SDL to override our main() */
int main( void ) { return SDL_Init (SDL_INIT_VIDEO); }
EOF
-if $cc -o $TMPE `$sdl_config --cflags 2> /dev/null` $TMPC `$sdl_config --libs 2> /dev/null` 2> /tmp/qemu-sdl-config.log ; then
-_sdlversion=`$sdl_config --version | sed 's/[^0-9]//g'`
-if test "$_sdlversion" -lt 121 ; then
-sdl_too_old=yes
-else
- if test "$cocoa" = "no" ; then
- sdl=yes
- fi
-fi
-if [ -s /tmp/qemu-sdl-config.log ]; then
- check_sdl_log=yes
-fi
-
-# static link with sdl ?
-if test "$sdl" = "yes" ; then
-aa="no"
-`$sdl_config --static-libs 2>/dev/null | grep \\\-laa > /dev/null` && aa="yes"
-sdl_static_libs=`$sdl_config --static-libs 2>/dev/null`
-if [ "$aa" = "yes" ] ; then
- sdl_static_libs="$sdl_static_libs `aalib-config --static-libs`"
-fi
+ if $cc -o $TMPE `$sdl_config --cflags 2> /dev/null` $TMPC `$sdl_config --libs 2> /dev/null` 2> /tmp/qemu-sdl-config.log ; then
+ _sdlversion=`$sdl_config --version | sed 's/[^0-9]//g'`
+ if test "$_sdlversion" -lt 121 ; then
+ sdl_too_old=yes
+ else
+ if test "$cocoa" = "no" ; then
+ sdl=yes
+ fi
+ fi
+ if [ -s /tmp/qemu-sdl-config.log ]; then
+ check_sdl_log=yes
+ fi
+
+ # static link with sdl ?
+ if test "$sdl" = "yes" ; then
+ aa="no"
+ `$sdl_config --static-libs 2>/dev/null | grep \\\-laa > /dev/null` && aa="yes"
+ sdl_static_libs=`$sdl_config --static-libs 2>/dev/null`
+ if [ "$aa" = "yes" ] ; then
+ sdl_static_libs="$sdl_static_libs `aalib-config --static-libs`"
+ fi
+
+ if $cc -o $TMPE `$sdl_config --cflags 2> /dev/null` $TMPC $sdl_static_libs 2> /dev/null; then
+ sdl_static=yes
+ fi
-if $cc -o $TMPE `$sdl_config --cflags 2> /dev/null` $TMPC $sdl_static_libs 2> /dev/null; then
- sdl_static=yes
-fi
+ fi # static link
-fi # static link
+ fi # sdl compile test
-fi # sdl compile test
-
-fi # cross compilation
+ fi # cross compilation
else
- # Make sure to disable cocoa if sdl was set
- if test "$sdl" = "yes" ; then
- cocoa="no"
- coreaudio="no"
- fi
+ # Make sure to disable cocoa if sdl was set
+ if test "$sdl" = "yes" ; then
+ cocoa="no"
+ coreaudio="no"
+ fi
fi # -z $sdl
##########################################
next reply other threads:[~2007-06-30 15:14 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-30 15:13 Ben Taylor [this message]
-- strict thread matches above, loose matches on Subject: below --
2015-07-14 8:12 [Qemu-devel] Patches Peter Crosthwaite
2015-07-14 9:58 ` Peter Maydell
2015-07-15 8:52 ` Stefan Hajnoczi
2015-07-15 11:26 ` Peter Maydell
2015-07-16 10:33 ` Stefan Hajnoczi
2015-07-14 14:45 ` Stefan Hajnoczi
2014-06-27 10:39 [Qemu-devel] patches lisa
2010-03-18 0:27 Edgar E. Iglesias
[not found] <4a3e994d.8a53f10a.3da6.ffff82adSMTPIN_ADDED@mx.google.com>
2009-06-22 1:05 ` G 3
2009-06-22 5:31 ` Avi Kivity
2005-10-28 23:57 [Qemu-devel] Patches malc
2004-11-19 17:57 [Qemu-devel] patches Thomas Schwinge
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=23518129.1183216398890.JavaMail.root@eastrmwml11 \
--to=sol10x86@cox.net \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.