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

Link in README points to "Path not found" #120

Open
karczex opened this issue Nov 6, 2019 · 2 comments
Open

Link in README points to "Path not found" #120

karczex opened this issue Nov 6, 2019 · 2 comments

Comments

@sscargal

This comment has been minimized.

@stellarhopper

This comment has been minimized.

Copy link
Member

@stellarhopper stellarhopper commented Nov 6, 2019

Ah, the original link broke because libnvdimm-for-next has moved the documentation to the driver-api 'book':

ae4a05027e2f ("docs: nvdimm: add it to the driver-api book")

The master link will also break the next time master moves forward to catch up with that change.
The correct link should be: https://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm.git/tree/Documentation/driver-api/nvdimm/nvdimm.rst?h=libnvdimm-for-next

Or indeed we can take the chance to point to the new Sphinx rendered HTML pages:
https://www.kernel.org/doc/html/latest/driver-api/nvdimm/index.html

I'll send a patch - @karczex if you can send me a "Name <email>" I'd be happy to add Reported-by attribution.

stellarhopper added a commit that referenced this issue Nov 12, 2019
Commit ae4a05027e2f ("docs: nvdimm: add it to the driver-api book") in
the kernel changes the location of the NVDIMM documentation. Use
this chance to update the documentation link in ndctl to the Sphinx
rendered HTML documentation hosted on kernel.org.

Link: #120
Cc: Dan Williams <dan.j.williams@intel.com>
Reviewed-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.