Skip to content
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.

Sign up for GitHub

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

Jump to bottom

forge build compile the YUL file with the error Field "settings.remappings" cannot be used for Yul. #7869

Closed
2 tasks done
gzeoneth opened this issue May 6, 2024 · 2 comments · Fixed by foundry-rs/compilers#122
Closed
2 tasks done

forge build compile the YUL file with the error Field "settings.remappings" cannot be used for Yul. #7869

gzeoneth opened this issue May 6, 2024 · 2 comments · Fixed by foundry-rs/compilers#122
Labels
T-bug Type: bug

Comments

@gzeoneth
Copy link

gzeoneth commented May 6, 2024

Component

Forge

Have you ensured that all of these are up to date?

  • Foundry
  • Foundryup

What version of Foundry are you on?

forge 0.2.0 (c486fca 2024-05-06T00:20:07.911403000Z)

What command(s) is the bug in?

forge build

Operating System

macOS (Apple Silicon)

Describe the bug

Same issue as reported in #3173

https://github.com/OffchainLabs/nitro-contracts/actions/runs/8970092142/job/24632823571?pr=170

$ FOUNDRY_PROFILE=yul forge build --skip *.sol
Compiling 1 files with Solc 0.8.9
Solc 0.8.9 finished in 2.78ms
Error: 
Compiler run failed:
Error: Field "settings.remappings" cannot be used for Yul.
@gzeoneth gzeoneth added the T-bug Type: bug label May 6, 2024
@gzeoneth gzeoneth mentioned this issue May 6, 2024
Merged
@gzeoneth
Copy link
Author

gzeoneth commented May 6, 2024

Looks like it is broken since this build
https://github.com/foundry-rs/foundry/releases/tag/nightly-aa333c6bebf43cb59de34cdf40b2e2bd70d98442

gzeoneth added a commit to OffchainLabs/bold that referenced this issue May 6, 2024
@gzeoneth
workaround: foundry issue 7869
9a21a97 
foundry-rs/foundry#7869
This was referenced May 6, 2024
Merged
Merged
@gzeoneth
Copy link
Author

gzeoneth commented May 6, 2024

We found a workaround

remappings = []
auto_detect_remappings = false

RISHABHAGRAWALZRA added a commit to availproject/nitro-contracts that referenced this issue May 9, 2024
@RISHABHAGRAWALZRA
fix: remapping issue for forge build compile the YUL file, foundry-rs…
ba5a33f 
…/foundry#7869, OffchainLabs@d615424, OffchainLabs@c36913f
@klkvr klkvr mentioned this issue May 10, 2024
Merged
mattsse pushed a commit to foundry-rs/compilers that referenced this issue May 13, 2024
@klkvr
fix: re-enable yul settings sanitization (#122)
b0c9baf 
Closes foundry-rs/foundry#7869
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T-bug Type: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: re-enable yul settings sanitization foundry-rs/compilers
1 participant
@gzeoneth