Fix race condition

2 files changed, 12 insertions, 9 deletions

diff --git a/packages/deployer/CHANGELOG.md b/packages/deployer/CHANGELOG.md
index b184f41b9..f1e9d38ee 100644
--- a/packages/deployer/CHANGELOG.md
+++ b/packages/deployer/CHANGELOG.md
@@ -3,7 +3,7 @@
 ## v0.2.0 - _??_
 
     * Check dependencies when determining if contracts should be recompiled.
-    
+
 ## v0.1.0 - _February 16, 2018_
 
     * Add the ability to pass in specific contracts to compile in CLI (#400)
diff --git a/packages/deployer/src/compiler.ts b/packages/deployer/src/compiler.ts
index 6eca4b57f..5004d4bb4 100644
--- a/packages/deployer/src/compiler.ts
+++ b/packages/deployer/src/compiler.ts
@@ -168,7 +168,6 @@ export class Compiler {
             : Array.from(this._specifiedContracts.values());
         await Promise.all(_.map(fileNames, async fileName => this._setCompileActionAsync(fileName)));
         await Promise.all(_.map(fileNames, async fileName => this._compileContractAsync(fileName)));
-
         this._solcErrors.forEach(errMsg => {
             utils.consoleLog(errMsg);
         });
@@ -185,10 +184,6 @@ export class Compiler {
         if (!contractSpecificSourceData.shouldCompile) {
             return;
         }
-        const source = this._contractSourcesIfExists[fileName];
-        const input = {
-            [fileName]: source,
-        };
 
         const fullSolcVersion = binPaths[contractSpecificSourceData.solc_version];
         const solcBinPath = `./solc/solc_bin/${fullSolcVersion}`;
@@ -196,6 +191,10 @@ export class Compiler {
         const solcInstance = solc.setupMethods(solcBin);
 
         utils.consoleLog(`Compiling ${fileName}...`);
+        const source = this._contractSourcesIfExists[fileName];
+        const input = {
+            [fileName]: source,
+        };
         const sourcesToCompile = {
             sources: input,
         };
@@ -271,10 +270,14 @@ export class Compiler {
                     contractNetworkData.solc_version !== contractSpecificSourceData.solc_version;
             }
         }
-        _.forEach(contractSpecificSourceData.dependencies, async dependency => {
-            await this._setCompileActionAsync(dependency);
+        await Promise.all(
+            _.map(contractSpecificSourceData.dependencies, async dependency => this._setCompileActionAsync(dependency)),
+        );
+        _.forEach(contractSpecificSourceData.dependencies, dependency => {
             contractSpecificSourceData.shouldCompile =
-                contractSpecificSourceData.shouldCompile || this._contractSourceData[dependency].shouldCompile;
+                contractSpecificSourceData.shouldCompile ||
+                (this._contractSourceData[dependency].shouldCompile &&
+                    (this._specifiedContracts.has('*') || this._specifiedContracts.has(dependency)));
         });
     }
     /**