aboutsummaryrefslogtreecommitdiffstats
path: root/packages
diff options
context:
space:
mode:
authorLeonid Logvinov <logvinov.leon@gmail.com>2018-03-16 00:08:48 +0800
committerLeonid Logvinov <logvinov.leon@gmail.com>2018-03-21 22:11:41 +0800
commitf45191d0e81e3ad3873e78c3891fdd5f9e9fd55c (patch)
tree4159dd919642790360a8501eab300e98fb9097f3 /packages
parent32feadee424b1ea8d96073d63790f5c5731dd30e (diff)
downloaddexon-sol-tools-f45191d0e81e3ad3873e78c3891fdd5f9e9fd55c.tar.gz
dexon-sol-tools-f45191d0e81e3ad3873e78c3891fdd5f9e9fd55c.tar.zst
dexon-sol-tools-f45191d0e81e3ad3873e78c3891fdd5f9e9fd55c.zip
Support proper semver version ranges
Diffstat (limited to 'packages')
-rw-r--r--packages/deployer/package.json2
-rw-r--r--packages/deployer/src/compiler.ts35
-rw-r--r--packages/deployer/src/utils/types.ts2
3 files changed, 23 insertions, 16 deletions
diff --git a/packages/deployer/package.json b/packages/deployer/package.json
index e6e8dcc4b..3982c0067 100644
--- a/packages/deployer/package.json
+++ b/packages/deployer/package.json
@@ -33,6 +33,7 @@
"@0xproject/monorepo-scripts": "^0.1.14",
"@0xproject/tslint-config": "^0.4.12",
"@types/require-from-string": "^1.2.0",
+ "@types/semver": "^5.5.0",
"@types/yargs": "^11.0.0",
"chai": "^4.0.1",
"copyfiles": "^1.2.0",
@@ -54,6 +55,7 @@
"isomorphic-fetch": "^2.2.1",
"lodash": "^4.17.4",
"require-from-string": "^2.0.1",
+ "semver": "^5.5.0",
"solc": "^0.4.18",
"web3": "^0.20.0",
"web3-eth-abi": "^1.0.0-beta.24",
diff --git a/packages/deployer/src/compiler.ts b/packages/deployer/src/compiler.ts
index 4563752f6..942777458 100644
--- a/packages/deployer/src/compiler.ts
+++ b/packages/deployer/src/compiler.ts
@@ -5,6 +5,7 @@ import 'isomorphic-fetch';
import * as _ from 'lodash';
import * as path from 'path';
import * as requireFromString from 'require-from-string';
+import * as semver from 'semver';
import solc = require('solc');
import * as Web3 from 'web3';
@@ -23,7 +24,7 @@ import {
import { utils } from './utils/utils';
const ALL_CONTRACTS_IDENTIFIER = '*';
-const SOLIDITY_VERSION_REGEX = /(?:solidity\s\^?)(\d+\.\d+\.\d+)/;
+const SOLIDITY_VERSION_RANGE_REGEX = /pragma solidity (.*);/;
const SOLIDITY_FILE_EXTENSION_REGEX = /(.*\.sol)/;
const IMPORT_REGEX = /(import\s)/;
const DEPENDENCY_PATH_REGEX = /"([^"]+)"/; // Source: https://github.com/BlockChainCompany/soljitsu/blob/master/lib/shared.js
@@ -85,10 +86,10 @@ export class Compiler {
private static _getContractSpecificSourceData(source: string): ContractSpecificSourceData {
const dependencies: string[] = [];
const sourceHash = ethUtil.sha3(source);
- const solcVersion = Compiler._parseSolidityVersion(source);
+ const solcVersionRange = Compiler._parseSolidityVersionRange(source);
const contractSpecificSourceData: ContractSpecificSourceData = {
dependencies,
- solcVersion,
+ solcVersionRange,
sourceHash,
};
const lines = source.split('\n');
@@ -105,17 +106,17 @@ export class Compiler {
return contractSpecificSourceData;
}
/**
- * Searches Solidity source code for compiler version.
+ * Searches Solidity source code for compiler version range.
* @param source Source code of contract.
- * @return Solc compiler version.
+ * @return Solc compiler version range.
*/
- private static _parseSolidityVersion(source: string): string {
- const solcVersionMatch = source.match(SOLIDITY_VERSION_REGEX);
- if (_.isNull(solcVersionMatch)) {
- throw new Error('Could not find Solidity version in source');
+ private static _parseSolidityVersionRange(source: string): string {
+ const solcVersionRangeMatch = source.match(SOLIDITY_VERSION_RANGE_REGEX);
+ if (_.isNull(solcVersionRangeMatch)) {
+ throw new Error('Could not find Solidity version range in source');
}
- const solcVersion = solcVersionMatch[1];
- return solcVersion;
+ const solcVersionRange = solcVersionRangeMatch[1];
+ return solcVersionRange;
}
/**
* Normalizes the path found in the error message.
@@ -189,8 +190,12 @@ export class Compiler {
if (!shouldCompile) {
return;
}
-
- const fullSolcVersion = binPaths[contractSpecificSourceData.solcVersion];
+ const availableCompilerVersions = _.keys(binPaths);
+ const solcVersion = semver.maxSatisfying(
+ availableCompilerVersions,
+ contractSpecificSourceData.solcVersionRange,
+ );
+ const fullSolcVersion = binPaths[solcVersion];
const compilerBinFilename = path.join(__dirname, '../../solc_bin', fullSolcVersion);
let solcjs: string;
const isCompilerAvailableLocally = fs.existsSync(compilerBinFilename);
@@ -208,7 +213,7 @@ export class Compiler {
}
const solcInstance = solc.setupMethods(requireFromString(solcjs, compilerBinFilename));
- logUtils.log(`Compiling ${fileName}...`);
+ logUtils.log(`Compiling ${fileName} with Solidity v${solcVersion}...`);
const source = this._contractSources[fileName];
const input = {
[fileName]: source,
@@ -243,7 +248,7 @@ export class Compiler {
const sources = _.keys(compiled.sources);
const updated_at = Date.now();
const contractNetworkData: ContractNetworkData = {
- solc_version: contractSpecificSourceData.solcVersion,
+ solc_version: solcVersion,
keccak256: sourceHash,
source_tree_hash: sourceTreeHash,
optimizer_enabled: this._optimizerEnabled,
diff --git a/packages/deployer/src/utils/types.ts b/packages/deployer/src/utils/types.ts
index 0068faf6a..f33f8f66a 100644
--- a/packages/deployer/src/utils/types.ts
+++ b/packages/deployer/src/utils/types.ts
@@ -84,7 +84,7 @@ export interface ContractSourceData {
export interface ContractSpecificSourceData {
dependencies: string[];
- solcVersion: string;
+ solcVersionRange: string;
sourceHash: Buffer;
sourceTreeHashIfExists?: Buffer;
}