Skip to content
This repository has been archived by the owner on Jul 29, 2020. It is now read-only.

Some changes #1

Merged
merged 13 commits into from Jul 22, 2015
Merged

Some changes #1

merged 13 commits into from Jul 22, 2015

Conversation

tiangolo
Copy link
Contributor

Thanks for your code!

I made some changes including a one line install, guessing the username in the ansible-playbook.bat and allowing to do not update ansible every time babun is started (setting a variable in ~/.zshrc), etc.

Feel free to use it if you like it.

Of course you would probably change it to use your repository again.

I make this PR only to let you know the changes I made and that you could use some of them if you like them.

@jonathanhle
Copy link
Owner

Hi tiangolo!

Thanks for the PR.

I've switched over to using Ansible on Linux directly now :-), but I'll merge the PR since it might be useful to those still using the repo.

Cheers,
Jonathan

jonathanhle added a commit that referenced this pull request Jul 22, 2015
@jonathanhle jonathanhle merged commit 931a3ff into jonathanhle:master Jul 22, 2015
@tiangolo
Copy link
Contributor Author

Great!

One question, just for curiosity, are you using a full linux machine or a virtual machine for controlling?

@jonathanhle
Copy link
Owner

I'm dual booting Ubuntu and Windows. In Ubuntu when I'm doing testing in Vagrant/need to provision Vagrant with Ansible. I still spend most of my time in Windows though, so if I'm not provisioning Vagrant, I'll use Ansible in VirtualBox 5.0 (that supports paravirtualization).

@tiangolo
Copy link
Contributor Author

Hmm, interesting, thanks for answering!

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.

None yet

2 participants