Skip to content

Relax the version checking for Arch packages (#32908)#32913

Merged
wxiaoguang merged 1 commit into
go-gitea:release/v1.23from
GiteaBot:backport-32908-v1.23
Dec 19, 2024
Merged

Relax the version checking for Arch packages (#32908)#32913
wxiaoguang merged 1 commit into
go-gitea:release/v1.23from
GiteaBot:backport-32908-v1.23

Conversation

@GiteaBot
Copy link
Copy Markdown
Collaborator

Backport #32908 by @ExplodingDragon

It is mentioned in https://man.archlinux.org/man/PKGBUILD.5: 'The variable is not allowed to contain colons, forward slashes, hyphens, or whitespace.'

_ is also an allowed character, and some software in the Arch Linux AUR uses this naming convention.

In libalpm/package.c#L884-L891, only the prohibited characters are checked.

It is mentioned in https://man.archlinux.org/man/PKGBUILD.5: 'The
variable is not allowed to contain colons, forward slashes, hyphens, or
whitespace.'

`_` is also an allowed character, and some software in the Arch Linux
AUR uses this naming convention.

---------

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@GiteaBot GiteaBot added this to the 1.23.0 milestone Dec 19, 2024
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 19, 2024
@wxiaoguang wxiaoguang enabled auto-merge (squash) December 19, 2024 10:15
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 19, 2024
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Dec 19, 2024
@wxiaoguang wxiaoguang merged commit c20642f into go-gitea:release/v1.23 Dec 19, 2024
@silverwind silverwind added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Dec 19, 2024
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Dec 19, 2024
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Mar 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants