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
More verbose error if plugin could not be loaded due to missing shared library #27908
Conversation
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-27908/11713
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-27908/11714
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-27908/11715
|
A new Pull Request was created by @guitargeek (Jonas Rembser) for master. It involves the following packages: CondCore/AlignmentPlugins @ggovi, @smuzaffar, @cmsbuild, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
Pull request #27908 was updated. @ggovi, @smuzaffar, @cmsbuild, @Dr15Jones can you please check and sign again. |
please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@ggovi the db part of this PR looks a technical cleaning, could you please have a look? |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
The error message that you get if you delete plugin that is in the realease in your work area, and then try to use that plugin, is not very informative. The way this is handled is by referring to "poisoned" libraries in the work area, which actually don't exist. So you get some error like this:
However, it would be great (in particular to catch mistakes in cleanup PRs) if the message told you the name of the plugin it tried to load in the first place. I propose to add this information as a context to the exception (thanks @Dr15Jones for the implementation idea). The error message would look like this then:
We talked a bit about this already a little bit in PR #27522 (in particular #27522 (comment)).
While investigating how to do that I "git grepped" for "SharedLibrary" and therefore found by chance many places where
SharedLibrary.h
is included but not used. We could also remove these as a little side effect of this PR.PR validation:
CMSSW compiles, local matrix tests pass.
if this PR is a backport please specify the original PR:
No backport intended.