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

Support uninstall_options for gem package resource like for install_options #8856

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

brandsr
Copy link

@brandsr brandsr commented Jan 11, 2022

The attribute install_options for the gem package resource is included before the gemname to install. While the uninstall_options are included after the gemname ... This is faulty. For now I have modified the uninstall_options to use the same login as for install_options.

The attribute `install_options` for the gem package resource is included _before_ the gemname to install. While the `uninstall_options` are included _after_ the gemname ... This is faulty. For now I have modified the uninstall_options to use the same login as for install_options.
@brandsr brandsr requested a review from a team January 11, 2022 10:49
@brandsr brandsr requested a review from a team as a code owner January 11, 2022 10:49
@brandsr brandsr requested a review from a team January 11, 2022 10:49
@puppetlabs-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@joshcooper
Copy link
Contributor

joshcooper commented Feb 18, 2022

Hi @brandsr thank you for your contribution! Do uninstall options not work because the gem provider is including them after the gem name, like gem uninstall <name> --param value? Do you have a scenario where this causes a failure, or did you want to make it consistent with install options?

To get this merged, could you:

  1. Create a JIRA ticket in the PUP project: https://tickets.puppetlabs.com/projects/PUP
  2. Describe the expected and actual behavior
  3. Prefix the subject line in your commit message with the ticket, e.g. (PUP-123) ...
  4. Update the tests in spec/unit/provider/package/gem_spec.rb as needed.
  5. Sign the CLA

Thank you!

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@joshcooper
Copy link
Contributor

joshcooper commented Nov 15, 2023

Hi @brandsr not sure if you saw my earlier comments? Also is there a particular case where the ordering of uninstall_options was causing problems or was it just inconsistent with respect to install_options?

@joshcooper joshcooper added the bug Something isn't working label Nov 15, 2023
@joshcooper
Copy link
Contributor

Hi @brandsr not sure if you saw my comment. What did you mean by "While the uninstall_options are included after the gemname ... This is faulty." Can you give an example of what was faulty? If we don't hear back in the next few weeks, then we'll need to close this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants