Sender | Message | Time |
---|---|---|
12 Jan 2021 | ||
jnaviasky | looks like EVM is pegged to minor version 0.7 but the new substrate uses 0.8 | 17:30:10 |
jnaviasky | * looks like EVM is pegged to minor version 0.7 of primitive-types but the new substrate uses 0.8 | 17:30:42 |
jnaviasky | Wei Tang: hi, wondering if you had any idea about getting this in: https://github.com/rust-blockchain/ethereum/pull/7 -- once this is in, we can bump the rust-blockchain/evm types and then fix these frontier build issues | 18:00:24 |
Wei Tang | Done! Will handle releases tomorrow! | 22:12:55 |
13 Jan 2021 | ||
jnaviasky | excellent, looking forward to it | 15:58:55 |
jnaviasky | Redacted or Malformed Event | 17:25:25 |
jnaviasky | trying to understand how the gasometer's working. i have a contract that spawns a sub-contract in one of its methods. here's a log snippet from the evm when i estimate that method's gas:
| 18:16:09 |
jnaviasky | notice that the gas drops by 67m at the Create opcode, and it's not reflected in the used gas until after the return -- but why such a big number? that seems like hugely more than 32000 + memory_cost as defined in the constants/logic | 18:16:51 |
jnaviasky | I believe this is the same sort of thing as raised here: https://github.com/paritytech/frontier/pull/252#issuecomment-754700523 | 18:17:22 |
Joshy Orndorff invited notlesh. | 18:54:33 | |
notlesh joined the room. | 18:54:43 | |
notlesh | 👋 | 18:54:57 |
Joshy Orndorff | In reply to @joshyorndorff:matrix.orgSo notlesh and I have spent a day diving in on this, and it seems the issue is that the ethereum crate's published version does not yet have a few dependencies updated. These updates are reflected in the ethereum crate's github repo thanks to the latest pull request merge. | 18:56:50 |
Joshy Orndorff | sorpaas: Would you be able to publish a release of ethereum with these updates? | 18:57:45 |
jnaviasky | I have a PR up on rust-blockchain/evm as well that should fix it there too | 18:57:48 |
jnaviasky | just need to wait for the publish | 18:57:54 |
jnaviasky | (this was also what i spent a day diving into yesterday, heh) | 18:58:35 |
notlesh | lol. | 18:58:50 |
notlesh | you must mean: https://github.com/rust-blockchain/evm/pull/7 | 18:58:52 |
jnaviasky | if you want a "for now" fix, I can give you my branches | 18:58:59 |
jnaviasky | yes | 18:59:00 |
notlesh | looks like CI failed: https://github.com/rust-blockchain/evm/pull/7/checks?check_run_id=1690296357 | 18:59:18 |
jnaviasky | the branch i was pegged to yesterday got deleted because it was merged in, so yeah the build of it will fail now | 19:00:08 |
notlesh | gotcha | 19:00:24 |
jnaviasky | or something weird happened with the git configuration -- but the code changes are pretty straightforward | 19:00:47 |
notlesh | in our case, we depend on ethereum which in turn depends on evm , so we'd need ethereum to bump its evm version once evm updates its ethereum-types version... | 19:00:50 |
jnaviasky | yes, same for me | 19:00:59 |
jnaviasky | I was using these overrides in my node though:
| 19:01:35 |
jnaviasky | * I was using these overrides in my node though:
| 19:01:59 |
notlesh | thanks for sharing, that might get us going for now 👍️ | 19:02:11 |