-
Notifications
You must be signed in to change notification settings - Fork 191
Update xerces-c to version 3.3.0 #9866
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
Conversation
cms-bot internal usage |
please test workflow 145.2,145.4,145.5,146.101 for CMSSW_15_1_ROOT6_X/el8_aarch64_gcc12 |
-1 Failed Tests: UnitTests RelVals AddOn The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: You can see more details here: Unit TestsI found 27 errors in the following unit tests: ---> test EoP had ERRORS ---> test testHLTFiltersDQMonitor had ERRORS ---> test PrimaryVertex had ERRORS and more ... RelVals
Expand to see more relval errors ...
AddOn Tests
Expand to see more addon errors ... |
Allow overriding the libibverbs configuration directory setting the environment variable VERBS_CONFIG_DIR. Set VERBS_CONFIG_DIR in the scram tool to point to the libibverbs configuration directory. Include the libibverbs configuration directory in the package. Update the libibverbs configuration to include the full path to the plugins, and relocate them in the post-install section. Include the user binaries in the package.
Include more libraries and tools in the ROCm package: - hipCUB - RCCL Merge hipRAND/rocRAND back into the base ROCm tool.
7aaa5a2
to
85c204c
Compare
This PR contains too many commits (4363 >= 240) and will not be processed. |
Pull request #9866 was updated. |
Pull request #9866 was updated. |
please test workflow 145.2,145.4,145.5,146.101 for CMSSW_15_1_ROOT6_X/el8_aarch64_gcc12 |
Is there an issue open about these? I have a vague recollection Xerces has thread-safety issues that need to be protected, and in the log it is being accessed concurrently. |
|
-1 Failed Tests: UnitTests RelVals AddOn The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Unit TestsI found 27 errors in the following unit tests: ---> test EoP had ERRORS ---> test testHLTFiltersDQMonitor had ERRORS ---> test PrimaryVertex had ERRORS and more ... RelVals
Expand to see more relval errors ...
AddOn Tests
Expand to see more addon errors ... |
We are observing occasional RelVal failures in ROOT6 IBs (first occurence) in Xerces-C code, either when loading predefined error message or when dealing with multibyte characters. Changelogs for Xerces-C do include references to similar issues, so maybe it's worth trying to update.
Notice that this update will force rebuild of at least Geant4, so maybe we should postpone it until we update Geant4 in master.