search

hashbangcode / vlad

Vlad - Vagrant LAMP Ansible Drupal
173 stars 53 forks source link

unify base boxes #317

Closed zxaos closed 8 years ago

zxaos commented 8 years ago

See #308

philipnorton42 commented 8 years ago

I like the changes here, but I'm guessing this will also create some backward compatibility issues?

zxaos commented 8 years ago

Yep, since it's moving everyone to bento/ base boxes instead of the various ones they're on now, VMs should be destroyed, vlad updated, then VMs recreated.

If you just update vlad, I suspect vagrant will simply say "oh hey, you don't have an existing VM that uses the base box I have configured so I'll start a new one."

Edit: If the whole PR isn't merged, you might still be interested in merging a9fb3fd since that's actually a bug fix that turned out to be unrelated to this PR as @danbohea discovered in #315 .

zxaos commented 8 years ago

But at the same time, if backwards compatibility is going to be interrupted with #312 it might be good to put this with it so it's only interrupted once?

dixhuit commented 8 years ago

Yep, I'm all for this. And like @zxaos says, if we've already merged stuff that's gonna effect existing installations then we might as well sneak a few more in. I'd love to get the current drush branch merged into the next release but can't currently test it fully due to CentOS not provisioning.