From: Stefan Weil <weil@mail.berlios.de>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel][PATCH] Re: [4792] Change the way audio is configured
Date: Fri, 27 Jun 2008 22:03:13 +0200 [thread overview]
Message-ID: <48654781.2070908@mail.berlios.de> (raw)
In-Reply-To: <0D57F194-2B76-460D-936E-B9E7ED70ADBE@lvivier.info>
[-- Attachment #1: Type: text/plain, Size: 696 bytes --]
Laurent Vivier schrieb:
>
> Le 27 juin 08 à 17:34, consul a écrit :
>
>> audio/audio.c:42: error: `AUDIO_DRIVERS' undeclared here (not in a
>> function)
>> audio/audio.c:42: error: initializer element is not constant
>> audio/audio.c:42: error: (near initialization for `drvtab[0]')
>>
>> Paste error?
>
> No you have to re-run "./configure".
Here is a small patch which automatically calls configure when the file
changes.
It also tells users to run configure when they call make without a
configuration.
The patch is not perfect: it tries to call configure with the last
options, but
it will fail at options containing white space. These are rarely used,
so it is
still useful.
Stefan
[-- Attachment #2: Makefile.patch --]
[-- Type: text/x-diff, Size: 856 bytes --]
Index: Makefile
===================================================================
--- Makefile (revision 4795)
+++ Makefile (working copy)
@@ -1,6 +1,6 @@
# Makefile for QEMU.
-include config-host.mak
+-include config-host.mak
.PHONY: all clean distclean dvi info install install-doc tar tarbin \
speed test html dvi info
@@ -180,6 +180,15 @@
dyngen$(EXESUF): dyngen.c
$(HOST_CC) $(CFLAGS) $(CPPFLAGS) -o $@ $^
+config-host.mak: configure
+ifneq ($(wildcard config-host.mak),)
+ @echo $@ is out-of-date, running configure
+ @fgrep "Configured with:" $@ | sed s/.*Configured.with:.// | sh
+else
+ @echo "Please call configure before running make!"
+ @exit 1
+endif
+
clean:
# avoid old build problems by removing potentially incorrect old files
rm -f config.mak config.h op-i386.h opc-i386.h gen-op-i386.h op-arm.h opc-arm.h gen-op-arm.h
prev parent reply other threads:[~2008-06-27 20:03 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-25 21:04 [Qemu-devel] [4792] Change the way audio is configured malc
2008-06-27 15:34 ` [Qemu-devel] " consul
2008-06-27 15:46 ` Laurent Vivier
2008-06-27 16:00 ` [Qemu-devel] " consul
2008-06-27 20:03 ` Stefan Weil [this message]
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=48654781.2070908@mail.berlios.de \
--to=weil@mail.berlios.de \
--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.