Fix ci failures in 111#19930
Fix ci failures in 111#19930bernd-edlinger wants to merge 2 commits intoopenssl:OpenSSL_1_1_1-stablefrom
Conversation
| tests: none | ||
| } | ||
| ] | ||
| runs-on: ubuntu-latest |
There was a problem hiding this comment.
Why is this only a problem on 1.1.1? (What is the reason for the change).
There was a problem hiding this comment.
the used gcc-11 version creates false-positive warnings in m68k-linux-gnu which make the build fail,
the warning is due to a large refactoring in the gcc-internal variable range computation engine ("ranger")
https://gcc.gnu.org/wiki/AndrewMacLeod/Ranger
that was rolled out in gcc-11 and gcc is still recovering from this major change.
So instead of disabling a potentially useful warning, I wanted to keep the test using ubuntu-20.04, and if that
version is no longer available, skip directly to ubuntu-24.04, which will be using gcc-13, and that warning
is already gone in gcc-trunk of today, I double-checked that.
|
OTC: Are these strict warning fixes something we want on 1.1.1 branch? |
|
Removing the OTC hold as we removed it from a similar (warning fix) issue on today's OTC meeting. |
|
I would be OK with treating this as urgent as this is a CI fix. |
Agreed |
|
I will merge shortly |
|
Pushed. Thanks. |
Reviewed-by: Tomas Mraz <[email protected]> Reviewed-by: Matt Caswell <[email protected]> (Merged from #19930)
Reviewed-by: Tomas Mraz <[email protected]> Reviewed-by: Matt Caswell <[email protected]> (Merged from #19930)
Reviewed-by: Tomas Mraz <[email protected]> Reviewed-by: Matt Caswell <[email protected]> (Merged from openssl#19930)
Reviewed-by: Tomas Mraz <[email protected]> Reviewed-by: Matt Caswell <[email protected]> (Merged from openssl#19930)
Checklist