Hi On Wed, Apr 20, 2022 at 6:37 PM Thomas Huth <[email protected]> wrote: > > On 20/04/2022 16.10, Daniel P. Berrangé wrote: > > On Wed, Apr 20, 2022 at 05:26:00PM +0400, [email protected] wrote: > >> From: Marc-André Lureau <[email protected]> > >> > >> According to our configure checks, this is the list of supported > >> compilers. > >> > >> Signed-off-by: Marc-André Lureau <[email protected]> > >> Reviewed-by: Damien Hedde <[email protected]> > >> --- > >> docs/about/build-platforms.rst | 10 ++++++++++ > >> 1 file changed, 10 insertions(+) > >> > >> diff --git a/docs/about/build-platforms.rst > >> b/docs/about/build-platforms.rst > >> index c29a4b8fe649..1980c5d2476f 100644 > >> --- a/docs/about/build-platforms.rst > >> +++ b/docs/about/build-platforms.rst > >> @@ -92,6 +92,16 @@ hosted on Linux (Debian/Fedora). > >> The version of the Windows API that's currently targeted is Vista / > >> Server > >> 2008. > >> > >> +Supported compilers > >> +------------------- > >> + > >> +To compile, QEMU requires either: > >> + > >> +- GCC >= 7.4.0 > >> +- Clang >= 6.0 > >> +- XCode Clang >= 10.0 > > > > Do we need to spell out the versions explicitly ? These versions are > > all derived from what's available in the repos of the supported build > > platforms, similar to any other build deps we have. I don't think we > > want to start a precedent of duplicating versions in this doc for > > build deps we have, and there's nothing particularly special about > > compilers in this respect. > > I agree with Daniel - when I saw this patch, the first thought that I had > was: "This will be getting out of sync quickly" ...
I don't have the impression we bump our compiler requirement regularly or lightly. > so I'd also recommend to rather not add this here. Outdated documentation is still better than no documentation. YMMV.
