Skip to content

Conversation

@ellert
Copy link
Contributor

@ellert ellert commented Sep 16, 2024

The files generated by doxygen whose file names contain the string unnamedN where N is a number are not generated using predictable names.

This happens when running doxygen using multiple threads.

Set NUM_PROC_THREADS=1 to work around this issue.

See: doxygen/doxygen#11138

The files generated by doxygen whose file names contain the string
unnamedN where N is a number are not generated using predictable
names.

This happens when running doxygen using multiple threads.

Set NUM_PROC_THREADS=1 to work around this issue.

See: doxygen/doxygen#11138
@amadio amadio added this to the 5.7.2 milestone Sep 16, 2024
@amadio
Copy link
Member

amadio commented Sep 16, 2024

How important is this for you? If consistency is important in the distro DEB/RPM packages, I'm ok to take this, although then users building the docs with doxygen Doxydoc will have slower behavior by default, which is not ideal.

@ellert
Copy link
Contributor Author

ellert commented Sep 16, 2024

The RPM package build fails on Fedora/EPEL if the documentation package is declared to be architecture independent (noarch) in the spec file without this change. The build server (koji) checks that the list of files in an architecture independent package is the same for the builds on all architectures.

Debian checks package build reproducibility by building a package twice with slightly different conditions (e.g. different date and time, different path to the source directory) and checks that produced packages are the same. If these tests fail the package is flagged as having non-reproducible builds. This is not a critical error that gets the package kicked out od Debian, but its nice to avoid.

I can of course keep this as a patch in the package builds if the change is not suitable for upstream.

@amadio
Copy link
Member

amadio commented Sep 30, 2024

Since this doesn't really affect CI running time anymore, let's just have the reproducible build be the default. Thanks for the patch.

@amadio amadio merged commit 6d2157a into xrootd:devel Sep 30, 2024
@ellert ellert deleted the reproducible-doxygen branch October 2, 2024 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants