-
Notifications
You must be signed in to change notification settings - Fork 274
Fix documentation links #7578
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
Fix documentation links #7578
Conversation
tautschnig
commented
Mar 6, 2023
- Make sure .c source files are considered by Doxygen. (Notably, cprover_contracts.c contains relevant documentation.)
- Reference other files in ways understood by Doxygen.
- Each commit message has a non-empty body, explaining why the change was made.
- n/a Methods or procedures I have added are documented, following the guidelines provided in CODING_STANDARD.md.
- The feature or user visible behaviour I have added or modified has been documented in the User Guide in doc/cprover-manual/
- n/a Regression or unit tests are included, or existing tests cover the modified code (in this case I have detailed which ones those are in the commit message).
- n/a My commit message includes data points confirming performance improvements (if claimed).
- My PR is restricted to a single feature or bugfix.
- n/a White-space or formatting changes outside the feature-related changed lines are in commits of their own.
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## develop #7578 +/- ##
========================================
Coverage 78.50% 78.50%
========================================
Files 1670 1670
Lines 191767 191767
========================================
Hits 150548 150548
Misses 41219 41219 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@@ -61,7 +61,7 @@ transformation tools (see \ref other-tools). | |||
|
|||
For an explanation of the data structures involved in the modeling of a GOTO | |||
program (the GOTO Intermediate Representation used by CBMC and assorted tools) | |||
please see \subpage central-data-structures . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -61,7 +61,7 @@ transformation tools (see \ref other-tools). | |||
|
|||
For an explanation of the data structures involved in the modeling of a GOTO | |||
program (the GOTO Intermediate Representation used by CBMC and assorted tools) | |||
please see \subpage central-data-structures . | |||
please see [central data structures](central-data-structures.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change seems to break the link. When I build the documentation site locally it now links to here -
I think this link should not be updated as it seems to be working already for the generated website and is not included in the list of errors in this issue - #7576
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much for careful checking! It seems that some other warning got me confused. I have now reverted this change.
9207e52
to
f98fcbc
Compare
@@ -128,4 +128,4 @@ potential paths within it. | |||
|
|||
This is a high-level overview of symex and goto-program instructions. | |||
For more information (and a more robust introduction), please have a look | |||
at \ref symbolic-execution. | |||
at [symbolic-execution](../../src/goto-symex/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The old link here works for the built doxygen website, but doesn't work when viewing as github markdown. The new link works when viewing as github markdown, but not when viewing the doxygen website. I haven't been able to work out a link format which works both ways round.
The root of these problems as I understand it is that doxygen formatted input files are not github markdown formatted input files. Github markdown does not support doxygens tags and doxygen doesn't work with github markdown links. Therefore there are features for one file format which don't work for the other, making the files correct for one can result in broken links or raw tags being displayed in the other.
As I understand it, the build system attempts to work around this problem for the files in /doc/cprover-manual
by attempting to rewrite the links from github markdown format into doxygen format. But it can't be run over the rest of the docmentation tree as the pandoc + pythonscript based solution doesn't understand doxygen tags and will break them in the process. This includes the tags for dot diagrams and sub page organisation among others.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We already used links to ../../src/goto-symex/
several times in this file, but indeed those were just broken. I have now fixed all of them (and left the above as it was before).
1) Make sure .c source files are considered by Doxygen. (Notably, cprover_contracts.c contains relevant documentation.) 2) Reference other files in ways understood by Doxygen.
f98fcbc
to
ccf1e61
Compare