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

WIP: Add Vagrant configuration #3266

Merged
merged 2 commits into from
Jun 3, 2013
Merged

WIP: Add Vagrant configuration #3266

merged 2 commits into from
Jun 3, 2013

Conversation

pao
Copy link
Member

@pao pao commented Jun 2, 2013

Provides a ready-to-go environment for working on Julia, with an eye towards Windows developers who would like to contribute to core, or to test on Linux. The Julia repository is shared between the host and guest, so editing files can be done in the host environment in your editor of choice.

It's a bit clunky--you have to run git submodule init and git submodule update from the host before our Make process gets to it or it messes up the submodule references; you also need to use the make $BUILDOPTS (aliased to jlmake) to ensure you get a fully USE_SYSTEM_* build. At the very least, a full build in VirtualBox on Windows is impossible due to the aux/ directory in the libunwind tarball.

Due to the limitations of VirtualBox shared folders, symlinks need to be replaced with either file copies or elided entirely.

Due to the limitations of VirtualBox shared folders, symlinks need to be
replaced with either file copies or elided entirely.
@@ -30,3 +30,4 @@

/usr
/COMMIT
/contrib/vagrant/.vagrant
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

I would prefer this to be ignored via a /.vagrant entry in contrib/vagrant/.gitignore. That was the directory is relocatable.

Copy link
Member Author

Choose a reason for hiding this comment

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

Can do.

@StefanKarpinski
Copy link
Sponsor Member

I don't know anything about Vagrant, but this sounds cool and useful and the patch looks safe to me.

@Keno
Copy link
Member

Keno commented Jun 3, 2013

I have done a bunch of work with/on Vagrant and this is an excellent idea. LGTM (though I haven't actually tested it).

@pao
Copy link
Member Author

pao commented Jun 3, 2013

I still need to test this on my regular VM to make sure my Makefile changes are right and Linux still builds. I also need to add a blurb to README.md. Will merge after.

@pao
Copy link
Member Author

pao commented Jun 3, 2013

Actually, obviously the Travis build worked, so I guess we're good here.

pao added a commit that referenced this pull request Jun 3, 2013
WIP: Add Vagrant configuration
@pao pao merged commit 3482737 into JuliaLang:master Jun 3, 2013
@pao pao deleted the topic/vagrant branch June 3, 2013 01:58
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

3 participants