-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Added GetRunfilesDir() to be able to reliably determine prelude location under various invocation scenarios #1207
Conversation
…into fuzzer_runfiles
Now that I've had some time to check this: can you please provide a repro to show what this change is fixing? That should help me review this. |
$ bazel build -c opt executable_semantics/fuzzing:executable_semantics_fuzzer (missing prelude crash) |
…into fuzzer_runfiles
…into fuzzer_runfiles
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make sure to update the PR title/description before committing, as it's no longer GetRunfilesDir.
One of the scenarios used by the fuzzing framework is to place the binary and its runfiles into a folder, resulting in a directory layout like this:
[temp dir]
fuzzer
fuzzer.runfiles/
Tested: