From f57c671a70e04deb775dffd4c0efe7b9c0ca5bf8 Mon Sep 17 00:00:00 2001 From: Martijn Walraven Date: Mon, 31 Aug 2015 15:44:15 +0200 Subject: [PATCH] Only list individual platform requirements when in verbose mode Listing them all by default may confuse people, and make them overlook the installation instructions URL. --- tools/cordova/project.js | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/tools/cordova/project.js b/tools/cordova/project.js index f8d1a0d01a..890ea2e830 100644 --- a/tools/cordova/project.js +++ b/tools/cordova/project.js @@ -222,15 +222,25 @@ ${displayNameForPlatform(platform)}`); if (!satisfied) { Console.info(); - Console.info(`Not all installation requirements are satisfied \ -for building and running apps for ${displayNameForPlatform(platform)}.`); + Console.info(`Your system does not yet seem to fulfill all requirements \ +to build apps for ${displayNameForPlatform(platform)}.`); + const url = installationInstructionsUrlForPlatform(platform); if (url) { - Console.info("Please follow the instructions here:"); + Console.info(); + Console.info("Please follow the installation instructions here:"); Console.info(Console.url(url)); } + Console.info(); - Console.info("More details about the individual requirements:"); + + if (!Console.verbose) { + Console.info("Specify the --verbose option to see more details about \ +the status of individual requirements."); + return false; + } + + Console.info("Status of the requirements:"); for (requirement of requirements) { const name = requirement.name; if (requirement.installed) {