|
From: | Stefan Weil |
Subject: | Re: [Qemu-devel] [PULL 0/6] pixman build fixes |
Date: | Tue, 13 Nov 2012 19:07:29 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux i686; rv:16.0) Gecko/20121028 Thunderbird/16.0.2 |
Am 13.11.2012 18:31, schrieb Peter Maydell:
On 13 November 2012 08:42, Gerd Hoffmann <address@hidden> wrote:This patch series fixes the build issues with the internal pixman submodule. It also adds licensing info to the qemu-pixman.[ch] files.These look OK (and I've tested that we can crosscompile using internal pixman in a separate builddirectory). Reviewed-by: Peter Maydell <address@hidden> One remaining problem it would be nice to fix (probably as a separate patch given this lot is at the pullreq stage): "make distclean" in qemu doesn't do a distclean in the pixman/ directory. -- PMM
I just sent reviews for patches 1 and 2. IMHO patch 2 needs some small corrections, and patch 1 can optionally be improved. With at least the corrections, you may add Reviewed-by: Stefan Weil <address@hidden> The patch series clearly improves the current build code. As Peter noted, more problems can be addressed later. Cheers Stefan
[Prev in Thread] | Current Thread | [Next in Thread] |