[openssl/openssl] 259b9d: CI: add GCC 11

Sam James noreply at github.com
Thu Oct 27 09:46:52 UTC 2022


  Branch: refs/heads/openssl-3.1
  Home:   https://github.com/openssl/openssl
  Commit: 259b9dd55ee6de8a9cd963a84f578734069c0b04
      https://github.com/openssl/openssl/commit/259b9dd55ee6de8a9cd963a84f578734069c0b04
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M .github/workflows/compiler-zoo.yml

  Log Message:
  -----------
  CI: add GCC 11

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Paul Dale <pauli at openssl.org>

(cherry picked from commit 6332f4c4a2c153869b169d250d9736962abe12c6)
Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Hugo Landau <hlandau at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19500)

(cherry picked from commit e701d98618eb7ab08fe52621d65708d3fbaaac83)


  Commit: 8024730835da13cb1c0fc3c5f8d650d02aa00939
      https://github.com/openssl/openssl/commit/8024730835da13cb1c0fc3c5f8d650d02aa00939
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M .github/workflows/compiler-zoo.yml

  Log Message:
  -----------
  CI: Upgrade to Ubuntu 22.04 to add GCC 12, Clang 13, Clang 14

Notably, this might have caught #18225, as Clang 14 wasn't - and is not yet
until this commit - in OpenSSL's CI.

It makes sense to ensure CI tests compilers used in newer Linux distributions:
* Fedora 36 ships with GCC 12
* Ubuntu 22.04 ships with Clang 14

We switch from 'ubuntu-latest' (which can change meaning but currently points
to ubuntu-20.04) to ubuntu-20.04 for the older existing compilers, and
ubuntu-22.04 for the newer ones added by this commit.

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Paul Dale <pauli at openssl.org>

(cherry picked from commit 712c13c57b97e2e25ca23048f3ba6f50115cacd7)
Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Hugo Landau <hlandau at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19500)

(cherry picked from commit 33da0e2b0d0be8566d939fd1a5b9d4e4455215d9)


  Commit: 1462e6f9a654e567ac24decf28d3c8bc26ae6a73
      https://github.com/openssl/openssl/commit/1462e6f9a654e567ac24decf28d3c8bc26ae6a73
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M .github/workflows/compiler-zoo.yml

  Log Message:
  -----------
  CI: add Clang 15

We have to use the PPA provided by LLVM because Clang 15 isn't
officially part of Ubuntu 22.04 (or any other Ubuntu release yet),
see https://apt.llvm.org/ for details.

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>

(cherry picked from commit 75ecda930e0a961f9605ce090af64d95c98ed161)
Signed-off-by: Sam James <sam at gentoo.org>

(Merged from https://github.com/openssl/openssl/pull/19500)

(cherry picked from commit 49c2c81d55dab8764c7575815e7566116d4f395a)


  Commit: e556b05083f4a1d2734887c6d1add2f0256d46f3
      https://github.com/openssl/openssl/commit/e556b05083f4a1d2734887c6d1add2f0256d46f3
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M crypto/pem/pem_lib.c

  Log Message:
  -----------
  pem: fix -Wunused-but-set-variable

The loop never uses the value of 'line'.

Fixes this error with Clang 15:
```
crypto/pem/pem_lib.c:821:14: error: variable 'line' set but not used [-Werror,-Wunused-but-set-variable]
    int len, line, ret = 0, end = 0, prev_partial_line_read = 0, partial_line_read = 0;
             ^
1 error generated.
```

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>

(cherry picked from commit 71bc497dc321adeb08e7541556dea019c81c9a87)
Signed-off-by: Sam James <sam at gentoo.org>

(Merged from https://github.com/openssl/openssl/pull/19500)

(cherry picked from commit 15d698d2f82dd7e42d412f7012a2dfeab28aa458)


  Commit: 6bfde830c860c21e6407d79ccbf1d39bcc268779
      https://github.com/openssl/openssl/commit/6bfde830c860c21e6407d79ccbf1d39bcc268779
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M crypto/txt_db/txt_db.c

  Log Message:
  -----------
  txt_db: fix -Wunused-but-set-variable

The loop never uses the value of 'ln'.

Fixes this error with Clang 15:
```
crypto/txt_db/txt_db.c:24:10: error: variable 'ln' set but not used [-Werror,-Wunused-but-set-variable]
    long ln = 0;
         ^
1 error generated.
```

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>

(cherry picked from commit f9e8e2c0ab73409862bb78a9285c1b72e0511750)
Signed-off-by: Sam James <sam at gentoo.org>

(Merged from https://github.com/openssl/openssl/pull/19500)

(cherry picked from commit e8c8f6faeac54867793d1936a31eef7e644ed487)


  Commit: 551aa3b6f96ff54ee5971e6e83b4e8ea59af7c81
      https://github.com/openssl/openssl/commit/551aa3b6f96ff54ee5971e6e83b4e8ea59af7c81
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M crypto/x509/x_name.c

  Log Message:
  -----------
  x509: fix -Wunused-but-set-variable

The value of 'l' isn't ever actually used.

Fixes this error with Clang 15:
```
crypto/x509/x_name.c:506:9: error: variable 'l' set but not used [-Werror,-Wunused-but-set-variable]
    int l, i;
        ^
1 error generated.
```

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>

(cherry picked from commit c71318668571b3680fe10035a1a350ff46e459af)
Signed-off-by: Sam James <sam at gentoo.org>

(Merged from https://github.com/openssl/openssl/pull/19500)

(cherry picked from commit 1b85fc806a7eeef14ee269761bf051c3abdca073)


  Commit: 49e5865692f23fddf07a7bfc6298818dbddcdbd2
      https://github.com/openssl/openssl/commit/49e5865692f23fddf07a7bfc6298818dbddcdbd2
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-27 (Thu, 27 Oct 2022)

  Changed paths:
    M test/testutil/driver.c

  Log Message:
  -----------
  test: driver: fix -Wunused-but-set-variable

The value of 'num_failed_inner' isn't ever used.

Fixes this error with Clang 15:
```
test/testutil/driver.c:341:17: error: variable 'num_failed_inner' set but not used [-Werror,-Wunused-but-set-variable]
            int num_failed_inner = 0;
                ^
1 error generated.
```

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>

(cherry picked from commit 6a94c5849ea7d1f08d4fcaa9a6fc0a947e19da66)
Signed-off-by: Sam James <sam at gentoo.org>

(Merged from https://github.com/openssl/openssl/pull/19500)

(cherry picked from commit 1cfc95b25c930df4de19c428a74d35f7a7411ab5)


Compare: https://github.com/openssl/openssl/compare/9fb0ff30343a...49e5865692f2


More information about the openssl-commits mailing list