Merge pull request #692 from NativeScript/fatme/do-not-enumerate-inspector.app-files

Fatme · Fatme · commit f46027d41265 · 2015-07-24T16:29:58.000+03:00
Do not enumerate Inspector.app files
diff --git a/lib/npm-installation-manager.ts b/lib/npm-installation-manager.ts
@@ -116,7 +116,8 @@ export class NpmInstallationManager {
 	private isPackageUnpacked(packagePath: string): IFuture<boolean> {
 		return (() => {
 			return this.$fs.getFsStats(packagePath).wait().isDirectory() &&
-				this.$fs.enumerateFilesInDirectorySync(packagePath).length > 1;
+				this.$fs.exists(path.join(packagePath, "framework")).wait() && 
+				this.$fs.enumerateFilesInDirectorySync(path.join(packagePath, "framework")).length > 1;
 		}).future<boolean>()();
 	}	
 }

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,8 @@ export class NpmInstallationManager {`
`116`	`116`	`private isPackageUnpacked(packagePath: string): IFuture<boolean> {`
`117`	`117`	`return (() => {`
`118`	`118`	`return this.$fs.getFsStats(packagePath).wait().isDirectory() &&`
`119`		`- this.$fs.enumerateFilesInDirectorySync(packagePath).length > 1;`
	`119`	`+ this.$fs.exists(path.join(packagePath, "framework")).wait() &&`
	`120`	`+ this.$fs.enumerateFilesInDirectorySync(path.join(packagePath, "framework")).length > 1;`
`120`	`121`	`}).future<boolean>()();`
`121`	`122`	`}`
`122`	`123`	`}`