Andy Fingerhut
Andy Fingerhut
@AdarshRawat1 These changes make sense to me regarding my confusions about running doxygen. Let me know if you see anything in these changes that is different than what you would...
@fruffy I am guessing the one failing test has nothing to do with changes to the README.md. I am leaving it there for now, in case you want to examine...
@AdarshRawat1 Could you please take a look at these changes and give any comments, or suggestions for further changes, that you think of?
There are some more broken links on the P4Smith page, in particular the ones with the link text listed below: + targets/generic + targets/bmv2 + targets/pna + targets/ebpf It looks...
All of the links reported in my original issue are working reasonably now in the version generated and published here https://p4lang.github.io/p4c/index.html after this PR was merged in: https://github.com/p4lang/p4c/pull/4793 Thus, closing...
I have verified that at least the version of changes as of commit 6 on #4793 fixes this issue.
Closing this issue, as the links are correct after this PR was merged in: https://github.com/p4lang/p4c/pull/4793
If they are kept separate, it would be nice in the doxygen output if they were adjacent to each other.
Closing this issue, as even though these two pages remain separate, for which there are good reasons to do so mentioned by Chris Dodd in his comment above, they are...
As of commit 6 of #4753, I have confirmed that all broken links mentioned in this issue _except_ the last one have been fixed.