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

cmluciano
Copy link

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
Copy link
Contributor

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

@cmluciano
Copy link
Author

Gahhhh. Will fix

As of [PR25](sous-chefs/erlang#25),
yum-epel and yum-erlang_solutions are included by
default for all 'rhel' systems greater than 5.
@cmluciano
Copy link
Author

@jjasghar Removed

@jjasghar
Copy link
Contributor

jjasghar commented Mar 1, 2015

Travis failure ;)

@cmluciano
Copy link
Author

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

@cmluciano
Copy link
Author

OK, just need a rebuild from travis now

@cmluciano cmluciano closed this Mar 2, 2015
@cmluciano cmluciano reopened this Mar 2, 2015
@cmluciano cmluciano self-assigned this Mar 2, 2015
@cmluciano
Copy link
Author

Travis passes :)

jjasghar pushed a commit that referenced this pull request Mar 3, 2015
Remove yum-epel case statement.
@jjasghar jjasghar merged commit 5a40ffd into rabbitmq:master Mar 3, 2015
@kramvan1 kramvan1 mentioned this pull request Apr 22, 2015
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