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.

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

out: Calculate cache path based on workspace path hash #38

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

eszpotanski
Copy link
Contributor

This PR modifies behavior of out script, so instead of choosing the newest Bazel cache folder, its name is calculated as MD5 hash of workspace root path, as described in Bazel docs.
Moreover, recurse function was fixed as it's searched only one (the newest) branch, even if it was empty.

@oharboe oharboe merged commit 51326a2 into The-OpenROAD-Project:main Apr 15, 2024
3 of 5 checks passed
@oharboe
Copy link
Collaborator

oharboe commented Apr 15, 2024

Will test at next oportunity. Merging now does no harm, because what is on master doesn't work.

@eszpotanski eszpotanski deleted the log-watcher-hash-path branch June 20, 2024 10:23
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.

2 participants