Skip to content

Comments

[master] adding usermod, groupadd, useradd to requires for rpm#65051

Closed
felippeb wants to merge 2 commits intosaltstack:masterfrom
felippeb:spec-shadow-master
Closed

[master] adding usermod, groupadd, useradd to requires for rpm#65051
felippeb wants to merge 2 commits intosaltstack:masterfrom
felippeb:spec-shadow-master

Conversation

@felippeb
Copy link

What does this PR do?

add binaries required for package installation to package requires.

What issues does this PR fix or reference?

Fixes: installation issue on photon4

https://rpm-software-management.github.io/rpm/manual/boolean_dependencies.html

@felippeb felippeb requested a review from a team as a code owner August 24, 2023 21:03
@felippeb felippeb requested review from MKLeb and removed request for a team August 24, 2023 21:03
@salt-project-bot-prod-environment salt-project-bot-prod-environment bot changed the title adding usermod, groupadd, useradd to requires for rpm [master] adding usermod, groupadd, useradd to requires for rpm Aug 24, 2023
@felippeb felippeb temporarily deployed to ci August 24, 2023 21:27 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 21:27 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 21:27 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 21:27 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 21:46 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 21:49 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 22:46 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 22:46 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 22:46 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 22:46 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 22:46 — with GitHub Actions Inactive
@felippeb felippeb temporarily deployed to ci August 24, 2023 22:46 — with GitHub Actions Inactive
Copy link
Contributor

@dwoz dwoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this go to 3006.x instead of master?

@dwoz
Copy link
Contributor

dwoz commented Sep 5, 2023

Should this go to 3006.x instead of master?

Nevermind, I see the other PR.

@dwoz dwoz self-requested a review September 5, 2023 20:21
dwoz
dwoz previously approved these changes Sep 5, 2023
Signed-off-by: Felippe Burk <burkf@vmware.com>
@twangboy
Copy link
Contributor

twangboy commented Apr 1, 2025

@felippeb Is this still necessary on master?

@felippeb felippeb closed this Apr 1, 2025
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