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

Fix the 'index-url' and the 'install' bugs #5

Merged
merged 1 commit into from
May 18, 2015
Merged

Fix the 'index-url' and the 'install' bugs #5

merged 1 commit into from
May 18, 2015

Conversation

boaz0
Copy link
Contributor

@boaz0 boaz0 commented May 18, 2015

Hey,

I am sorry but I have pushed an incorrect code.
It should be index-url and not index_url and default instead of defaults.
In addition, the pip::install can be executed before the pip${python_version} resource, so I have put the require meta-parameter inside the exec { "install-${package}" (correct me if I am wrong).

Tested on my Fedora 19 and RHEL 6 VMs.

@seocam
Copy link
Member

seocam commented May 18, 2015

It seems ok.

seocam added a commit that referenced this pull request May 18, 2015
Fix the 'index-url' and the 'install' bugs
@seocam seocam merged commit 4def2fa into TracyWebTech:master May 18, 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