[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] configure: improve multiarch support for pkgcon
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] configure: improve multiarch support for pkgconfig |
Date: |
Fri, 10 Apr 2015 13:55:31 +0100 |
On 10 April 2015 at 13:21, Stefan Hajnoczi <address@hidden> wrote:
> On Thu, Apr 09, 2015 at 05:36:12PM -0400, John Snow wrote:
>> This will improve the multi-arch compilation for hosts using gcc.
>> configurations using clang won't see an improvement, but also won't
>> see a regression.
>
> The commit description does not explain what this patch "improves".
> Please add this explanation (or something equivalent) when merging:
>
> 32-bit compilation on 64-bit hosts is broken because pkgconfig isn't
> multi-arch aware and selects the 64-bit glibconfig.h header file. That
> file assumes the LP64 data model so guint64 is defined as unsigned long.
> This does not work for 32-bit builds where sizeof(unsigned long) == 4
> bytes.
Is this actually a fix for multiarch (which is the Debian/Ubuntu
approach where everything is split into /lib/i386-linux-gnu and
/lib/x86_64-linux-gnu and so on) or for the RedHat setup with /lib
and /lib64? (I forget the name of the latter, maybe multilib?)
-- PMM