-
Notifications
You must be signed in to change notification settings - Fork 10
project_template/bin/setup doesn't check for required dependencies #25
Comments
@flarik @eirvandelden there's a couple of issues currently all regarding the same thing, IMO. #25 #26 and #27 are all about pre-requisites. I would prefer to not mess with automatically installing pre-requisites. This used to work fine when we only needed to support OS X, but now we have to support Linux and Windows too this gets really nasty real quick. Maji is not a configuration management tool :-) My proposal would be to list the following pre-requisites in the README:
Everything else can be considered application dependencies and will be installed automatically. This includes rspec, cordova, phantomjs etc etc. What do you think? |
I completely agree it shouldn't be a configuration management tool, it probably will get messy soon. I was just following the current flow. In case you want people to manually install their own software, instead of with a script, there still needs to be way of checking if people have the necessary "base software" installed, it can be as easy as checking for the commands in the current $PATH and just bail out if it's not there. Not completely sure how this can be done on Windows. (Same goes for git hooks) It also probably means we need more documentation regarding installing apps and tools. For example I just installed rvm, but it didn't have bundler, I installed Android studio, but the installation failed because of the need for android 4.4.2. I can see these things now because I'm on a virgin MacBook. |
I agree, we should add that.
Definitely.
There's a (sparse) section in the readme about that, did you see that? |
This is partially addressed in #27. We do still need to add a couple of checks to warn users when the prerequisites are not met. |
Reworded this issue since we don't have a ruby dependency anymore in Maji 3. However conceptually we're still not checking (all) prerequisites. |
The following command below is just run from bin/setup:
I think several things should be considered here:
The text was updated successfully, but these errors were encountered: