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

add documentation to readme for setting up dev env #164

Merged
merged 1 commit into from Feb 2, 2015
Merged

add documentation to readme for setting up dev env #164

merged 1 commit into from Feb 2, 2015

Conversation

mockdeep
Copy link
Member

No description provided.


<pre>
vagrant ssh
git clone https://github.com/gemhome/rmagick.git # or your fork
Copy link
Member

Choose a reason for hiding this comment

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

I prefer to clone it, edit it, and commit/push changes outside of the ssh session. I only use the ssh session to run tests (and install other rubies, packages etc). If you commit within the session, I don't think it forwards your git config. Try git config --global --list | grep user

Copy link
Member Author

Choose a reason for hiding this comment

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

@bf4 Ah, okay. When I ran vagrant ssh it put me in the home directory and when I didn't see the project I assumed I didn't have access to it. Should have read the docs telling that it's in /vagrant.

@bf4
Copy link
Member

bf4 commented Jan 31, 2015

💯 Awesome, thanks for sharing. Now we just need to figure out and document cross-compilation :)

Sounds like we can Close #163 (comment) and ref #161 (comment)

@mockdeep
Copy link
Member Author

@bf4 readme updated.

@bf4
Copy link
Member

bf4 commented Jan 31, 2015

Looks good to me. We'll need to add more info on building for release, but that can be in another PR

@mockdeep
Copy link
Member Author

mockdeep commented Feb 1, 2015

Yeah, I don't think I'm ready to dig into that part just yet. I think know that I've got a passing build I can start working on porting the tests over to rspec.

@bf4
Copy link
Member

bf4 commented Feb 2, 2015

I'm ok to merge this.

@mockdeep
Copy link
Member Author

mockdeep commented Feb 2, 2015

Have at it. I don't have commit rights.

@bf4
Copy link
Member

bf4 commented Feb 2, 2015

Ok, sent you an invite

@mockdeep
Copy link
Member Author

mockdeep commented Feb 2, 2015

Sweet, thanks!

mockdeep added a commit that referenced this pull request Feb 2, 2015
add documentation to readme for setting up dev env
@mockdeep mockdeep merged commit 73c0ed5 into rmagick:master Feb 2, 2015
@mockdeep mockdeep deleted the dev_readme branch February 2, 2015 04:59
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.

None yet

2 participants