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

Fix a racecondition in unpacking proj-src #197

Merged

Conversation

weiznich
Copy link
Contributor

@weiznich weiznich commented Apr 4, 2024

Unpacking the proj source code to the cargo registry cache can run into potential data races for concurrent builds. This can be fixed by unpacking the source code into the build directory of the current build. This also ensures that we don't infer with a otherwise assumed static registry cache directory.

Unpacking the proj source code to the cargo registry cache can run
into potential data races for concurrent builds. This can be fixed by
unpacking the source code into the build directory of the current build.
This also ensures that we don't infer with a otherwise assumed static
registry cache directory.
@urschrei urschrei added this pull request to the merge queue Apr 4, 2024
Merged via the queue into georust:main with commit 54e973c Apr 4, 2024
27 checks passed
urschrei pushed a commit to kylebarron/proj that referenced this pull request Jul 1, 2024
Unpacking the proj source code to the cargo registry cache can run into
potential data races for concurrent builds. This can be fixed by
unpacking the source code into the build directory of the current build.
This also ensures that we don't infer with a otherwise assumed static
registry cache directory.
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.

3 participants