Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Adding an option to enable the gui in the vagrant box... #1175

Merged
merged 1 commit into from
Feb 21, 2017

Conversation

finnlewis
Copy link
Contributor

... for times when developers might not want a headless box.

Hi Jeff!
Thanks for the drupal-vm, it's great!
I'm working with a team who are on old-ish Windows machines, and would rather work inside a Linux VM with a desktop, which requires enabling the gui the Vagrant options.
This allows that, while keeping the default option to headless.
Any chance it would be useful to merge in?
Cheers,
FInn

@oxyc
Copy link
Collaborator

oxyc commented Feb 21, 2017

You can also add this using a Vagrantfile.local

config.vm.provider :virtualbox do |v|
  v.gui = true
end

@finnlewis
Copy link
Contributor Author

Thanks oxyc, I hadn't realised or thought about that.
I can certainly see the argument for separating out vagrant specific changes in that way, but at the same time it is kind of nice to have those options in config.yml to allow keep to a single local config file. There are also plenty of other options that alter the vagrant config in config.yml, so I guess it seemed the natural place for it.

@geerlingguy
Copy link
Owner

I like having it configurable like this—there have been a few times when I've even wanted to pop the GUI for certain debugging/testing, so I'm fine with this var (instead of requiring a Vagrantfile.local to do it).

@geerlingguy geerlingguy merged commit 8fd3db4 into geerlingguy:master Feb 21, 2017
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.

3 participants