From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.33) id 1BedkO-0002Yc-5P for qemu-devel@nongnu.org; Sun, 27 Jun 2004 13:47:20 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.33) id 1BedkM-0002YD-D4 for qemu-devel@nongnu.org; Sun, 27 Jun 2004 13:47:19 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.33) id 1BedkM-0002YA-A3 for qemu-devel@nongnu.org; Sun, 27 Jun 2004 13:47:18 -0400 Received: from [38.113.3.61] (helo=babyruth.hotpop.com) by monty-python.gnu.org with esmtp (Exim 4.34) id 1Bedig-0001zZ-W2 for qemu-devel@nongnu.org; Sun, 27 Jun 2004 13:45:35 -0400 Received: from phreaker.net (kubrick.hotpop.com [38.113.3.103]) by babyruth.hotpop.com (Postfix) with SMTP id 1ED926886B1 for ; Sun, 27 Jun 2004 17:04:27 +0000 (UTC) Date: Sun, 27 Jun 2004 13:42:32 -0400 From: "Jim C. Brown" Subject: Re: [Qemu-devel] qemu - patch to check X authorization Message-ID: <20040627174232.GB32322@jbrown.mylinuxbox.org> References: <200406211614.39031.bobb@absamail.co.za> <20040626180621.GB18324@jbrown.mylinuxbox.org> <200406270948.31733.bobb@absamail.co.za> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200406270948.31733.bobb@absamail.co.za> Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: bobb@absamail.co.za, qemu-devel@nongnu.org On Sun, Jun 27, 2004 at 09:48:31AM +0200, Bob Barry wrote: > Jim - > > On Saturday 26 June 2004 20:06, you wrote: > > That doesn't happen to me. When I try that, I get a few messages from Xlib > > saying that it can't open the X server and then a message from SDL saying that > > its quitting. > > That is the way it should work. Mine didn't. The patch makes mine behave like yours. As you say below, it's an SDL bug and should be fixed. > > > I don't have the problem that you do. > Then your SDL must be different from mine. Qemu should not lock any system, so > the patch is beneficial for systems like mine, and there are probably a lot of them. > My SDL is libsdl-1.2.7 with a few Gentoo patches. Even if the SDL source is > patched to act like yours (and it should be), there will still be a lot of qemu users with > old SDL who should be accomodated. I have libsdl-1.2.2 .... the bug was clearly introduced in a later version. Of course, we could have the configure script detect the SDL version, and if that version is known to have this bug (and we are on an applicable platform) then it could turn on the #DEFINE that would compile the patch. (I think this was already suggested, in fact... can't remember.) > > > Your patch requires that the user must have X and assumes that SDL uses X/Xv. > If there are systems without X which can run qemu with SDL, then the patch code > should be excluded from them with "#ifdef". Are there such systems? I know of only one - the windows port of qemu. You're right here though, a few simple #ifdef's or #ifndef's would do the trick. Of course, it could be trickier if someone decided to let qemu use the framebuffer or the SVGAlib interface on Linux... (SDL supports this but apparently qemu does not atm). > > Thanks, > > Bob Barry > > > _______________________________________________ > Qemu-devel mailing list > Qemu-devel@nongnu.org > http://lists.nongnu.org/mailman/listinfo/qemu-devel -- Infinite complexity begets infinite beauty. Infinite precision begets infinite perfection.