search

esp-rs / rust-build

Installation tools and workflows for deploying/building Rust fork esp-rs/rust with Xtensa and RISC-V support
MIT License
253 stars 33 forks source link

Include `rust-analyzer-proc-macro-srv` for all targets #223

Closed jessebraham closed 1 year ago

jessebraham commented 1 year ago

While #215 included rust-analyzer-proc-macro-srv for a few targets, the majority of targets did not receive this update. I have not run the build with these changes, so apologies if I've missed something here and these were omitted for a reason. But I believe this should all be okay.

georgik commented 1 year ago

I've added the changes in separate commit to build/1.71.0.0. macOS build seems to be correct, following file appears: ~/.rustup/toolchains/esp/libexec/rust-analyzer-proc-macro-srv

Windows GNU build is in progress. In case of Windows it was necessary to include rustc/libexec in Repackage script.

georgik commented 1 year ago

I've solved it as separate commit based on PR, because of some conflicts with main. Closing PR. Big thanks to @jessebraham for solving the issue.