Add platforms to the warnings and update the doctor checks

Author: TsvetanMilanov

lib/doctor.ts

@@ -7,7 +7,7 @@ import { IosLocalBuildRequirements } from "./local-build-requirements/ios-local-
 import { Helpers } from "./helpers";
 import * as semver from "semver";
 
-export class Doctor {
+export class Doctor implements NativeScriptDoctor.IDoctor {
 	private static MIN_SUPPORTED_POD_VERSION = "0.38.2";
 
 	constructor(private androidLocalBuildRequirements: AndroidLocalBuildRequirements,
@@ -38,7 +38,8 @@ export class Doctor {
 				additionalInformation: "For Android-related operations, the AppBuilder CLI will use a built-in version of adb." + EOL
 				+ "To avoid possible issues with the native Android emulator, Genymotion or connected" + EOL
 				+ "Android devices, verify that you have installed the latest Android SDK and" + EOL
-				+ "its dependencies as described in http://developer.android.com/sdk/index.html#Requirements" + EOL
+				+ "its dependencies as described in http://developer.android.com/sdk/index.html#Requirements" + EOL,
+				platforms: [Constants.ANDROID_PLATFORM_NAME]
 			});
 		}
 
@@ -47,7 +48,8 @@ export class Doctor {
 				warning: "WARNING: The Android SDK is not installed or is not configured properly.",
 				additionalInformation: "You will not be able to run your apps in the native emulator. To be able to run apps" + EOL
 				+ "in the native Android emulator, verify that you have installed the latest Android SDK " + EOL
-				+ "and its dependencies as described in http://developer.android.com/sdk/index.html#Requirements" + EOL
+				+ "and its dependencies as described in http://developer.android.com/sdk/index.html#Requirements" + EOL,
+				platforms: [Constants.ANDROID_PLATFORM_NAME]
 			});
 		}
 
@@ -56,23 +58,34 @@ export class Doctor {
 				result.push({
 					warning: "WARNING: Xcode is not installed or is not configured properly.",
 					additionalInformation: "You will not be able to build your projects for iOS or run them in the iOS Simulator." + EOL
-					+ "To be able to build for iOS and run apps in the native emulator, verify that you have installed Xcode." + EOL
+					+ "To be able to build for iOS and run apps in the native emulator, verify that you have installed Xcode." + EOL,
+					platforms: [Constants.IOS_PLATFORM_NAME]
 				});
 			}
 
 			if (!sysInfoData.xcodeprojGemLocation) {
 				result.push({
 					warning: "WARNING: xcodeproj gem is not installed or is not configured properly.",
 					additionalInformation: "You will not be able to build your projects for iOS." + EOL
-					+ "To be able to build for iOS and run apps in the native emulator, verify that you have installed xcodeproj." + EOL
+					+ "To be able to build for iOS and run apps in the native emulator, verify that you have installed xcodeproj." + EOL,
+					platforms: [Constants.IOS_PLATFORM_NAME]
 				});
 			}
 
 			if (!sysInfoData.cocoaPodsVer) {
 				result.push({
 					warning: "WARNING: CocoaPods is not installed or is not configured properly.",
 					additionalInformation: "You will not be able to build your projects for iOS if they contain plugin with CocoaPod file." + EOL
-					+ "To be able to build such projects, verify that you have installed CocoaPods."
+					+ "To be able to build such projects, verify that you have installed CocoaPods.",
+					platforms: [Constants.IOS_PLATFORM_NAME]
+				});
+			}
+
+			if (sysInfoData.cocoaPodsVer && sysInfoData.isCocoaPodsUpdateRequired) {
+				result.push({
+					warning: "WARNING: CocoaPods update required.",
+					additionalInformation: `You are using CocoaPods version ${sysInfoData.cocoaPodsVer} which does not support Xcode ${sysInfoData.xcodeVer} yet.${EOL}${EOL}You can update your cocoapods by running $sudo gem install cocoapods from a terminal.${EOL}${EOL}In order for the NativeScript CLI to be able to work correctly with this setup you need to install xcproj command line tool and add it to your PATH.Xcproj can be installed with homebrew by running $ brew install xcproj from the terminal`,
+					platforms: [Constants.IOS_PLATFORM_NAME]
 				});
 			}
 
@@ -81,7 +94,8 @@ export class Doctor {
 				if (!isCocoaPodsWorkingCorrectly) {
 					result.push({
 						warning: "WARNING: There was a problem with CocoaPods",
-						additionalInformation: "Verify that CocoaPods are configured properly."
+						additionalInformation: "Verify that CocoaPods are configured properly.",
+						platforms: [Constants.IOS_PLATFORM_NAME]
 					});
 				}
 			}
@@ -90,13 +104,15 @@ export class Doctor {
 				result.push({
 					warning: `WARNING: Your current CocoaPods version is earlier than ${Doctor.MIN_SUPPORTED_POD_VERSION}.`,
 					additionalInformation: "You will not be able to build your projects for iOS if they contain plugin with CocoaPod file." + EOL
-					+ `To be able to build such projects, verify that you have at least ${Doctor.MIN_SUPPORTED_POD_VERSION} version installed.`
+					+ `To be able to build such projects, verify that you have at least ${Doctor.MIN_SUPPORTED_POD_VERSION} version installed.`,
+					platforms: [Constants.IOS_PLATFORM_NAME]
 				});
 			}
 		} else {
 			result.push({
 				warning: "NOTE: You can develop for iOS only on Mac OS X systems.",
-				additionalInformation: "To be able to work with iOS devices and projects, you need Mac OS X Mavericks or later." + EOL
+				additionalInformation: "To be able to work with iOS devices and projects, you need Mac OS X Mavericks or later." + EOL,
+				platforms: [Constants.IOS_PLATFORM_NAME]
 			});
 		}
 
@@ -107,7 +123,8 @@ export class Doctor {
 				+ "to perform some Android-related operations. To ensure that you can develop and" + EOL
 				+ "test your apps for Android, verify that you have installed the JDK as" + EOL
 				+ "described in http://docs.oracle.com/javase/8/docs/technotes/guides/install/install_overview.html (for JDK 8)" + EOL
-				+ "or http://docs.oracle.com/javase/7/docs/webnotes/install/ (for JDK 7)." + EOL
+				+ "or http://docs.oracle.com/javase/7/docs/webnotes/install/ (for JDK 7)." + EOL,
+				platforms: [Constants.ANDROID_PLATFORM_NAME]
 			});
 		}
 
@@ -116,7 +133,8 @@ export class Doctor {
 				warning: "WARNING: Git is not installed or not configured properly.",
 				additionalInformation: "You will not be able to create and work with Screen Builder projects." + EOL
 				+ "To be able to work with Screen Builder projects, download and install Git as described" + EOL
-				+ "in https://git-scm.com/downloads and add the git executable to your PATH." + EOL
+				+ "in https://git-scm.com/downloads and add the git executable to your PATH." + EOL,
+				platforms: Constants.SUPPORTED_PLATFORMS
 			});
 		}
 
@@ -133,7 +151,7 @@ export class Doctor {
 		}
 
 		if (!this.isPlatformSupported(platform)) {
-			throw new Error(`Platform ${platform} is not supported. The supported platforms are: ${Constants.SUPPORTED_PLATFORMS.join(", ")}`);
+			throw new Error(`Platform ${platform} is not supported.The supported platforms are: ${Constants.SUPPORTED_PLATFORMS.join(", ")} `);
 		}
 	}
 }

lib/sys-info.ts

@@ -8,8 +8,9 @@ import { platform } from "os";
 import * as path from "path";
 import * as osenv from "osenv";
 import * as temp from "temp";
+import * as semver from "semver";
 
-export class SysInfo {
+export class SysInfo implements NativeScriptDoctor.ISysInfo {
 	// Different java has different format for `java -version` command.
 	private static JAVA_VERSION_REGEXP = /(?:openjdk|java) version \"((?:\d+\.)+(?:\d+))/i;
 
@@ -37,7 +38,9 @@ export class SysInfo {
 	private gradleVerCache: string;
 	private sysInfoCache: NativeScriptDoctor.ISysInfoData;
 	private isCocoaPodsWorkingCorrectlyCache: boolean = null;
-	private nativeScriptCliVersion: string;
+	private nativeScriptCliVersionCache: string;
+	private xcprojInfoCache: NativeScriptDoctor.IXcprojInfo;
+	private isCocoaPodsUpdateRequiredCache: boolean = null;
 
 	constructor(private childProcess: ChildProcess,
 		private fileSystem: FileSystem,
@@ -218,7 +221,7 @@ export class SysInfo {
 			const output = await this.execCommand("gradle -v");
 			const matches = SysInfo.GRADLE_VERSION_REGEXP.exec(output);
 
-		this.gradleVerCache = matches && matches[1];
+			this.gradleVerCache = matches && matches[1];
 		}
 
 		return this.gradleVerCache;
@@ -253,6 +256,7 @@ export class SysInfo {
 			result.gradleVer = await this.getGradleVersion();
 			result.isCocoaPodsWorkingCorrectly = await this.isCocoaPodsWorkingCorrectly();
 			result.nativeScriptCliVersion = await this.getNativeScriptCliVersion();
+			result.isCocoaPodsUpdateRequired = await this.isCocoaPodsUpdateRequired();
 
 			this.sysInfoCache = result;
 		}
@@ -286,12 +290,47 @@ export class SysInfo {
 	}
 
 	public async getNativeScriptCliVersion(): Promise<string> {
-		if (!this.nativeScriptCliVersion) {
+		if (!this.nativeScriptCliVersionCache) {
 			const output = await this.execCommand("tns --version");
-			this.nativeScriptCliVersion = output.trim();
+			this.nativeScriptCliVersionCache = output ? output.trim() : output;
 		}
 
-		return this.nativeScriptCliVersion;
+		return this.nativeScriptCliVersionCache;
+	}
+
+	public async getXcprojInfo(): Promise<NativeScriptDoctor.IXcprojInfo> {
+		if (!this.xcprojInfoCache) {
+			const cocoaPodsVersion = await this.getCocoaPodsVersion();
+			const xcodeVersion = await this.getXcodeVersion();
+
+			// CocoaPods with version lower than 1.0.0 don't support Xcode 7.3 yet
+			// https://github.com/CocoaPods/CocoaPods/issues/2530#issuecomment-210470123
+			// as a result of this all .pbxprojects touched by CocoaPods get converted to XML plist format
+			const shouldUseXcproj = cocoaPodsVersion && !!(semver.lt(cocoaPodsVersion, "1.0.0") && semver.gte(xcodeVersion, "7.3.0"));
+			let xcprojAvailable: boolean;
+
+			if (shouldUseXcproj) {
+				// If that's the case we can use xcproj gem to convert them back to ASCII plist format
+				xcprojAvailable = !!(await this.exec("xcproj --version"))
+			}
+
+			this.xcprojInfoCache = { shouldUseXcproj, xcprojAvailable };
+		}
+
+		return this.xcprojInfoCache;
+	}
+
+	public async isCocoaPodsUpdateRequired(): Promise<boolean> {
+		if (this.isCocoaPodsUpdateRequiredCache === null) {
+			let xcprojInfo = await this.getXcprojInfo();
+			if (xcprojInfo.shouldUseXcproj && !xcprojInfo.xcprojAvailable) {
+				this.isCocoaPodsUpdateRequiredCache = true;
+			} else {
+				this.isCocoaPodsUpdateRequiredCache = false;
+			}
+		}
+
+		return this.isCocoaPodsUpdateRequiredCache;
 	}
 
 	private async exec(cmd: string, execOptions?: ExecOptions): Promise<IProcessInfo> {

typings/interfaces.ts

@@ -99,6 +99,18 @@ declare module NativeScriptDoctor {
 		 */
 		getNativeScriptCliVersion(): Promise<string>;
 
+		/**
+		 * Checks if xcproj is required to build projects and if it is installed.
+		 * @return {Promise<IXcprojInfo>} Returns the collected information aboud xcproj.
+		 */
+		getXcprojInfo(): Promise<IXcprojInfo>;
+
+		/**
+		 * Checks if the current version of CocoaPods is compatible with the installed Xcode.
+		 * @return {boolean} true if an update us require.
+		 */
+		isCocoaPodsUpdateRequired(): Promise<boolean>;
+
 		/**
 		 * Returns the whole system information.
 		 * @return {Promise<ISysInfoData>} The system information.
@@ -253,16 +265,52 @@ declare module NativeScriptDoctor {
 		 * @type {string}
 		 */
 		nativeScriptCliVersion: string;
+
+		/**
+		 * Information about xcproj.
+		 * @type {string}
+		 */
+		xcprojInfo: IXcprojInfo
+
+		/**
+		 * true if the system requires xcproj to build projects successfully and the CocoaPods version is not compatible with the Xcode.
+		 */
+		isCocoaPodsUpdateRequired: boolean;
 	}
 
 	/**
 	 * Describes warning returned from nativescript-doctor check.
 	 */
 	interface IWarning {
-		/** The warning. */
+		/** The warning.
+		 * @type {string}
+		 */
 		warning: string;
-		/** Additional information for the warning. */
+
+		/** Additional information for the warning.
+		 * @type {string}
+		 */
 		additionalInformation: string;
+
+		/** The platforms which are affected by this warning.
+		 * @type {string[]}
+		 */
+		platforms: string[];
+	}
+
+	/**
+	 * Describes information about xcproj brew formula.
+	 */
+	interface IXcprojInfo {
+		/**
+		 * Determines whether the system needs xcproj to execute ios builds sucessfully.
+		 */
+		shouldUseXcproj: boolean;
+
+		/**
+		 * Determines whether xcproj can be called from the command line.
+		 */
+		xcprojAvailable: boolean;
 	}
 
 	/**