Message ID | 1431367963-24437-1-git-send-email-jsnow@redhat.com |
---|---|
State | New |
Headers | show |
On 11 May 2015 at 19:12, John Snow <jsnow@redhat.com> wrote: > The following changes since commit 9ad2c8cd41a086020e21aa6d616b73bd5e2a800b: > > Merge remote-tracking branch 'remotes/mjt/tags/pull-trivial-patches-2015-05-09' into staging (2015-05-11 13:54:00 +0100) > > are available in the git repository at: > > https://github.com/jnsnow/qemu.git tags/ide-pull-request > > for you to fetch changes up to 6e8d74ed17e0526c34386283df6b7935076d983a: > > qtest: pre-buffer hex nibs (2015-05-11 11:00:05 -0400) > Doesn't build on 32-bit: /root/qemu/qtest.c: In function ‘qtest_process_command’: /root/qemu/qtest.c:519:28: error: format ‘%zu’ expects argument of type ‘size_t’, but argument 2 has type ‘uint64_t’ [-Werror=format] -- PMM
On 05/12/2015 06:44 AM, Peter Maydell wrote: > On 11 May 2015 at 19:12, John Snow <jsnow@redhat.com> wrote: >> The following changes since commit 9ad2c8cd41a086020e21aa6d616b73bd5e2a800b: >> >> Merge remote-tracking branch 'remotes/mjt/tags/pull-trivial-patches-2015-05-09' into staging (2015-05-11 13:54:00 +0100) >> >> are available in the git repository at: >> >> https://github.com/jnsnow/qemu.git tags/ide-pull-request >> >> for you to fetch changes up to 6e8d74ed17e0526c34386283df6b7935076d983a: >> >> qtest: pre-buffer hex nibs (2015-05-11 11:00:05 -0400) >> > > Doesn't build on 32-bit: > > /root/qemu/qtest.c: In function ‘qtest_process_command’: > /root/qemu/qtest.c:519:28: error: format ‘%zu’ expects argument of > type ‘size_t’, but argument 2 has type ‘uint64_t’ [-Werror=format] > > -- PMM > More motivation for me to try another stab at fixing up the -m32 support, I guess. --js
On 12 May 2015 at 16:22, John Snow <jsnow@redhat.com> wrote: > On 05/12/2015 06:44 AM, Peter Maydell wrote: >> Doesn't build on 32-bit: >> >> /root/qemu/qtest.c: In function ‘qtest_process_command’: >> /root/qemu/qtest.c:519:28: error: format ‘%zu’ expects argument of >> type ‘size_t’, but argument 2 has type ‘uint64_t’ [-Werror=format] > More motivation for me to try another stab at fixing up the -m32 > support, I guess. You could also build for Windows 32 bit, or on OSX (where it's only a warning but does detect this even on 64-bit hosts). -- PMM
On 05/12/2015 11:25 AM, Peter Maydell wrote: > On 12 May 2015 at 16:22, John Snow <jsnow@redhat.com> wrote: >> On 05/12/2015 06:44 AM, Peter Maydell wrote: >>> Doesn't build on 32-bit: >>> >>> /root/qemu/qtest.c: In function ‘qtest_process_command’: >>> /root/qemu/qtest.c:519:28: error: format ‘%zu’ expects argument of >>> type ‘size_t’, but argument 2 has type ‘uint64_t’ [-Werror=format] > >> More motivation for me to try another stab at fixing up the -m32 >> support, I guess. > > You could also build for Windows 32 bit, or on OSX (where it's > only a warning but does detect this even on 64-bit hosts). > > -- PMM > Limping along using -m32 and some hacks for now. v2 sent, sorry for the noise. --js