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

Remove yum-epel case statement. #236

Merged
merged 1 commit into from Mar 3, 2015

Conversation

Projects
None yet
2 participants
@cmluciano
Copy link
Collaborator

commented Feb 27, 2015

As of PR25,
yum-epel and yum-erlang_solutions are included by
default for all 'rhel' systems greater than 5.

Tests will fail for this branch until a new release of the Erlang cookbook. I tested
with my Berksfile pointed at master and all was good.

@jjasghar

This comment has been minimized.

Copy link
Collaborator

commented Feb 27, 2015

I'm seeing commenting these out? Not removing them?

@cmluciano

This comment has been minimized.

Copy link
Collaborator Author

commented Feb 27, 2015

Gahhhh. Will fix

@cmluciano cmluciano force-pushed the cmluciano:cml/removeyumepel branch from f567f00 to 405e89c Mar 1, 2015

Remove yum-epel case statement.
As of [PR25](chef-cookbooks/erlang#25),
yum-epel and yum-erlang_solutions are included by
default for all 'rhel' systems greater than 5.

@cmluciano cmluciano force-pushed the cmluciano:cml/removeyumepel branch from 405e89c to 80c6b30 Mar 1, 2015

@cmluciano

This comment has been minimized.

Copy link
Collaborator Author

commented Mar 1, 2015

@jjasghar Removed

@jjasghar

This comment has been minimized.

Copy link
Collaborator

commented Mar 1, 2015

Travis failure ;)

@cmluciano

This comment has been minimized.

Copy link
Collaborator Author

commented Mar 1, 2015

Indeed, this PR is awaiting a new release of the Erlang cookbook

@cmluciano

This comment has been minimized.

Copy link
Collaborator Author

commented Mar 2, 2015

OK, just need a rebuild from travis now

@cmluciano cmluciano closed this Mar 2, 2015

@cmluciano cmluciano reopened this Mar 2, 2015

@jjasghar jjasghar added the in progress label Mar 2, 2015

@cmluciano cmluciano self-assigned this Mar 2, 2015

@cmluciano

This comment has been minimized.

Copy link
Collaborator Author

commented Mar 3, 2015

Travis passes :)

jjasghar pushed a commit that referenced this pull request Mar 3, 2015

JJ Asghar
Merge pull request #236 from cmluciano/cml/removeyumepel
Remove yum-epel case statement.

@jjasghar jjasghar merged commit 5a40ffd into rabbitmq:master Mar 3, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jjasghar jjasghar removed the in progress label Mar 3, 2015

@kramvan1 kramvan1 referenced this pull request Apr 22, 2015

Merged

Added ssl_ciphers #255

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.