Rename package.json to package.json.orig when copying it to build fol…

…der so it does not collide with schemas.
jupyterlab · Sep 5, 2018 · 728032a · 728032a
1 parent b3d5bb6
commit 728032a
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/buildutils/src/build.ts b/buildutils/src/build.ts
@@ -126,16 +126,13 @@ export namespace Build {
         // Copy schemas.
         schemas.forEach(schema => {
           const file = path.basename(schema);
-          if (file === 'package.json') {
-            throw new Error('Cannot use name "package.json" for schema file');
-          }
           fs.copySync(schema, path.join(destination, file));
         });
 
         // Write the package.json file for future comparison.
         fs.copySync(
           path.join(packageDir, 'package.json'),
-          path.join(destination, 'package.json')
+          path.join(destination, 'package.json.orig')
         );
       }
 
@@ -147,10 +144,9 @@ export namespace Build {
         // Remove the existing directory if necessary.
         if (fs.existsSync(destination)) {
           try {
-            const oldPackageData = require(path.join(
-              destination,
-              'package.json'
-            ));
+            const oldPackageData = utils.readJSONFile(
+              path.join(destination, 'package.json.orig')
+            );
             if (oldPackageData.version === packageData.version) {
               fs.removeSync(destination);
             }
@@ -168,7 +164,7 @@ export namespace Build {
         // Write the package.json file for future comparison.
         fs.copySync(
           path.join(packageDir, 'package.json'),
-          path.join(destination, 'package.json')
+          path.join(destination, 'package.json.orig')
         );
       }
     });