Add package gcc to ruby.pp for successful installation on CentOS 6.3. #78

Merged
merged 5 commits into from Nov 15, 2012

Conversation

Projects
None yet
2 participants
Contributor

geauxvirtual commented Nov 7, 2012

Add package gcc to ruby.pp in order for bson_ext and json gems to build native extensions successfully during installation. This is touched upon in issue #54 and issue #77.

geauxvirtual added some commits Nov 6, 2012

@geauxvirtual geauxvirtual Adding package gcc to ruby.pp to resolve issues installing bson_ext a…
…nd json gems when trying to build native extensions
3d574f4
@geauxvirtual geauxvirtual Updating location of razor "executable" to /usr/bin instead of /usr/l…
…ocal/bin as puppet daemon does not have /usr/local/bin in it's path.
86d0912
Contributor

nanliu commented Nov 12, 2012

Can you update the spec tests to reflect these changes. Otherwise looks great.

Contributor

geauxvirtual commented Nov 12, 2012

I've updated the spec tests to include the gcc package. It looks like another commit got tagged into this issue, so let me know if I need to break these out into a separate branch. The other commit resolves an issue with puppet daemon not being able to run the razor command as /usr/local/bin is not in the path of the daemon.

Contributor

nanliu commented Nov 12, 2012

Looks like travis is having some issues. This seems ok to merge from local testing.

Contributor

geauxvirtual commented Nov 15, 2012

Looks like some of the other fixes have been getting grouped into this pull request. Let me go clean this up and issue another pull request with a commit just for this issue.

@nanliu nanliu added a commit that referenced this pull request Nov 15, 2012

@nanliu nanliu Merge pull request #78 from geauxvirtual/master
Add package gcc to ruby.pp for successful installation on CentOS 6.3.
595ca47

@nanliu nanliu merged commit 595ca47 into puppetlabs:master Nov 15, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment