[Pkg-xen-devel] [patch] Activate Qemu options

[Ian Campbell]

On Thu, 2011-09-01 at 19:39 +0800, Thomas Goirand wrote:
> On 09/01/2011 03:48 PM, Ian Campbell wrote:
> > On Thu, 2011-09-01 at 15:29 +0800, Thomas Goirand wrote:
> >> On 08/30/2011 06:49 PM, Ian Campbell wrote:
> >>> On Mon, 2011-08-29 at 17:51 +0800, Thomas Goirand wrote:
> > However the configure file seems to default to auto detecting sdl and
> > enables it if the required libraries etc are present. AFAICT simply
> > having the libsdl-dev package in the build-depends should be sufficient
> > to enable sdl support. I've just confirmed that installing libsdl-dev on
> > my Debian machines causes qemu to DTRT and enable SDL.
> 
> But if I don't add:
> 
> LIBS += -lpulse -lSDL
> 
> then I have a link error. I agree it'd be nice to have this in the
> ./configure script, but it's kind of messy in there. I've search for
> "-lSDL" in the file and didn't find it anywhere in there (even grep -r
> lSDL * wont find anything).

It comes from running `sdl-config --libs`. On my system this returns:

$ sdl-config --libs
-L/usr/lib -lSDL

Perhaps this used to be broken but is ok now? Are you sure it is still
an issue? I just tried it (including with your additional configure
options) and it seems to work as expected.

>  Do you have an idea where/how to patch
> better? And for pulse?

Pulse does appear to be broken (I expect as fallout from the use of
--as-needed as the linker default). The link line contains
-lpulse-simple but not -lpulse. Probably the right place to add -lpulse
is next to the -lpulse-simple.

> 
> Thomas
> 

-- 
Ian Campbell

Emacs, n.:
	A slow-moving parody of a text editor.