Skip to content
This repository was archived by the owner on Oct 2, 2018. It is now read-only.

[COOK-3719] Allow more custom configuration#56

Closed
garrypolley wants to merge 2 commits into
poise:masterfrom
garrypolley:COOK-3719-make-altinstall-python
Closed

[COOK-3719] Allow more custom configuration#56
garrypolley wants to merge 2 commits into
poise:masterfrom
garrypolley:COOK-3719-make-altinstall-python

Conversation

@garrypolley
Copy link
Copy Markdown
Contributor

https://tickets.opscode.com/browse/COOK-3719

This will allow an easier altinstall of Python. Allows for using the built-in altinstall command option to make. As well as, allows for finer grained control of which pip and virtualenv to use.

Comment thread attributes/default.rb Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo, also don't use the method_missing form of attributes.

@garrypolley
Copy link
Copy Markdown
Contributor Author

@garrypolley
Copy link
Copy Markdown
Contributor Author

@coderanger Thanks for the comments, I've updated the code with those comments in mind. Any more suggestions before this would get merged?

@coderanger
Copy link
Copy Markdown
Member

Looks fine, an probably better than #52 for now. 👍 for merging.

@troygrosfield
Copy link
Copy Markdown

looks fine to me.

@cjflory
Copy link
Copy Markdown

cjflory commented Oct 9, 2013

👍 Looks good

@sethvargo
Copy link
Copy Markdown
Contributor

Merged in 5e02477

@sethvargo sethvargo closed this Oct 9, 2013
@garrypolley garrypolley deleted the COOK-3719-make-altinstall-python branch October 10, 2013 03:45
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.

5 participants