-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
4.1.0 release: Invalid C++ declaration: Expected identifier in nested name #9433
Comments
Thanks for reporting. However, I can't immediately reproduce the problem (though with Doxygen 1.9.1). breathe_debug_trace_directives = True It looks like Breathe is feeding Sphinx some strange declarations, so that config var should log a bit more of what is going on. |
@jakobandersen sorry, that's because https://github.com/microsoft/CCF/pull/2787/files was merged, which pins sphinx to <4.1.0 Full stack:
Breathe debug output
|
Thanks for the call stack. It confirms my suspicion that it's not on the Sphinx side that things go wrong. Please open an issue over at Breathe, then we can continue the discussion there. |
@jakobandersen I've built a copy of doxygen 1.9.1 locally, and using that with the latest sphinx + breathe does work correctly. I presume that this is a bug in doxygen that was fixed between 1.8.13 and 1.9.1. We will upgrade, sorry about the noise. |
No problem, it's tricky to see what is going on, and in this case Breathe should have caught the exception and done something else. I'm not completely sure if a newer Doxygen will fix it, and it may be that breathe-doc/breathe#711 is needed. But feel free to open issues. |
@jakobandersen thank you for the |
@jakobandersen Thanks for the quick response! As @achamayou has already mentioned, we had merged microsoft/CCF#2787 by the time you'd tested it. We'll move further comments about this to the Breathe project 👍 |
fix #847. See also sphinx-doc/sphinx#9433.
see sphinx-doc/sphinx#9433 Co-Authored-By: Llewellyn Falco <isidore@users.noreply.github.com>
Describe the bug
After the recent upgrade to 4.1.0, we now get the following error when building documentation for the CCF project (full error in our CI):
This happens when reading heavily-templated C++ sources (
src/kv/
folder), documented with Doxygen (1.8.13).How to Reproduce
Expected behavior
No response
Your project
https://github.com/microsoft/CCF
Screenshots
No response
OS
Linux
Python version
3.8.9
Sphinx version
4.1.0
Sphinx extensions
No response
Extra tools
Doxygen
Additional context
No response
The text was updated successfully, but these errors were encountered: