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

Check yum plugin enabled #353

Closed
wants to merge 1 commit into from

Conversation

knqyf263
Copy link
Contributor

What did you implement:

Closes #284

How did you implement it:

Check if the following plugins are enabled.

  • CentOS
    • changelog
  • RHEL
    • security

I use yum --help and parse the first line.
e.g. Loaded plugins: changelog, fastestmirror, security

How can we verify it:

Use prepare command.

Todos:

You don't have to satisfy all of the following.

  • Write tests
  • Write documentation
  • Check that there aren't other open pull requests for the same issue/feature
  • Format your source code by make fmt
  • Pass the test by make test
  • Provide verification config / commands
  • Enable "Allow edits from maintainers" for this PR
  • Update the messages below

Is this ready for review?: NO (I haven't test on RHEL yet.)
Is it a breaking change?: NO

@kotakanbe
Copy link
Member

Thanks for sending P/R.
But the prepare subcommand has been deprecated.
I decided to use configtest to check if yum-plugin-security is enabled. #375

Sorry I could not merge this P/R to master though you sent a pull request.

@kotakanbe kotakanbe closed this Mar 13, 2017
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.

2 participants