Skip to content
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

Different results on 64 and 32 bit #8022

Closed
ellert opened this issue Sep 9, 2020 · 9 comments
Closed

Different results on 64 and 32 bit #8022

ellert opened this issue Sep 9, 2020 · 9 comments
Labels

Comments

@ellert
Copy link
Contributor

ellert commented Sep 9, 2020

This is a forwarding of downstream Fedora bug https://bugzilla.redhat.com/show_bug.cgi?id=1814356

Starting with doxygen 1.8.17, different results are generated on 32 bit and 64 bit architectures.
See the downstream bug report for a reproducer.

A git bisect between 1.8.16 and 1.8.17 using the reproducer in the downstream report identifies commit ae0a5ec as the one introducing the regression.

This regression is terribly annoying, since Fedora’s official build system koji checks that architecture independent subpackages contains the same set of files when built on different architectures. This regression means that documentation built using doxygen no longer can be put into architecture independent packages.

@doxygen
Copy link
Owner

doxygen commented Sep 9, 2020

@ellert You say "starting with doxygen 1.8.17"; does this mean this problem is also/still present in new releases (e.g. 1.8.20) or did you only check 1.8.17?

@ellert
Copy link
Contributor Author

ellert commented Sep 9, 2020

Yes it is still present in 1.8.20.

@doxygen
Copy link
Owner

doxygen commented Sep 11, 2020

I'm able to reproduce the problem. It seems to happen on 64 bit Linux only so far. The 64bit version on MacOSX is producing the same output as the 32bit Linux version. Valgrind didn't show any wrong, so it needs further investigation...

@doxygen doxygen added the bug label Sep 11, 2020
@albert-github
Copy link
Collaborator

Just as a small extension to " It seems to happen on 64 bit Linux only so far. ", I see the same problem on Windows as well, the number of inheritance graphs is different.

@doxygen
Copy link
Owner

doxygen commented Sep 11, 2020

Please verify if the commit I just pushed fixes the problem.

@doxygen doxygen added the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Sep 11, 2020
@albert-github
Copy link
Collaborator

I tested on Windows and the both version give the same result.
I now see 2 inheritance graphs, one for block" and one for "gr::fcdproplus:fcd_impl" but I see no inheritance graph for "hier_block2" is this expected?

@doxygen
Copy link
Owner

doxygen commented Sep 12, 2020

@albert-github I think the validity of the output is indeed something that needs to be checked. The output was dependent on the order of processing, which it should not be. With the fix the order is at least deterministic, but may still not be correct.

@ellert
Copy link
Contributor Author

ellert commented Sep 12, 2020

Please verify if the commit I just pushed fixes the problem.

The commit fixes the issue. Thank you for the quick response.

insooth pushed a commit to insooth/doxygen that referenced this issue Sep 16, 2020
insooth pushed a commit to UmlautSoftwareDevelopmentAccount/doxygen that referenced this issue Sep 16, 2020
insooth added a commit to UmlautSoftwareDevelopmentAccount/doxygen that referenced this issue Sep 17, 2020
@doxygen
Copy link
Owner

doxygen commented Dec 27, 2020

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.0.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Dec 27, 2020
@doxygen doxygen closed this as completed Dec 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants