From fe105ef6ce4f9a81ef2dcf3f89cc2709fd140c44 Mon Sep 17 00:00:00 2001
From: Jeff Glass <glass.jeffrey@gmail.com>
Date: Wed, 12 Oct 2022 12:05:14 -0500
Subject: [PATCH] Use only single rollup-plugin-copy() to avoid race condition
 (#843)

* Use only single copy() plugin to avoid race condition

* clean production def, better copy_target structure

* Restore '!process.env.ROLLUP_WATCH to production definition

Co-authored-by: Ted Patrick <tpatrick@anaconda.com>
---
 pyscriptjs/rollup.config.js | 40 ++++++++++++++++++-------------------
 1 file changed, 19 insertions(+), 21 deletions(-)

diff --git a/pyscriptjs/rollup.config.js b/pyscriptjs/rollup.config.js
index fb629fe0fc6..9ac279599a3 100644
--- a/pyscriptjs/rollup.config.js
+++ b/pyscriptjs/rollup.config.js
@@ -1,6 +1,5 @@
 import commonjs from "@rollup/plugin-commonjs";
 import resolve from "@rollup/plugin-node-resolve";
-import livereload from "rollup-plugin-livereload";
 import { terser } from "rollup-plugin-terser";
 import legacy from '@rollup/plugin-legacy';
 import typescript from "@rollup/plugin-typescript";
@@ -11,15 +10,25 @@ import copy from 'rollup-plugin-copy'
 
 const production = !process.env.ROLLUP_WATCH || (process.env.NODE_ENV === "production");
 
+const copy_targets = {
+  targets: [
+    { src: 'public/index.html', dest: 'build' }
+  ]
+}
+
+if( !production ){
+  copy_targets.targets.push({ src: 'build/*', dest: 'examples/build' })
+}
+
 export default {
   input: "src/main.ts",
-  output: [
+  output:[
     {
-      sourcemap: true,
-      format: "iife",
-      inlineDynamicImports: true,
-      name: "app",
-      file: "build/pyscript.js",
+    sourcemap: true,
+    format: "iife",
+    inlineDynamicImports: true,
+    name: "app",
+    file: "build/pyscript.js",
     },
     {
       file: "build/pyscript.min.js",
@@ -46,23 +55,12 @@ export default {
       inlineSources: !production,
     }),
     // This will make sure that examples will always get the latest build folder
-    !production && copy({
-      targets: [
-        { src: 'build/*', dest: 'examples/build' },
-      ]
-    }),
-    copy({
-      targets: [
-        { src: 'public/index.html', dest: 'build' },
-      ]
-    }),
-    !production && serve(),
-    !production && livereload("public"),
+    copy(copy_targets),
     // production && terser(),
     !production && serve({
       port: 8080,
-      contentBase: 'examples'
-    })
+      contentBase: 'examples'}
+      )
   ],
   watch: {
     clearScreen: false,