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

Added library as optional path for RUST_SRC_PATH #149

Conversation

rodested
Copy link

@rodested rodested commented Feb 21, 2021

Now dinamically tests if the std library can be found (and by this order) in:
~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src
~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library
and uses the found path for RUST_SRC_PATH.
Besides, if no path is found, shows ERROR messages in the messages buffer so that users get a clue of what is happening and how to fix it

Rodested.

Now dinamically tests if the std library can be found (and by this order) in: 
~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src
~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library
and uses the found path.
Besides, if no path is found, shows ERROR messages in the messages buffer so that users get a clue of what is happening and how to fix it
@rodested rodested changed the title Added library as optional path Added library as optional path for RUST_SRC_PATH Feb 21, 2021
@rodested rodested mentioned this pull request Feb 21, 2021
@Wilfred
Copy link
Member

Wilfred commented Mar 7, 2021

#143 is merged, which fixes the same issue. Thanks for the PR though :)

@Wilfred Wilfred closed this Mar 7, 2021
@rodested
Copy link
Author

rodested commented Mar 11, 2021

Hello,

Wilfred, would you consider adding this commit as an improvement to the committed #143 (see above).
This commit includes several improvements:

  • This one works with older and newer versions by checking dinamically the current path for the user
  • Provides error messages useful for the end user to solve it or to to give additional information if a new issue arises

I'm new merging and adding improvements to Emacs, as I am a recent adopter. I hope to be able to help more in the future :).

Regards,

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.

None yet

2 participants