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

This might be overkill... #25

Merged
merged 2 commits into from
Mar 21, 2015
Merged

This might be overkill... #25

merged 2 commits into from
Mar 21, 2015

Conversation

whikloj
Copy link
Contributor

@whikloj whikloj commented Mar 21, 2015

so have a look and let me know what you think before you merge anything.
Addresses #20 and some of #21

Added config script for variables
Files downloaded with wget are stored to /vagrant/downloads to persist through rebuilds
Put WGET files to shared directory, and copy from there if possible.
Change VagrantFile to reference shared directory for config file.
Wrapped some mkdir with tests to avoid trying to create directory that already exists.
@ruebot
Copy link
Contributor

ruebot commented Mar 21, 2015

Love it!

@ruebot ruebot closed this Mar 21, 2015
@ruebot ruebot reopened this Mar 21, 2015
ruebot added a commit that referenced this pull request Mar 21, 2015
@ruebot ruebot merged commit b3ec9b4 into master Mar 21, 2015
@ruebot ruebot deleted the issue-20 branch May 20, 2015 14:26
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