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

Changed the psycopg2 install method for Windows. #28

Merged
merged 1 commit into from
Jul 17, 2014

Conversation

bmispelon
Copy link
Contributor

The current suggested method relies on git being installed
on the user's machine.
This new one is slightly more complex (there's an
extra step of downloading a prebuilt binary) but
it should work with nothing else installed.

The current suggested method relies on git being installed
on the user's machine.
This new one is slightly more complex (there's an
extra step of downloading a prebuilt binary) but
it should work with nothing else installed.
@maciejgryka
Copy link
Contributor

Another option would be to ask to install Heroku Toolbelt (which includes git) at this point instead of later on. Maybe that would be easier?

@bmispelon
Copy link
Contributor Author

True, I hadn't thought about that.

Frankly, I'm too tired to think right now but I wanted to open the PR before going to bed. Hopefully I'll be smarter in the morning and we can find a way that works.

Thanks.

@maciejgryka
Copy link
Contributor

Oh crap, I'm sorry, I accidentally pushed an alternative solution to this repo instead of my private one. Happy to take it back if people prefer this option.

@olasitarska
Copy link
Member

I think the solution proposed by @maciejgryka is easier. @bmispelon, can you confirm that it would work?

@olasitarska olasitarska reopened this Jul 16, 2014
@maciejgryka
Copy link
Contributor

FYI - we've just finished a setup session, and the way I suggested didn't work (not sure why, I'll try to investigate tomorrow). In the end @bmispelon's solution did the trick though!

@olasitarska
Copy link
Member

@maciejgryka ok, so I'm going to resolve conflict and merge @bmispelon solution. Thanks guys!

@olasitarska olasitarska merged commit 5d34744 into DjangoGirls:master Jul 17, 2014
@bmispelon bmispelon deleted the psycopg2-win-altinstall branch August 9, 2014 10:12
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.

3 participants