Skip to content

Conversation

MozirDmitriy
Copy link
Contributor

This fix is necessary because the previous check validated the wrong variable, causing the script to erroneously fail when a valid fuzz cache path was provided (or pass when binpath existed but the fuzz cache path did not).

Arguments:

  • The option explicitly sets fuzzcachepath and later is passed to -test.fuzzcachedir, so the check must target that variable.
  • Docker usage creates and passes /fuzzcache to --fuzzcache-path, confirming the intended semantics.
  • Other flags (--binary-path, --nitro-path) correctly validate their own variables; this brings --fuzzcache-path in line with that pattern.

@eljobe eljobe self-assigned this Sep 22, 2025
@joshuacolvin0 joshuacolvin0 added this pull request to the merge queue Sep 22, 2025
Merged via the queue into OffchainLabs:master with commit b6bad4a Sep 22, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants