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

Incorrect path for doc include range proof #283

Closed
vlopes11 opened this issue Jul 29, 2019 · 3 comments
Closed

Incorrect path for doc include range proof #283

vlopes11 opened this issue Jul 29, 2019 · 3 comments

Comments

@vlopes11
Copy link

Seems I'm having some problems to evaluate the path for the doc includes with the latest Rust.

error: couldn't read src/range_proof/../docs/range-proof-protocol.md: No such file or directory (os error 2)
 --> src/range_proof/mod.rs:2:18
  |
2 | #![doc(include = "../docs/range-proof-protocol.md")]
  |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ couldn't read file

The absolute path should consider one more level. Here is a pull request that fixes it.
#282

@hdevalence
Copy link
Contributor

Thanks for spotting this -- I think that there might have been a change to doc import paths. Right now we're in a bit of a weird state because the travis config is using an outdated nightly, so I'll circle back to this after updating it.

@hdevalence
Copy link
Contributor

https://rust-lang.github.io/rustup-components-history/ suggests that all of the components we like (rls, fmt, etc) are available as of today's nightly, let's try upgrading.

@vlopes11
Copy link
Author

vlopes11 commented Aug 1, 2019

Tested and fixed. I'll close the PR + this issue.

Thanks!

@vlopes11 vlopes11 closed this as completed Aug 1, 2019
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

No branches or pull requests

2 participants