Move virtualbox to its own command line option

[dirkjot]

• it is large and few people need it; we also have vmware after all

[nertzy]

I'm OK with this. What do you think @professor?

[professor]

My intuition is to remove virtualbox from workstation-setup, however, I'm fine with accepting this change.

[dirkjot]

I think we should one day collect metrics on which options are used. I was reluctant to remove the virtualbox functionality without any insight into who may be using it.

On Wed, Apr 11, 2018 at 5:21 PM, Todd Sedano notifications@github.com wrote:

My intuition is to remove virtualbox from workstation-setup, however, I'm fine with accepting this change.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/pivotal/workstation-setup/pull/200#issuecomment-380635560, or mute the thread https://github.com/notifications/unsubscribe-auth/AAy5NVTLo6Pw9w9de2tz9VVq2TaUtsM6ks5tnp4QgaJpZM4TOxS0 .

-- Dirk P. Janssen

[nertzy]

Yeah it’s very easy to figure out how to install VirtualBox another way and it’s not typical to most of our workflows anymore, especially with Docker being more broadly used.

[professor]

I'd prefer to keep the options to programming languages + docker. Having docker in that list feels odd. Maybe docker should be pulled into standard install for all installations.

I've added https://github.com/pivotal/workstation-setup/pull/201

[professor]

Here's another idea. First we remove virtualbox. If anyone complains, then we can look for this pull request to bring it back in. Thought?

[nertzy]

Yes, let’s remove it.

[professor]

We successfully removed virtualbox from workstation-setup.