aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorF. Eugene Aumson <feuGeneA@users.noreply.github.com>2018-10-03 03:00:35 +0800
committerF. Eugene Aumson <feuGeneA@users.noreply.github.com>2018-10-03 05:50:58 +0800
commit98c1952956d87adf5e73a42a1b78b8bf8b4119d2 (patch)
tree55fb830653e12438e4f9c6ddac78404e8e47d986
parentf1ecb8c5cb28a0a7ca6f7ad2ff11194091df62a4 (diff)
downloaddexon-sol-tools-98c1952956d87adf5e73a42a1b78b8bf8b4119d2.tar
dexon-sol-tools-98c1952956d87adf5e73a42a1b78b8bf8b4119d2.tar.gz
dexon-sol-tools-98c1952956d87adf5e73a42a1b78b8bf8b4119d2.tar.bz2
dexon-sol-tools-98c1952956d87adf5e73a42a1b78b8bf8b4119d2.tar.lz
dexon-sol-tools-98c1952956d87adf5e73a42a1b78b8bf8b4119d2.tar.xz
dexon-sol-tools-98c1952956d87adf5e73a42a1b78b8bf8b4119d2.tar.zst
dexon-sol-tools-98c1952956d87adf5e73a42a1b78b8bf8b4119d2.zip
fix: persist artifacts with only relevant sources
https://github.com/0xProject/0x-monorepo/pull/1108 https://app.asana.com/0/684263176955174/842516551768097/f
-rw-r--r--packages/sol-compiler/src/compiler.ts57
1 files changed, 52 insertions, 5 deletions
diff --git a/packages/sol-compiler/src/compiler.ts b/packages/sol-compiler/src/compiler.ts
index a29367485..350a5a125 100644
--- a/packages/sol-compiler/src/compiler.ts
+++ b/packages/sol-compiler/src/compiler.ts
@@ -296,13 +296,12 @@ export class Compiler {
compilerOutput: solc.StandardOutput,
): Promise<void> {
const compiledContract = compilerOutput.contracts[contractPath][contractName];
- const sourceCodes = _.mapValues(
- compilerOutput.sources,
- (_1, sourceFilePath) => this._resolver.resolve(sourceFilePath).source,
- );
+
+ const { sourceCodes, sources } = this._getSourcesWithDependencies(contractPath);
+
const contractVersion: ContractVersionData = {
compilerOutput: compiledContract,
- sources: compilerOutput.sources,
+ sources,
sourceCodes,
sourceTreeHashHex,
compiler: {
@@ -333,6 +332,54 @@ export class Compiler {
await fsWrapper.writeFileAsync(currentArtifactPath, artifactString);
logUtils.warn(`${contractName} artifact saved!`);
}
+ private _getSourcesWithDependencies(
+ contractPath: string,
+ ): { sourceCodes: { [sourceName: string]: string }; sources: { [sourceName: string]: { id: number } } } {
+ const sources = { [contractPath]: { id: 0 } };
+ const sourceCodes = { [contractPath]: this._resolver.resolve(contractPath).source };
+ this._recursivelyGatherDependencySources(contractPath, sourceCodes[contractPath], sources, sourceCodes, 1);
+ return { sourceCodes, sources };
+ }
+ private _recursivelyGatherDependencySources(
+ contractPath: string,
+ contractSource: string,
+ sourcesToAppendTo: { [sourceName: string]: { id: number } },
+ sourceCodesToAppendTo: { [sourceName: string]: string },
+ nextId: number,
+ ): number {
+ let nextId_ = nextId;
+
+ const importStatementMatches = contractSource.match(/import[^;]*;/g);
+ if (importStatementMatches === null) {
+ return nextId_;
+ }
+ for (const importStatementMatch of importStatementMatches) {
+ const importPathMatches = importStatementMatch.match(/\"([^\"]*)\"/);
+ if (importPathMatches === null || importPathMatches.length === 0) {
+ continue;
+ }
+
+ let importPath = importPathMatches[1];
+ const lastPathSeparatorPos = contractPath.lastIndexOf('/');
+ const importFolder = lastPathSeparatorPos === -1 ? '' : contractPath.slice(0, lastPathSeparatorPos + 1);
+ importPath = importPath.slice(0, 2) === './' ? importPath.replace(/^.\//, importFolder) : importPath;
+
+ if (_.isUndefined(sourcesToAppendTo[importPath])) {
+ sourcesToAppendTo[importPath] = { id: nextId_ };
+ sourceCodesToAppendTo[importPath] = this._resolver.resolve(importPath).source;
+ nextId_ += 1;
+
+ nextId_ = this._recursivelyGatherDependencySources(
+ importPath,
+ this._resolver.resolve(importPath).source,
+ sourcesToAppendTo,
+ sourceCodesToAppendTo,
+ nextId_,
+ );
+ }
+ }
+ return nextId_;
+ }
private _compile(solcInstance: solc.SolcInstance, standardInput: solc.StandardInput): solc.StandardOutput {
const compiled: solc.StandardOutput = JSON.parse(
solcInstance.compileStandardWrapper(JSON.stringify(standardInput), importPath => {