[openssl/openssl] e701d9: CI: add GCC 11

Sam James noreply at github.com
Thu Oct 27 09:43:54 UTC 2022


  Branch: refs/heads/openssl-3.0
  Home:   https://github.com/openssl/openssl
  Commit: e701d98618eb7ab08fe52621d65708d3fbaaac83
      https://github.com/openssl/openssl/commit/e701d98618eb7ab08fe52621d65708d3fbaaac83
  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)


  Commit: 33da0e2b0d0be8566d939fd1a5b9d4e4455215d9
      https://github.com/openssl/openssl/commit/33da0e2b0d0be8566d939fd1a5b9d4e4455215d9
  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)


  Commit: 49c2c81d55dab8764c7575815e7566116d4f395a
      https://github.com/openssl/openssl/commit/49c2c81d55dab8764c7575815e7566116d4f395a
  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)


  Commit: 15d698d2f82dd7e42d412f7012a2dfeab28aa458
      https://github.com/openssl/openssl/commit/15d698d2f82dd7e42d412f7012a2dfeab28aa458
  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)


  Commit: e8c8f6faeac54867793d1936a31eef7e644ed487
      https://github.com/openssl/openssl/commit/e8c8f6faeac54867793d1936a31eef7e644ed487
  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)


  Commit: 1b85fc806a7eeef14ee269761bf051c3abdca073
      https://github.com/openssl/openssl/commit/1b85fc806a7eeef14ee269761bf051c3abdca073
  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)


  Commit: 1cfc95b25c930df4de19c428a74d35f7a7411ab5
      https://github.com/openssl/openssl/commit/1cfc95b25c930df4de19c428a74d35f7a7411ab5
  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)


Compare: https://github.com/openssl/openssl/compare/3efb41c026d7...1cfc95b25c93


More information about the openssl-commits mailing list