Skip to content
This repository has been archived by the owner on Dec 22, 2023. It is now read-only.

Add support for common package managers #1

Merged
merged 11 commits into from
Dec 5, 2023

Conversation

philippwaller
Copy link
Contributor

Add support for common package managers

This PR adds support for the following package managers:

  • apk
  • dnf
  • zypper
  • pacman

Signed-off-by: Philipp Waller <1090452+philippwaller@users.noreply.github.com>
@philippwaller philippwaller marked this pull request as draft January 6, 2022 12:17
Signed-off-by: Philipp Waller <1090452+philippwaller@users.noreply.github.com>
Signed-off-by: Philipp Waller <1090452+philippwaller@users.noreply.github.com>
@philippwaller philippwaller marked this pull request as ready for review January 6, 2022 17:58
@philippwaller
Copy link
Contributor Author

Hey @robertdebock, sorry for pushing but do you have an opinion on this PR?

@robertdebock
Copy link
Owner

Looks good. Sorry for the late reply. Let's wait for CI and pull when successful.

@robertdebock
Copy link
Owner

The build failures have been resolved in master. Can you rebase the PR to master?

@philippwaller
Copy link
Contributor Author

Sorry @robertdebock, it took a while. It should be fine now and is ready to go.

BTW: I did the same implementation in this PR: robertdebock/ansible-role-bootstrap#57. Unfortunately, ansible does not provide a generic module to update packages. But I put it on my bucket lists for rainy days 😉.

@philippwaller
Copy link
Contributor Author

PR is back on track and ready to merge

@philippwaller
Copy link
Contributor Author

@robertdebock: I have implemented an update similar to the one in PR robertdebock/ansible-role-bootstrap#57. Thank you for your work on this matter.

Capitals.
This can not work on Alpine and Suse.
Wow, a typo.
@robertdebock robertdebock merged commit 90b61f1 into robertdebock:master Dec 5, 2023
14 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants