Skip to content

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

Merged
merged 1 commit into from
Mar 7, 2023

Conversation

tautschnig
Copy link
Collaborator

  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.
  • 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
Copy link

codecov bot commented Mar 6, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (014188e) 78.50% compared to head (ccf1e61) 78.50%.

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.
📢 Do you have feedback about the report comment? Let us know in this issue.

@@ -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 .
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ℹ️ Removing this use of \subpage means that the Central Data Structures page is no longer considered a sub page of the architecture page which moves it to the root level in the navigation tree.
image

@@ -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).
Copy link
Contributor

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 -
image

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

Copy link
Collaborator Author

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.

@@ -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/).
Copy link
Contributor

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.

Copy link
Collaborator Author

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.
@tautschnig tautschnig force-pushed the bugfixes/7576-doc-links branch from f98fcbc to ccf1e61 Compare March 7, 2023 14:39
@tautschnig tautschnig merged commit 88e3145 into diffblue:develop Mar 7, 2023
@tautschnig tautschnig deleted the bugfixes/7576-doc-links branch March 7, 2023 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants