Bugfix #5614: When offline pruning, don't unlink prebuilt packages. (#5616)

theandrewlane · bestander · commit c84489685747 · 2018-04-04T13:37:44.000-07:00
diff --git a/src/cli/commands/install.js b/src/cli/commands/install.js
@@ -767,7 +767,9 @@ export class Install {
     const mirrorFiles = await fs.walk(mirror);
     for (const file of mirrorFiles) {
       const isTarball = path.extname(file.basename) === '.tgz';
-      if (isTarball && !requiredTarballs.has(file.basename)) {
+      // if using experimental-pack-script-packages-in-mirror flag, don't unlink prebuilt packages
+      const hasPrebuiltPackage = file.relative.startsWith('prebuilt/');
+      if (isTarball && !hasPrebuiltPackage && !requiredTarballs.has(file.basename)) {
         await fs.unlink(file.absolute);
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -767,7 +767,9 @@ export class Install {`
`767`	`767`	`const mirrorFiles = await fs.walk(mirror);`
`768`	`768`	`for (const file of mirrorFiles) {`
`769`	`769`	`const isTarball = path.extname(file.basename) === '.tgz';`
`770`		`- if (isTarball && !requiredTarballs.has(file.basename)) {`
	`770`	`+ // if using experimental-pack-script-packages-in-mirror flag, don't unlink prebuilt packages`
	`771`	`+ const hasPrebuiltPackage = file.relative.startsWith('prebuilt/');`
	`772`	`+ if (isTarball && !hasPrebuiltPackage && !requiredTarballs.has(file.basename)) {`
`771`	`773`	`await fs.unlink(file.absolute);`
`772`	`774`	`}`
`773`	`775`	`}`