Changeset 43931

Timestamp:

11/22/2018 02:04:51 AM (6 years ago)

Author:

pento

Message:

Build Tools: Don't include .map files in the build.

These files are fairly large, and while they're useful in development, they're not needed in the final build.

Props pento, mcsf.
See #45201.

Location:

branches/5.0

Files:

• Gruntfile.js (modified) (2 diffs)
• tools/webpack/packages.js (modified) (1 diff)

branches/5.0/Gruntfile.js

@@ -72 +72 @@
         },
         clean: {
-            all: [BUILD_DIR],
+            all: [
+                BUILD_DIR,
+                SOURCE_DIR + 'wp-includes/js/dist',
+                SOURCE_DIR + 'wp-includes/css/dist'
+            ],
             dynamic: {
                 dot: true,
@@ -99 +103 @@
                             '!wp-includes/js/jquery/ui/*.js',
                             '!wp-includes/js/tinymce/tinymce.js',
-                            '!wp-includes/version.php' // Exclude version.php
+                            '!wp-includes/version.php', // Exclude version.php
+                            '!**/*.map' // The build doesn't need .map files.
                         ],
                         dest: BUILD_DIR

branches/5.0/tools/webpack/packages.js

@@ -265 +265 @@
     };
 
-    if ( config.mode !== 'production' ) {
+    if ( config.mode !== 'production' && ! env.forceBuildTarget ) {
         config.devtool = process.env.SOURCEMAP || 'source-map';
     }