Made sure to not do the work copying if already done

gatsbyjs · Jul 19, 2017 · 188426f · 188426f
1 parent a81d972
commit 188426f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/packages/gatsby-transformer-sharp/package.json b/packages/gatsby-transformer-sharp/package.json
@@ -16,9 +16,8 @@
   "license": "MIT",
   "dependencies": {
     "bluebird": "^3.5.0",
-    "image-size": "^0.6.0",
-    "ncp": "^2.0.0",
-    "util.promisify": "^1.0.0"
+    "fs-extra": "^4.0.0",
+    "image-size": "^0.6.0"
   },
   "devDependencies": {
     "babel-cli": "^6.24.1"

diff --git a/packages/gatsby-transformer-sharp/src/extend-node-type.js b/packages/gatsby-transformer-sharp/src/extend-node-type.js
@@ -17,8 +17,7 @@ const {
 
 const sharp = require(`sharp`)
 const promisify = require('util.promisify');
-const ncp = require('ncp');
-const ncpAsync = promisify(ncp);
+const fsExtra = require('fs-extra');
 const sizeOf = require('image-size');
 const path = require('path');
 
@@ -86,7 +85,13 @@ module.exports = ({ type, pathPrefix, getNodeAndSavePathDependency }) => {
           `static/${imageName}`
         );
 
-        await ncpAsync(details.absolutePath, publicPath);
+        if (!fsExtra.existsSync(publicPath)) {
+          fsExtra.copy(details.absolutePath, publicPath, err => {
+            if (err) {
+              console.error(`error copying file`, err)
+            }
+          })
+        }
 
         return {
           width: dimensions.width,