refactor(@angular/cli): remove webpack nodeJS globals shimming

BREAKING CHANGE: Libraries/Packages must contain a web version if not web specific.
angular · Feb 28, 2018 · c7a8b61 · c7a8b61 · vecernik · Mar 5, 2018
1 parent 2543d4f
commit c7a8b61
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 16 deletions.
diff --git a/packages/@angular/cli/models/webpack-configs/browser.ts b/packages/@angular/cli/models/webpack-configs/browser.ts
@@ -124,16 +124,6 @@ export function getBrowserConfig(wco: WebpackConfigOptions) {
         deployUrl: buildOptions.deployUrl,
       }),
     ]),
-    node: {
-      fs: 'empty',
-      global: true,
-      crypto: 'empty',
-      tls: 'empty',
-      net: 'empty',
-      process: true,
-      module: false,
-      clearImmediate: false,
-      setImmediate: false
-    }
+    node: false,
   };
 }
diff --git a/packages/@angular/cli/models/webpack-configs/common.ts b/packages/@angular/cli/models/webpack-configs/common.ts
@@ -271,11 +271,6 @@ export function getCommonConfig(wco: WebpackConfigOptions) {
             warnings: buildOptions.verbose,
             safari10: true,
             compress: {
-              // Disabled because of an issue with Mapbox GL when using the Webpack node global:
-              // https://github.com/mapbox/mapbox-gl-js/issues/4359#issuecomment-303880888
-              // https://github.com/angular/angular-cli/issues/5804
-              // https://github.com/angular/angular-cli/pull/7931
-              typeofs : false,
               pure_getters: buildOptions.buildOptimizer,
               // PURE comments work best with 3 passes.
               // See https://github.com/webpack/webpack/issues/2899#issuecomment-317425926.