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 racey bug at TLC execution #152

Merged
merged 2 commits into from
Dec 8, 2021
Merged

Conversation

rnbguy
Copy link
Member

@rnbguy rnbguy commented Dec 8, 2021

Closes: #148
Closes: #149
Closes: #150

Description

The change makes sure, the TLA+ standard modules are extracted in unique directory for each different TLC executions, avoiding racey file access.

Fixes the mbt test failures at informalsystems/ibc-rs.


For contributor use:

  • If applicable, unit tests are written and added to CI. (NA)
  • Ran go fmt, cargo fmt and etc. (or had formatting run automatically on all files edited)
  • Updated relevant documentation (docs/) and code comments. (NA)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Re-reviewed Files changed in the Github PR explorer.
  • Added a changelog entry, using unclog.

@rnbguy rnbguy marked this pull request as ready for review December 8, 2021 18:41
@rnbguy rnbguy merged commit b6ddedc into main Dec 8, 2021
@rnbguy rnbguy mentioned this pull request Dec 9, 2021
@danwt danwt deleted the rnbguy/rust/concurrent-tlc-runtime branch January 18, 2022 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant