Skip to content

fix: fallback to rosetta for versions under natively compilable v0.8.5 #29

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 13, 2022

Conversation

roynalnaruto
Copy link
Contributor

Related to issue: ethereum/solidity#12784

This is a temporary fix to allow support for all solc versions for MacOS users. If the solc version in question is >= 0.8.5 (which have been natively compiled and hosted in solc-builds) then we use those. For all other versions under 0.8.5 we fallback to macos-amd64 builds that require Rosetta support.

We expect to eventually fix the compilation issues in the above mentioned GH issue, but for now this is the most reasonable solution to support as many versions as possible via Foundry

@roynalnaruto roynalnaruto merged commit ae79a29 into master Mar 13, 2022
@roynalnaruto roynalnaruto deleted the fix/macos-aarch64-conditional branch March 13, 2022 16:53
gakonst added a commit to gakonst/ethers-rs that referenced this pull request Mar 13, 2022
gakonst added a commit to foundry-rs/foundry that referenced this pull request Mar 13, 2022
Bumps SVM making Solidity compiler fall back to Rosetta for versions
before 0.8.5

ref:
alloy-rs/svm-rs#29
ethereum/solidity#12784
jsLover1117 added a commit to jsLover1117/ethers-rs that referenced this pull request Jul 17, 2022
charisma98 added a commit to charisma98/foundry that referenced this pull request Mar 4, 2023
Bumps SVM making Solidity compiler fall back to Rosetta for versions
before 0.8.5

ref:
alloy-rs/svm-rs#29
ethereum/solidity#12784
0129general added a commit to 0129general/FoundryProject that referenced this pull request May 8, 2024
Bumps SVM making Solidity compiler fall back to Rosetta for versions
before 0.8.5

ref:
alloy-rs/svm-rs#29
ethereum/solidity#12784
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.

1 participant