-
Notifications
You must be signed in to change notification settings - Fork 37
dsh install prerequisites installs SW I already have #114
Comments
Yep, completely agree. |
@lmakarov I'd recommend uninstalling version that was installed not with brew. At the same time if vagrant/whatever_else was installed via homebrew there is no harm in running |
I was on BoF on drupalcon and interested in this. I have installed not only vagrant but also docker, docker compose and probably most of the necessary tools. So I wonder if at least, there should not be way of simply allowing manual install of tools (in whatever method we are comfortable with), with the required versions and allow us to still use drude. The install could also check the version number and request for a manual upgrade if it cannot do it on its own. |
@hanoii you can still use Drude for sure! Simply don't run |
@hanoii I think I was sitting next to you at that BoF! Anyway, I am trying out Kalabox now on windows and it's doing the same thing, re-installing all this software I already have. This seems to be a theme for me lately.
|
@stkrzysiak yes sure, but PR would also need to check that software is not installed with homebrew (if we're talking Mac) because if it was installed with brew then it can be upgraded and there's no need in manuall installation/upgrades. On Windows there is another packet manager, though. So windows would require it's own different checks. Of course you could only cover Mac in your PR but you see a can of worms it opens, right? update: I'm not sure why would you need it though. If you don't want to re-install software it's totally fine. Don't re-install it. Don't run |
But if you have 3/4 installed it should just install the last remaining prerequisite. Using this flag would still leave you at 3/4 requisites installed, correct? |
Despite having vagrant 1.8.1 on my Mac, dsh install prerequisites is downloading and installing 1.8.1. This behavior is not ideal, i may have 2/3 prerequisites installed, so it should only install the missing one. I'd be open to fixing this, unless this is desired behavior and I'm overlooking the reason for it.
The text was updated successfully, but these errors were encountered: