Changeset 44282

Timestamp:

12/18/2018 03:13:58 AM (6 years ago)

Author:

SergeyBiryukov

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.
Merges [43931] and [43932] to trunk.
See #45201.

Location:

trunk

Files:

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

trunk/Gruntfile.js

@@ -106 +106 @@
             plugins: [BUILD_DIR + 'wp-content/plugins'],
             themes: [BUILD_DIR + 'wp-content/themes'],
-            all: cleanFiles,
+            all: [
+                cleanFiles,
+                SOURCE_DIR + 'wp-includes/js/dist',
+                SOURCE_DIR + 'wp-includes/css/dist'
+            ],
             js: [BUILD_DIR + 'wp-admin/js/', BUILD_DIR + 'wp-includes/js/'],
             dynamic: {
@@ -139 +143 @@
                             '!.{svn,git}', // Exclude version control folders.
                             '!wp-includes/version.php', // Exclude version.php
+                            '!**/*.map', // The build doesn't need .map files.
                             '!index.php', '!wp-admin/index.php',
                             '!_index.php', '!wp-admin/_index.php'

trunk/tools/webpack/packages.js

@@ -269 +269 @@
     }
 
+    if ( env.forceBuildTarget ) {
+        delete config.devtool;
+        config.mode = 'production';
+        config.optimization = {
+            minimize: false
+        };
+    }
+
     if ( config.mode === 'development' ) {
         config.plugins.push( new LiveReloadPlugin( { port: process.env.WORDPRESS_LIVE_RELOAD_PORT || 35729 } ) );