How about changing the build system (configure.ac I assume) to just use always C99 mode?

- Martin

On 23 Aug 2016, at 4:57, Paul Jakma wrote:

On Mon, 22 Aug 2016, Martin Winter wrote:

Yes, CFLAGS are overridden for some platforms (to find the needed libraries).

Hmm, so maybe we should just unconditionally add the required C99-mode flag to CFLAGS then? That seems kind of bad though.

I thought a later commit fixed the c99 issue (Or you changed the loop?). Not sure which commit this would be. Potentially adding this commit to the set of pull req 17 should fix it.

Yes, there's a later one that does.

Just, in general, I bet this won't be the last of having the CI fail cause someone used a C99 construct (and we require some C99 constructs, but GCC accepts some in C89 and errors on others).

regards,
--
Paul Jakma | [email protected] | @pjakma | Key ID: 0xD86BF79464A2FF6A
Fortune:
Enjoy your life; be pleasant and gay, like the birds in May.

_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

Reply via email to