mbox series

[GIT,PULL] Upgrade to clang-17 (for liburing's CI)

Message ID a9aac5c7-425d-8011-3c7c-c08dfd7d7c2f@gnuweeb.org (mailing list archive)
State New
Headers show
Series [GIT,PULL] Upgrade to clang-17 (for liburing's CI) | expand

Pull-request

https://github.com/ammarfaizi2/liburing.git tags/upgrade-to-clang17-2023-02-02

Message

Ammar Faizi Feb. 1, 2023, 6:30 p.m. UTC
Hi Jens,

clang-17 is now available. Upgrade the clang version in the liburing's
CI to clang-17.

Two prep patches to address `-Wextra-semi-stmt` warnings:

   - Remove unnecessary semicolon (Alviro)

   - Wrap the CHECK() macro with a do-while statement (Alviro)

A patch for the CI:

   - Upgrade the clang version to 17 and append -Wextra-semi-stmt (me)

Please pull!

The following changes since commit 313aece03ab7dc7447a19cff5b5f542d0c1b2a1e:

   multicqes_drain: make trigger event wait before reading (2023-01-30 09:26:42 -0700)

are available in the Git repository at:

   https://github.com/ammarfaizi2/liburing.git tags/upgrade-to-clang17-2023-02-02

for you to fetch changes up to c1b65520c05413dbf8aa4ed892dbe730a8379de3:

   github: Upgrade the clang version to 17 (2023-02-02 01:12:17 +0700)

----------------------------------------------------------------
Pull CI updates from Ammar Faizi:

   Two prep patches to address `-Wextra-semi-stmt` warnings:

     - Remove unnecessary semicolon (Alviro)

     - Wrap the CHECK() macro with a do-while statement (Alviro)

   A patch for the CI:

     - Upgrade the clang version to 17 and append -Wextra-semi-stmt (me)

----------------------------------------------------------------
Alviro Iskandar Setiawan (2):
       tests: Remove unnecessary semicolon
       tests: Wrap the `CHECK()` macro with a do-while

Ammar Faizi (1):
       github: Upgrade the clang version to 17

  .github/workflows/build.yml |  8 ++++----
  test/accept-link.c          |  2 +-
  test/defer-taskrun.c        | 11 ++++++++---
  test/defer.c                |  2 +-
  test/fixed-link.c           |  2 +-
  test/lfs-openat-write.c     |  9 +++++----
  test/lfs-openat.c           |  9 +++++----
  test/pipe-bug.c             | 10 +++++++---
  test/recv-multishot.c       |  2 +-
  test/timeout.c              |  2 +-
  10 files changed, 34 insertions(+), 23 deletions(-)

Comments

Jens Axboe Feb. 2, 2023, 3:09 a.m. UTC | #1
On 2/1/23 11:30 AM, Ammar Faizi wrote:
> Hi Jens,
> 
> clang-17 is now available. Upgrade the clang version in the liburing's
> CI to clang-17.
> 
> Two prep patches to address `-Wextra-semi-stmt` warnings:
> 
>   - Remove unnecessary semicolon (Alviro)
> 
>   - Wrap the CHECK() macro with a do-while statement (Alviro)
> 
> A patch for the CI:
> 
>   - Upgrade the clang version to 17 and append -Wextra-semi-stmt (me)
> 
> Please pull!
> 
> The following changes since commit 313aece03ab7dc7447a19cff5b5f542d0c1b2a1e:
> 
>   multicqes_drain: make trigger event wait before reading (2023-01-30 09:26:42 -0700)
> 
> are available in the Git repository at:
> 
>   https://github.com/ammarfaizi2/liburing.git tags/upgrade-to-clang17-2023-02-02
> 
> for you to fetch changes up to c1b65520c05413dbf8aa4ed892dbe730a8379de3:
> 
>   github: Upgrade the clang version to 17 (2023-02-02 01:12:17 +0700)
> 
> ----------------------------------------------------------------
> Pull CI updates from Ammar Faizi:
> 
>   Two prep patches to address `-Wextra-semi-stmt` warnings:
> 
>     - Remove unnecessary semicolon (Alviro)
> 
>     - Wrap the CHECK() macro with a do-while statement (Alviro)
> 
>   A patch for the CI:
> 
>     - Upgrade the clang version to 17 and append -Wextra-semi-stmt (me)

Pulled, thanks.