ethereum / ethereum/populus

`StandardJSON` compilation backend does not correctly handle compiler settings

Open
#326 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
No language data
Stars
319
Forks
110
PR merge metrics
No merged PRs in 30d

Description

* Populus Version: 1.8.x
* OS: osx/linux/win

### What was wrong?

The `populus.compilation.backends.SolcStandardJSONBackend` does not correctly handle compiler settings.

Currently, the backend merges all compiler settings into the JSON input for compilation:

https://github.com/pipermerriam/populus/blob/3a0445e550baa344a0931a2b94f3c779c98505fe/populus/compilation/backends/solc_standard_json.py#L117

This is wrong as this config should only be updated with settings keys documented here: http://solidity.readthedocs.io/en/develop/using-the-compiler.html?highlight=standard-json#input-description

* `remappings`
* `optimizer`
* `metadata`
* `libraries`
* `outputSelection`

The rest of the settings should be passed into `compile_standard` as `**kwargs`

#### Cute Animal Picture

> put a cute animal picture here.

![contest-newborn-giraffe](https://user-images.githubusercontent.com/824194/28979282-7953b74a-7906-11e7-874d-eefcaf86724f.jpg)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.