ovmf-native build error with GCC 11 host compiler

ovmf-native build error with GCC 11 host compiler

[Zoltan Boszormenyi]

Hi,

it's a genuine build error with Hardknott on Fedora 34:

brotli/c/dec/decode.c:2033:41: error: argument 2 of type ‘const uint8_t *’ {aka ‘const 
unsigned char *’} declared as a pointer [-Werror=vla-parameter]
  2033 |     size_t encoded_size, const uint8_t* encoded_buffer, size_t* decoded_size,
       |                          ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
In file included from brotli/c/dec/decode.c:7:
./brotli/c/include/brotli/decode.h:204:19: note: previously declared as a variable length 
array ‘const uint8_t[*decoded_size]’ {aka ‘const unsigned char[*decoded_size]’}
   204 |     const uint8_t encoded_buffer[BROTLI_ARRAY_PARAM(encoded_size)],
       |     ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
brotli/c/dec/decode.c:2034:14: error: argument 4 of type ‘uint8_t *’ {aka ‘unsigned char 
*’} declared as a pointer [-Werror=vla-parameter]
  2034 |     uint8_t* decoded_buffer) {
       |     ~~~~~~~~~^~~~~~~~~~~~~~
In file included from brotli/c/dec/decode.c:7:
./brotli/c/include/brotli/decode.h:206:13: note: previously declared as a variable length 
array ‘uint8_t[encoded_size]’ {aka ‘unsigned char[encoded_size]’}
   206 |     uint8_t decoded_buffer[BROTLI_ARRAY_PARAM(*decoded_size)]);
       |     ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

Comparing GCC 10 and 11, -Wvla-parameter is new in GCC 11 and
it's part of to -Wall.

Best regards,
Zoltán Böszörményi

Re: [oe] ovmf-native build error with GCC 11 host compiler

[Khem Raj]

we should disable werror on this recipe

On Mon, May 10, 2021 at 10:52 AM Zoltan Boszormenyi via
lists.openembedded.org <zboszor=pr.hu@lists.openembedded.org> wrote:
>
> Hi,
>
> it's a genuine build error with Hardknott on Fedora 34:
>
> brotli/c/dec/decode.c:2033:41: error: argument 2 of type ‘const uint8_t *’ {aka ‘const
> unsigned char *’} declared as a pointer [-Werror=vla-parameter]
>   2033 |     size_t encoded_size, const uint8_t* encoded_buffer, size_t* decoded_size,
>        |                          ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
> In file included from brotli/c/dec/decode.c:7:
> ./brotli/c/include/brotli/decode.h:204:19: note: previously declared as a variable length
> array ‘const uint8_t[*decoded_size]’ {aka ‘const unsigned char[*decoded_size]’}
>    204 |     const uint8_t encoded_buffer[BROTLI_ARRAY_PARAM(encoded_size)],
>        |     ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> brotli/c/dec/decode.c:2034:14: error: argument 4 of type ‘uint8_t *’ {aka ‘unsigned char
> *’} declared as a pointer [-Werror=vla-parameter]
>   2034 |     uint8_t* decoded_buffer) {
>        |     ~~~~~~~~~^~~~~~~~~~~~~~
> In file included from brotli/c/dec/decode.c:7:
> ./brotli/c/include/brotli/decode.h:206:13: note: previously declared as a variable length
> array ‘uint8_t[encoded_size]’ {aka ‘unsigned char[encoded_size]’}
>    206 |     uint8_t decoded_buffer[BROTLI_ARRAY_PARAM(*decoded_size)]);
>        |     ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> cc1: all warnings being treated as errors
>
> Comparing GCC 10 and 11, -Wvla-parameter is new in GCC 11 and
> it's part of to -Wall.
>
> Best regards,
> Zoltán Böszörményi
>
> 
>