Skip to content
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

AUR: further solve .SRCINFO issues #2940

Merged
merged 1 commit into from Jul 28, 2022
Merged

AUR: further solve .SRCINFO issues #2940

merged 1 commit into from Jul 28, 2022

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Jul 28, 2022

Due to the release CI job running on an Ubuntu machine, we do not have
easy access to makepkg to generate the .SRCINFO using --printsrcinfo
as this is a pacman specific utility, and instead we maintain a template.

Historically seen, something went wrong here while the PKGBUILD file
became more complex and certain fields added there were not also
included in the .SRCINFO template.

This commit ensures everything is restored to working state, and
provides the proper fix for what was attempted in #2917. In addition,
checksums are now included in the file.

@hiddeco hiddeco added the area/ci CI related issues and pull requests label Jul 28, 2022
Due to the release CI job running on an Ubuntu machine, we do not have
easy access to `makepkg` to generate the `.SRCINFO` using `--printsrcinfo`
as this is a `pacman` specific utility, and instead we maintain a
template.

Historically seen, something went wrong here while the `PKGBUILD` file
became more complex and certain fields added there were not _also_
included in the `.SRCINFO` template.

This commit ensures everything is restored to working state, and
provides the proper fix for what was attempted in #2917. In addition,
checksums are now included in the file.

Signed-off-by: Hidde Beydals <hello@hidde.co>
@hiddeco hiddeco merged commit ea2de24 into main Jul 28, 2022
@hiddeco hiddeco deleted the fix-srcinfo-bin branch July 28, 2022 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants