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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).