Skip to content

Upgrade ubuntu-20.04 jobs #743

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 9, 2025
Merged

Upgrade ubuntu-20.04 jobs #743

merged 3 commits into from
Apr 9, 2025

Conversation

dscho
Copy link
Member

@dscho dscho commented Apr 8, 2025

As per actions/runner-images#11101, the ubuntu-20.04 pool is going to the GitHub Graveyard.

@dscho dscho requested a review from mjcheetham April 8, 2025 20:02
@dscho dscho self-assigned this Apr 8, 2025
@dscho dscho marked this pull request as draft April 8, 2025 20:03
dscho added 2 commits April 8, 2025 22:29
The `sparse` job still uses the `ubuntu-20.04` runner pool, but that
pool is about to go away, so let's stop using it.

There is no `sparse-22.04` artifact provided by the "Build sparse for
Ubuntu" Azure Pipeline, but that is not necessary anyway because Ubuntu
22.04 has the `sparse` package: https://packages.ubuntu.com/jammy/sparse

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
The `ubuntu-20.04` pool is going away, let's not use it anymore.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho force-pushed the upgrade-ubuntu-20.04-jobs branch from 044bdc3 to cbfd8c0 Compare April 8, 2025 20:29
With at least glibc 2.39, glibc provides a function declaration that
matches with this POSIX interface:

    int regexec(const regex_t *restrict preg, const char *restrict string,
           size_t nmatch, regmatch_t pmatch[restrict], int eflags);

such prototype requires variable-length-array for `pmatch'.

Thus, sparse reports this error:

> ../add-patch.c: note: in included file (through ../git-compat-util.h):
> /usr/include/regex.h:682:41: error: undefined identifier '__nmatch'
> /usr/include/regex.h:682:41: error: bad constant expression type
> /usr/include/regex.h:682:41: error: Variable length array is used.

Note: `__nmatch' is POSIX's nmatch.

The glibc's intention is informing their users to provides a large
enough buffer to hold `__nmatch' results and provides diagnosis if
necessary.  It's merely a glibc' implementation detail.

Hide that usage from sparse by using standard C11's macro:
__STDC_NO_VLA__

Signed-off-by: Đoàn Trần Công Danh <congdanhqx@gmail.com>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho marked this pull request as ready for review April 9, 2025 10:24
@dscho dscho merged commit 88e9410 into vfs-2.49.0 Apr 9, 2025
119 checks passed
@dscho dscho deleted the upgrade-ubuntu-20.04-jobs branch April 9, 2025 10:25
dscho added a commit that referenced this pull request Jun 6, 2025
As per actions/runner-images#11101, the
`ubuntu-20.04` pool is going to the GitHub Graveyard.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants