On 01/17/2010 05:06 PM, Loïc Minier wrote:
On Sun, Jan 17, 2010, Stefan Weil wrote:
On systems were sdl-config isn't installed, ./configure triggers this
warning:
./configure: 957: sdl-config: not found

which version did you test?
Git master has no sdl-config call at configure:957.

But I get warning messages, too, when pkg-config or
sdl-config are missing.

  Yes, the line is bogus for me as well; not sure why.  I'm using master.

Your patch fixes the warning for sdl-config and sets
sdl=no. If configure was called with --enable-sdl,
this solution is wrong: configure should abort with
an error message (feature_not_found).

  Ack; how about the attached one instead?

Acked-By: Paolo Bonzini <pbonz...@redhat.com>

Paolo



Reply via email to