[PATCH v3 14/24] build-sys: Move to compiling with C11 support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2016-01-18 at 13:06 +0530, arun at accosted.net wrote:
> @@ -176,6 +175,8 @@ esac
>  
>  #### Compiler flags ####
>  
> +AX_CHECK_COMPILE_FLAG([-std=c11], [CFLAGS="$CFLAGS -std=c11"], [], [-pedantic -Werror])

This does nothing if the compile flag isn't supported. Shouldn't we
fail in configure if -std=c11 doesn't work?

Also, I don't think we should set CFLAGS. Adding -std=c11 to AM_CFLAGS
in src/Makefile.am seems like the right thing to do. See
https://www.gnu.org/software/automake/manual/html_node/User-Variables.html#User-Variables

-- 
Tanu


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux