-
-
Notifications
You must be signed in to change notification settings - Fork 198
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
Does not work with sphinx 1.8.1 #394
Comments
Do you have an example for reproducing this? I cannot reproduce it in a small test project. |
I've used the breathe project itself. Full output:
|
This looks like a problem in Sphinx with the recent proper support for overloaded functions. Can you open a bug over there? (and bonus points for creating a trimmed down version of the Breathe docs that reproduces the problem :-)) |
@jakobandersen Well not so straightforward, but here you go -> sphinx-doc/sphinx#5496. |
Thanks a lot to both of you for doing all the research to solve this, I will monitor upstream issue too. |
@melvinvermeeren Sure, glad to help. We are using it for a lot of our projects. For example shameless self promotion. |
Appears this is fixed in upstream PR sphinx-doc/sphinx#5526, probably a good idea to validate it here too. (I'll probably check this within a week, bit busy right now.) |
@melvinvermeeren This particular assertion is fixed. I still get a gazillon errors/warnings when running make though. But that is something for a different issue. I'll update the PR once sphinx 1.8.2 is released. |
For reference: fixed in #410. |
When calling make from the toplevel folder I get
I have the same issue on linux.
The text was updated successfully, but these errors were encountered: