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

Reactor runtime module excluded from the Rust test modules #1351

Merged
merged 1 commit into from
Sep 7, 2022

Conversation

oowekyala
Copy link
Collaborator

Including it causes version conflicts when one develops with a local copy of the runtime (using the env var LOCAL_RUST_REACTOR_RT env var). This problem was introduced when lfc started unpacking the runtime into src-gen.

For reference, here's an example failure caused by this:

error: package collision in the lockfile: packages reactor_rt v0.1.0 (/home/clem/Documents/LF/lingua-franca/test/Rust/src-gen/reactor-rs) and reactor_rt v0.1.0 (/home/clem/Documents/LF/reactor-rs) are different, but only one can be written to lockfile unambiguously

Including it causes version conflicts when one develops
with a local copy of the runtime (LOCAL_RUST_REACTOR_RT
env var).
Copy link
Member

@lhstrh lhstrh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@cmnrd cmnrd added rust Related to the Rust target bugfix labels Sep 7, 2022
@cmnrd cmnrd merged commit 6e7d4f0 into master Sep 7, 2022
@cmnrd cmnrd deleted the clem.fix-rust-test-path branch September 7, 2022 06:42
@lhstrh lhstrh changed the title Exclude reactor runtime module from the Rust test modules Reactor runtime module excluded from the Rust test modules Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix rust Related to the Rust target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants