matter-labs / foundry-zksync

Fork of Foundry tailored for zkSync environment
Apache License 2.0
299 stars 130 forks source link

fix: bring back zkvm solc resolution from settings #548

Closed elfedy closed 2 months ago

elfedy commented 2 months ago

Motivation

foundry-compilers api refactor brought a regression regarding using zkvm solc when a solc requirement is specified as logic was lost in the new way to do compiler resolution.

Solution

Use dedicated method with zk specific solc resolution for determining how to assign ZkSolcCompiler's solc field.

Pending

This regression highlights the need to address the following in future PRs: