From 9ef8d8b85a27a85f560a0cc483b72f79c2d05ab8 Mon Sep 17 00:00:00 2001
From: Peter Bacon Darwin <pete@bacondarwin.com>
Date: Fri, 6 Oct 2017 10:48:32 +0100
Subject: [PATCH] build(aio): support ignoring dist packages in "local" mode
 (#19511)

PR Close #19511
---
 aio/tools/ng-packages-installer/index.js      | 12 +++++++++---
 aio/tools/ng-packages-installer/index.spec.js |  8 ++++++++
 2 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/aio/tools/ng-packages-installer/index.js b/aio/tools/ng-packages-installer/index.js
index 665500019d..38e8c8460d 100644
--- a/aio/tools/ng-packages-installer/index.js
+++ b/aio/tools/ng-packages-installer/index.js
@@ -37,6 +37,7 @@ class NgPackagesInstaller {
   constructor(projectDir, options = {}) {
     this.debug = options.debug;
     this.force = options.force;
+    this.ignorePackages = options.ignorePackages || [];
     this.projectDir = path.resolve(projectDir);
     this.localMarkerPath = path.resolve(this.projectDir, LOCAL_MARKER_PATH);
 
@@ -145,9 +146,13 @@ class NgPackagesInstaller {
       .map(filePath => filePath.slice(ANGULAR_DIST_PACKAGES.length + 1))
       .filter(filePath => PACKAGE_JSON_REGEX.test(filePath))
       .forEach(packagePath => {
-        const packageConfig = require(path.resolve(ANGULAR_DIST_PACKAGES, packagePath));
         const packageName = `@angular/${packagePath.slice(0, -PACKAGE_JSON.length -1)}`;
-        packageConfigs[packageName] = packageConfig;
+        if (this.ignorePackages.indexOf(packageName) === -1) {
+          const packageConfig = require(path.resolve(ANGULAR_DIST_PACKAGES, packagePath));
+          packageConfigs[packageName] = packageConfig;
+        } else {
+          this._log('Ignoring package', packageName);
+        }
       });
     this._log('Found the following Angular distributables:', Object.keys(packageConfigs).map(key => `\n - ${key}`));
     return packageConfigs;
@@ -216,8 +221,9 @@ function main() {
 
     .option('debug', { describe: 'Print additional debug information.', default: false })
     .option('force', { describe: 'Force the command to execute even if not needed.', default: false })
+    .option('ignore-packages', { describe: 'List of Angular packages that should not be used in local mode.', default: [], array: true })
 
-    .command('overwrite <projectDir> [--force] [--debug]', 'Install dependencies from the locally built Angular distributables.', () => {}, argv => {
+    .command('overwrite <projectDir> [--force] [--debug] [--ignore-packages package1 package2]', 'Install dependencies from the locally built Angular distributables.', () => {}, argv => {
       const installer = new NgPackagesInstaller(argv.projectDir, argv);
       installer.installLocalDependencies();
     })
diff --git a/aio/tools/ng-packages-installer/index.spec.js b/aio/tools/ng-packages-installer/index.spec.js
index a910c034a8..fa2e14b9e1 100644
--- a/aio/tools/ng-packages-installer/index.spec.js
+++ b/aio/tools/ng-packages-installer/index.spec.js
@@ -162,6 +162,14 @@ describe('NgPackagesInstaller', () => {
       expect(ngPackages['@angular/upgrade']).toBeDefined();
 
       expect(ngPackages['@angular/upgrade/static']).not.toBeDefined();
+
+      it('should not include packages that have been ignored', () => {
+        installer = new NgPackagesInstaller(rootDir, { ignorePackages: ['@angular/router'] });
+        const ngPackages = installer._getDistPackages();
+
+        expect(ngPackages['@angular/common']).toBeDefined();
+        expect(ngPackages['@angular/router']).toBeUndefined();
+      });
     });
   });