Merge pull request #2000 from aleksandrychev/ENT-13273

Removed node_modules after build

Author: aleksandrychev

build-scripts/bootstrap-tarballs

@@ -137,8 +137,8 @@ log_debug "Installing javascript npm dependencies..."
         run_and_print_on_failure npm ci --prefix "$BASEDIR"/mission-portal/public/scripts/
         # build react components
         run_and_print_on_failure npm run build --prefix "$BASEDIR"/mission-portal/public/scripts/
-        # remove the packages specified in devDependencies
-        run_and_print_on_failure npm prune --omit=dev --prefix "$BASEDIR"/mission-portal/public/scripts/
+        # remove node_modules since the bundles are already built
+        run_and_print_on_failure rm -rf "$BASEDIR"/mission-portal/public/scripts/node_modules
     fi
 )
 

ci/setup-projects.sh

@@ -16,8 +16,9 @@ if test -f "mission-portal/public/scripts/package.json"; then
 	npm ci
 	# build react components
 	npm run build
-	# remove the packages specified in devDependencies
-	npm prune --omit=dev
+	# remove node_modules since the bundles are already built
+	# we do not need them to be presented in the package
+	rm -rf node_modules
 fi
 )
 

packaging/common/cfengine-hub/postinstall.sh

@@ -997,10 +997,7 @@ chmod 0700 -R $PREFIX/httpd/htdocs/ldap/config
 
 # changed permissions and owner of PHP and JS dependencies
 chown root:$MP_APACHE_USER -R $PREFIX/httpd/htdocs/vendor
-chown root:$MP_APACHE_USER -R $PREFIX/httpd/htdocs/public/scripts/node_modules
-
 chmod -R ug=rX,o= $PREFIX/httpd/htdocs/vendor # 440 for files, 550 for dirs
-chmod -R ug=rX,o= $PREFIX/httpd/htdocs/public/scripts/node_modules
 
 ##
 # Start Apache server (and php-fpm if present)