Merge branch 'release' into tachev/merge-release-into-master

Author: DimitarTachev

CHANGELOG.md

@@ -1,6 +1,43 @@
 NativeScript CLI Changelog
 ================
 
+6.1.2 (2019, September 18)
+==
+
+### Fixed
+* [Fixed #5018](https://github.com/NativeScript/nativescript-cli/issues/5018): Track runtime versions on add and on build, run, deploy
+
+
+6.1.1 (2019, September 17)
+==
+
+### Fixed
+
+* [Fixed #5015](https://github.com/NativeScript/nativescript-cli/pull/5015): CLI passes `--preserve-symlinks` to the webpack itself, not to the Node.js
+* [Fixed #4893](https://github.com/NativeScript/nativescript-cli/issues/4893): `tns preview` crashes when scanning on devices with different platforms
+* [Fixed #4939](https://github.com/NativeScript/nativescript-cli/issues/4939): Xcode 11 warning: `CFBundleIdentifier value must be the same as PRODUCT_BUNDLE_IDENTIFIER`
+
+6.1.0 (2019, September 04)
+==
+
+### New
+
+* [Implemented #4229](https://github.com/NativeScript/nativescript-cli/issues/4229): Do not display command usage help after execution is started
+* [Implemented #4909](https://github.com/NativeScript/nativescript-cli/issues/4909): Support for Xcode 11 and iOS 13
+* [Implemented #4926](https://github.com/NativeScript/nativescript-cli/issues/4926): Android SDK 29 support
+* [Implemented #4947](https://github.com/NativeScript/nativescript-cli/issues/4947): Add tracking for both React NativeScript and Svelte Native projects
+* [Implemented #4966](https://github.com/NativeScript/nativescript-cli/issues/4966): Support LiveSync to iOS Wi-Fi devices
+* [Implemented #4974](https://github.com/NativeScript/nativescript-cli/issues/4974): Ask the users why they've uninstalled NativeScript CLI
+* [Implemented #4976](https://github.com/NativeScript/nativescript-cli/issues/4976): Handle changes in iOS and Android Runtime 6.1.0 logging
+* [Implemented #4980](https://github.com/NativeScript/nativescript-cli/issues/4980): Update message for subscribing to NativeScript newsletter
+* [Implemented #4992](https://github.com/NativeScript/nativescript-cli/pull/4992): Allow tns to be able to use npm configuration properly
+
+### Fixed
+
+* [Fixed #4936](https://github.com/NativeScript/nativescript-cli/issues/4936): HMR not recovering after exception in Angular lazy routes
+* [Fixed #4958](https://github.com/NativeScript/nativescript-cli/issues/4958): `tns doctor` fails when setup is not correct and user selects to fix it manually
+* [Fixed #4971](https://github.com/NativeScript/nativescript-cli/issues/4971): Not needed checks are executed on `pod install`
+
 6.0.3 (2019, August 05)
 ==
 * [Fixed #4914](https://github.com/NativeScript/nativescript-cli/issues/4914): livesync not working with command tns test android

README.md

@@ -143,9 +143,10 @@ The <code>--insecure</code> flag allows you to perform insecure SSL connections
 #### Limitations
 
 * You can provide the `<Username>` and `<Password>` attributes only on Windows systems.
-* Proxy settings for the npm and the Android Gradle need to be configured separately. For more information, see the following articles:
+* Proxy settings for the npm, the Android Gradle and (optional) Docker need to be configured separately. For more information, see the following articles:
 	* [Configure the npm proxy](https://docs.npmjs.com/misc/config#https-proxy)
 	* [Configure the Android Gradle proxy](https://docs.gradle.org/3.3/userguide/build_environment.html#sec:accessing_the_web_via_a_proxy)
+    * [Configure the Docker proxy](https://docs.docker.com/network/proxy/)
 
 ### Display Current Proxy Settings
 

docs/man_pages/general/proxy-set.md

@@ -31,9 +31,10 @@ General | `$ tns proxy set [<Url> <% if(isWindows) {%>[<Username> [<Password>]]<
 ### Command Limitations
 
 * You can set credentials only on Windows systems.
-* Proxy settings for npm and (Android) Gradle need to be set separately.
+* Proxy settings for npm, (Android) Gradle and (optional) Docker need to be set separately.
     * configuring `npm` proxy - https://docs.npmjs.com/misc/config#https-proxy
     * (Android) configuring Gradle proxy - set global configuration in the user directory - _<USER_HOME>/.gradle/gradle.properties_ - https://docs.gradle.org/3.3/userguide/build_environment.html#sec:accessing_the_web_via_a_proxy
+    * configuring Docker proxy - https://docs.docker.com/network/proxy/
 
 ### Related Commands
 

lib/bootstrap.ts

@@ -15,6 +15,7 @@ $injector.require("gradleCommandService", "./services/android/gradle-command-ser
 $injector.require("gradleBuildService", "./services/android/gradle-build-service");
 $injector.require("gradleBuildArgsService", "./services/android/gradle-build-args-service");
 $injector.require("iOSEntitlementsService", "./services/ios-entitlements-service");
+$injector.require("iOSNativeTargetService", "./services/ios-native-target-service");
 $injector.require("iOSExtensionsService", "./services/ios-extensions-service");
 $injector.require("iOSWatchAppService", "./services/ios-watch-app-service");
 $injector.require("iOSProjectService", "./services/ios-project-service");

lib/constants.ts

@@ -186,7 +186,9 @@ export const enum TrackActionNames {
 	AcceptTracking = "Accept Tracking",
 	Performance = "Performance",
 	PreviewAppData = "Preview App Data",
-	UninstallCLI = "Uninstall CLI"
+	UninstallCLI = "Uninstall CLI",
+	UsingRuntimeVersion = "Using Runtime Version",
+	AddPlatform = "Add Platform"
 }
 
 export const AnalyticsEventLabelDelimiter = "__";

lib/controllers/deploy-controller.ts

@@ -11,7 +11,11 @@ export class DeployController {
 
 		const executeAction = async (device: Mobile.IDevice) => {
 			const deviceDescriptor = _.find(deviceDescriptors, dd => dd.identifier === device.deviceInfo.identifier);
-			await this.$prepareController.prepare(deviceDescriptor.buildData);
+			const prepareData = {
+				...deviceDescriptor.buildData,
+				nativePrepare: { skipNativePrepare: !!deviceDescriptor.skipNativePrepare }
+			};
+			await this.$prepareController.prepare(prepareData);
 			const packageFilePath = await deviceDescriptor.buildAction();
 			await this.$deviceInstallAppService.installOnDevice(device, { ...deviceDescriptor.buildData, buildForDevice: !device.isEmulator }, packageFilePath);
 		};

lib/controllers/prepare-controller.ts

@@ -1,10 +1,9 @@
 import * as choki from "chokidar";
 import { hook } from "../common/helpers";
-import { performanceLog } from "../common/decorators";
+import { performanceLog, cache } from "../common/decorators";
 import { EventEmitter } from "events";
 import * as path from "path";
-import { PREPARE_READY_EVENT_NAME, WEBPACK_COMPILATION_COMPLETE, PACKAGE_JSON_FILE_NAME, PLATFORMS_DIR_NAME } from "../constants";
-
+import { PREPARE_READY_EVENT_NAME, WEBPACK_COMPILATION_COMPLETE, PACKAGE_JSON_FILE_NAME, PLATFORMS_DIR_NAME, TrackActionNames, AnalyticsEventLabelDelimiter } from "../constants";
 interface IPlatformWatcherData {
 	hasWebpackCompilerProcess: boolean;
 	nativeFilesWatcher: choki.FSWatcher;
@@ -27,12 +26,14 @@ export class PrepareController extends EventEmitter {
 		private $projectChangesService: IProjectChangesService,
 		private $projectDataService: IProjectDataService,
 		private $webpackCompilerService: IWebpackCompilerService,
-		private $watchIgnoreListService: IWatchIgnoreListService
+		private $watchIgnoreListService: IWatchIgnoreListService,
+		private $analyticsService: IAnalyticsService
 	) { super(); }
 
 	public async prepare(prepareData: IPrepareData): Promise<IPrepareResultData> {
 		const projectData = this.$projectDataService.getProjectData(prepareData.projectDir);
 
+		await this.trackRuntimeVersion(prepareData.platform, projectData);
 		await this.$pluginsService.ensureAllDependenciesAreInstalled(projectData);
 
 		return this.prepareCore(prepareData, projectData);
@@ -127,6 +128,21 @@ export class PrepareController extends EventEmitter {
 	}
 
 	private async startNativeWatcherWithPrepare(platformData: IPlatformData, projectData: IProjectData, prepareData: IPrepareData): Promise<boolean> {
+		let newNativeWatchStarted = false;
+		let hasNativeChanges = false;
+
+		if (prepareData.watchNative) {
+			newNativeWatchStarted = await this.startNativeWatcher(platformData, projectData);
+		}
+
+		if (newNativeWatchStarted) {
+			hasNativeChanges = await this.$prepareNativePlatformService.prepareNativePlatform(platformData, projectData, prepareData);
+		}
+
+		return hasNativeChanges;
+	}
+
+	private async startNativeWatcher(platformData: IPlatformData, projectData: IProjectData): Promise<boolean> {
 		if (this.watchersData[projectData.projectDir][platformData.platformNameLowerCase].nativeFilesWatcher) {
 			return false;
 		}
@@ -155,9 +171,7 @@ export class PrepareController extends EventEmitter {
 
 		this.watchersData[projectData.projectDir][platformData.platformNameLowerCase].nativeFilesWatcher = watcher;
 
-		const hasNativeChanges = await this.$prepareNativePlatformService.prepareNativePlatform(platformData, projectData, prepareData);
-
-		return hasNativeChanges;
+		return true;
 	}
 
 	@hook('watchPatterns')
@@ -186,5 +200,24 @@ export class PrepareController extends EventEmitter {
 			this.persistedData.push(filesChangeEventData);
 		}
 	}
+
+	@cache()
+	private async trackRuntimeVersion(platform: string, projectData: IProjectData): Promise<void> {
+		let runtimeVersion: string = null;
+		try {
+			const platformData = this.$platformsDataService.getPlatformData(platform, projectData);
+			const runtimeVersionData = this.$projectDataService.getNSValue(projectData.projectDir, platformData.frameworkPackageName);
+			runtimeVersion = runtimeVersionData && runtimeVersionData.version;
+		} catch (err) {
+			this.$logger.trace(`Unable to get runtime version for project directory: ${projectData.projectDir} and platform ${platform}. Error is: `, err);
+		}
+
+		if (runtimeVersion) {
+			await this.$analyticsService.trackEventActionInGoogleAnalytics({
+				action: TrackActionNames.UsingRuntimeVersion,
+				additionalData: `${platform.toLowerCase()}${AnalyticsEventLabelDelimiter}${runtimeVersion}`
+			});
+		}
+	}
 }
 $injector.register("prepareController", PrepareController);

lib/controllers/preview-app-controller.ts

@@ -99,7 +99,7 @@ export class PreviewAppController extends EventEmitter implements IPreviewAppCon
 				data.env = data.env || {};
 				data.env.externals = this.$previewAppPluginsService.getExternalPlugins(device);
 
-				const prepareData = this.$prepareDataService.getPrepareData(data.projectDir, device.platform.toLowerCase(), { ...data, nativePrepare: { skipNativePrepare: true }, watch: true });
+				const prepareData = this.$prepareDataService.getPrepareData(data.projectDir, device.platform.toLowerCase(), { ...data, nativePrepare: { skipNativePrepare: true }, watch: true, watchNative: false });
 				await this.$prepareController.prepare(prepareData);
 
 				try {

lib/data/prepare-data.ts

@@ -5,6 +5,7 @@ export class PrepareData extends ControllerDataBase {
 	public hmr: boolean;
 	public env: any;
 	public watch?: boolean;
+	public watchNative: boolean = true;
 
 	constructor(public projectDir: string, public platform: string, data: any) {
 		super(projectDir, platform, data);
@@ -16,6 +17,9 @@ export class PrepareData extends ControllerDataBase {
 			hmr: data.hmr || data.useHotModuleReload
 		};
 		this.watch = data.watch;
+		if (_.isBoolean(data.watchNative)) {
+			this.watchNative = data.watchNative;
+		}
 	}
 }
 

lib/definitions/prepare.d.ts

@@ -7,6 +7,7 @@ declare global {
 		hmr: boolean;
 		env: any;
 		watch?: boolean;
+		watchNative: boolean
 	}
 
 	interface IiOSCodeSigningData {