Skip to content

rust-analyzer: ide support#83

Closed
jakobjung10 wants to merge 2 commits into
probe-rs:masterfrom
jakobjung10:rust-analyzer
Closed

rust-analyzer: ide support#83
jakobjung10 wants to merge 2 commits into
probe-rs:masterfrom
jakobjung10:rust-analyzer

Conversation

@jakobjung10

Copy link
Copy Markdown
  • this commit reenables rust analyzer to show the Run Test | Debug buttons for each test case

- Prebuild esp-rom-sys by building the examples/esp32c6 manifest when present
- Copy generated native artifacts into each test's CARGO_TARGET_DIR to avoid parallel build/linker races
- Replace platform-specific shell copy with recursive Rust helper copy_dir_recursively()
- Normalize output line endings for cross-platform comparisons
- Minor portability and cleanup in tests-build/src/lib.rs
Comment thread tests-build/src/lib.rs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you comment on the changes in this file?
Were they required to make the tests pass on your machine?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yes, they were required to make the tests pass on my machine (windows).
However, the same 4 tests also failed on the CI run.
Also, these tests fail when checking out the latest commit on master.
Do they pass on your machine or is this a known problem?
I rather should have separated these 2 commits into separate PRs as they seem orthogonal.
I have no problem with discarding the latter commit as I originally did not mean to touch this logic anyways.

@jakobjung10 jakobjung10 Jun 1, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I have opened a standalone PR for the Rust-Analyzer feature in rust-analyzer: ide support.
If you are still interested in the other changes, I will create another separate PR.

@jakobjung10 jakobjung10 closed this Jun 1, 2026
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