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

pacman-helper.sh quick_remove: skip uploading package files #598

Merged

Conversation

dscho
Copy link
Member

@dscho dscho commented Feb 20, 2025

The (dual) strategy of the new Pacman repository is to not only publish the latest state in the x86_64, aarch64 and i686 branches, but also to upload the files in the latest update to a GitHub Release.

Naturally, when adding a new package version, the package archive and its PGP signature need to be included in that Release.

However, when removing a package, the corresponding package archive should of course be omitted ;-)

@dscho dscho self-assigned this Feb 20, 2025
@dscho
Copy link
Member Author

dscho commented Feb 20, 2025

This is a quick follow-up to #597 because I noticed that the dry-run wanted to upload the package archives, which is of course incorrect.

The (dual) strategy of the new Pacman repository is to not only publish
the latest state in the `x86_64`, `aarch64` and `i686` branches, but
also to upload the files in the latest update to a GitHub Release.

Naturally, when adding a new package version, the package archive and
its PGP signature need to be included in that Release.

However, when _removing_ a package, the corresponding package archive
should of course be omitted ;-)

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho force-pushed the pacman-helper-quick-remove-follow-up branch from 0ffa94c to 8b144d2 Compare February 20, 2025 11:49
@dscho dscho merged commit e87b4c4 into git-for-windows:main Feb 20, 2025
6 checks passed
@dscho dscho deleted the pacman-helper-quick-remove-follow-up branch February 20, 2025 11:55
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.

1 participant