Skip to content
This repository has been archived by the owner on Feb 4, 2020. It is now read-only.

Fix for issue #8 with package requiring repo #9

Merged
merged 1 commit into from
Aug 27, 2014
Merged

Fix for issue #8 with package requiring repo #9

merged 1 commit into from
Aug 27, 2014

Conversation

bjwschaap
Copy link

Added fix to optionally add the require. Added spec tests as well. Tested succesfully on Centos 6.4 vagrant box with PE 3.2.

solarkennedy added a commit that referenced this pull request Aug 27, 2014
Fix for issue #8 with package requiring repo
@solarkennedy solarkennedy merged commit ff7235b into Yelp:master Aug 27, 2014
@solarkennedy
Copy link
Contributor

Thanks! It looks like we were taking for granted that the repo was available in the catalog all the time, but we should not be doing that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants