Skip to content

Patch truffle to ignore pre-release compiler warning.#3248

Merged
axic merged 1 commit intodevelopfrom
ignorePrerelease
Nov 29, 2017
Merged

Patch truffle to ignore pre-release compiler warning.#3248
axic merged 1 commit intodevelopfrom
ignorePrerelease

Conversation

@chriseth
Copy link
Contributor

@chriseth chriseth commented Nov 27, 2017

This is hacky but could do for now.

Fixes #3244 (hopefully)

Change request to truffle: ConsenSys-archive/truffle-compile#35

@axic
Copy link
Contributor

axic commented Nov 27, 2017

If we are including a patch I'd rather patch node_modules/solc/wrapper.js as we have way more control over the changes in there. I was however thinking about either creating a wrapper over solc or soljson (the latter is more messy due to the C-JS wrapping).

@axic axic merged commit 7c69d88 into develop Nov 29, 2017
@axic axic deleted the ignorePrerelease branch November 29, 2017 16:06
@axic axic removed the in progress label Nov 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants