[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-6.0? 0/6] extern "C" overhaul for C++ files
From: |
Peter Maydell |
Subject: |
Re: [PATCH for-6.0? 0/6] extern "C" overhaul for C++ files |
Date: |
Fri, 16 Apr 2021 18:07:46 +0100 |
On Fri, 16 Apr 2021 at 17:28, Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> On 16/04/21 15:55, Peter Maydell wrote:
> > Hi; this patchseries is:
> > (1) a respin of Paolo's patches, with the review issue Dan
> > noticed fixed (ie handle arm-a64.cc too)
> > (2) a copy of my "osdep.h: Move system includes to top" patch
> > (3) some new patches which try to more comprehensively address
> > the extern "C" issue
> >
> > I've marked this "for-6.0?", but more specifically:
> > * I think patches 1 and 2 should go in if we do an rc4
> > (and maybe we should do an rc4 given various things that
> > have appeared that aren't individually rc4-worthy)
> > * patches 3-6 are definitely 6.1 material
> > Patches 1, 2 and 3 have been reviewed (I kept Dan's r-by on
> > patch 1 since the change to it is just fixing the thing he
> > noticed). Further review, and opinions on the 6.0-ness, whether
> > we should do an rc4, etc, appreciated.
>
> I think at least 1-3 are 6.0 material because build on a supported
> distro (Fedora 34, due for release on April 27) is broken right now.
We don't support not-yet-released distros, because there's no way
to do that: they might always spring new surprises on us that we
don't have time to react to. But I agree that the weight of stuff
we've built up justifies an rc4.
Is patch 3 also required? I thought 1 and 2 would suffice, but
I don't have a system which has the newer glib.
-- PMM
- Re: [PATCH for-6.0? 4/6] osdep: Make os-win32.h and os-posix.h handle 'extern "C"' themselves, (continued)
- [PATCH for-6.0? 1/6] osdep: include glib-compat.h before other QEMU headers, Peter Maydell, 2021/04/16
- [PATCH for-6.0? 6/6] include/disas/dis-asm.h: Handle being included outside 'extern "C"', Peter Maydell, 2021/04/16
- [PATCH for-6.0? 5/6] include/qemu/bswap.h: Handle being included outside extern "C" block, Peter Maydell, 2021/04/16
- Re: [PATCH for-6.0? 0/6] extern "C" overhaul for C++ files, no-reply, 2021/04/16
- Re: [PATCH for-6.0? 0/6] extern "C" overhaul for C++ files, Philippe Mathieu-Daudé, 2021/04/16
- Re: [PATCH for-6.0? 0/6] extern "C" overhaul for C++ files, Paolo Bonzini, 2021/04/16
- Re: [PATCH for-6.0? 0/6] extern "C" overhaul for C++ files,
Peter Maydell <=