-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
bug(forge build
): Stack overflow caused by FOUNDRY_LIBS
or DAPP_LIBS
containing an absolute path
#9127
Open
2 tasks done
Labels
Comments
guidanoli
added
T-bug
Type: bug
T-needs-triage
Type: this issue needs to be labelled
labels
Oct 16, 2024
guidanoli
changed the title
Stack overflow caused by
Stack overflow caused by Oct 16, 2024
FOUNDRY_LIBS
or DAPP_LIBS
containing absolute pathsFOUNDRY_LIBS
or DAPP_LIBS
containing an absolute path
zerosnacks
changed the title
Stack overflow caused by
bug(Oct 16, 2024
FOUNDRY_LIBS
or DAPP_LIBS
containing an absolute pathforge build
): Stack overflow caused by FOUNDRY_LIBS
or DAPP_LIBS
containing an absolute path
zerosnacks
added
C-forge
Command: forge
Cmd-forge-build
Command: forge build
and removed
T-needs-triage
Type: this issue needs to be labelled
labels
Oct 16, 2024
Backtrace:
|
Happens because of this line which will end up trying to get remappings by recursing on itself
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Component
Forge
Have you ensured that all of these are up to date?
What version of Foundry are you on?
forge 0.2.0 (cc8e430 2024-10-16T00:22:08.572482504Z)
What command(s) is the bug in?
forge build
Operating System
Linux
Describe the bug
Steps to reproduce
FOUNDRY_LIBS
set to an array that contains an absolute path to a directory. Also works withDAPP_LIBS
. In this case, we use the absolute path to thelib
folder just created by theforge init
command.FOUNDRY_LIBS=["$(pwd)/lib"] forge build
The text was updated successfully, but these errors were encountered: