Re: C99 compliance for src/port/snprintf.c

From: Andres Freund <andres(at)anarazel(dot)de>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: David Steele <david(at)pgmasters(dot)net>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: C99 compliance for src/port/snprintf.c
Date: 2018-08-16 11:18:59
Message-ID: 20180816111859.n6vtt322jpjwflij@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-www

On 2018-08-16 01:41:34 -0700, Andres Freund wrote:
> I've pushed a minimal version adding the C99 test.

So, we get:

* lotsa animals, unsurprisingly, showing C99 work without any flags.

checking for ccache gcc option to accept ISO C99... none needed

* rhinoceros, nudibranch, grouse, ...:

https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=rhinoceros&dt=2018-08-16%2008%3A45%3A01&stg=configure
https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=nudibranch&dt=2018-08-16%2009%3A16%3A46&stg=configure
https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=grouse&dt=2018-08-16%2009%3A17%3A25&stg=configure
checking for ccache gcc option to accept ISO C99... -std=gnu99 (or variations thereof)

So, the autoconf magic is doing it's thing here.

* dunlin (icc):

https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=dunlin&dt=2018-08-16%2009%3A35%3A19&stg=configure
checking for icc option to accept ISO C99... -std=gnu99

(later fails, but not newly so, and just because of ENOSPC)

* anole (HP C compiler)

https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=anole&dt=2018-08-16 09%3A32%3A19
checking for cc option to accept ISO C99... none needed

* dromedary:

https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=dromedary&dt=2018-08-16%2008%3A37%3A28&stg=configure
checking for ccache gcc option to accept ISO C99... unsupported

I suspect that's because of the '-ansi' flag in CFLAGS, not because
the compiler is incapable of actually supporting C99.

Besides gaur, I'm also awaiting casteroides' results. The latter
definitely does support C99, but I'm not sure autconf pushes hard
enough. I think every other relevant animal has reported back.

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andreas Joseph Krogh 2018-08-16 11:31:14 Logical decoding currently not usable for all due to lack of LargeObject-support
Previous Message Alexander Korotkov 2018-08-16 11:16:07 Re: [HACKERS] WIP: long transactions on hot standby feedback replica / proof of concept

Browse pgsql-www by date

  From Date Subject
Next Message Magnus Hagander 2018-08-16 11:53:47 Re: PG list descriptions
Previous Message Andres Freund 2018-08-16 08:41:34 Re: C99 compliance for src/port/snprintf.c