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

ClassFiller.cc: do not use obsolete edmplugin::PluginCapabilities #9857

Closed
wants to merge 1 commit into from

Conversation

davidlt
Copy link
Contributor

@davidlt davidlt commented Jun 24, 2015

Capabilities (SEAL capabilities) files are being removed. Instead of
edmplugin::PluginCapabilities use edm::TypeWithDict to get
information about types/classes available in dictionaries.

ROOT will take care of loading the dictionary.

Should resolve:

===== Test "RunThis_t" ====
argument 0: RunThis_t
argument 1: /bin/bash
argument 2: IOPool/Streamer/test
argument 3: RunSimple_NewStreamer.sh
<..>
Failure cmsRun NewStreamIn_cfg.py: status 65
status = 16640

---> test RunThis_t had ERRORS

and

===== Test "RunThat_t" ====
<..>

MessageLogger Summary

Severity    # Occurrences   Total Occurrences
--------    -------------   -----------------
24-Jun-2015 03:41:00 CEST  Initiating request to open file file:multiprocess_test.dat
24-Jun-2015 03:41:00 CEST  Successfully opened file file:multiprocess_test.dat
24-Jun-2015 03:41:00 CEST  Closed file file:multiprocess_test.dat
----- Begin Fatal Exception 24-Jun-2015 03:41:00 CEST-----------------------
An exception of category 'PluginNotFound' occurred while
   [0] Constructing the EventProcessor
   [1] Constructing input source of type NewEventStreamFileReader
Exception Message:
Unable to find plugin 'LCGReflex/edm::StreamedProduct' because the category 'Capability' has no known plugins
----- End Fatal Exception -------------------------------------------------
Failure using streamer_multiprocess_cfg.py: status 65
status = 16640

---> test RunThat_t had ERRORS

https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc6_amd64_gcc491/CMSSW_7_6_DEVEL_X_2015-06-23-2300/unitTestLogs/IOPool/Streamer

@wmtan @Dr15Jones

Signed-off-by: David Abdurachmanov David.Abdurachmanov@cern.ch

Capabilities (SEAL capabilities) files are being removed. Instead of
`edmplugin::PluginCapabilities` use `edm::TypeWithDict` to get
information about types/classes available in dictionaries.

ROOT will take care of loading the dictionary.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
@davidlt
Copy link
Contributor Author

davidlt commented Jun 24, 2015

This also resolves another two failures in a different package, FWCore/Integration.

---> test TestIntegrationSeriesOfProcesses had ERRORS
---> test TestIntegrationThinningTests had ERRORS

https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc6_amd64_gcc491/CMSSW_7_6_DEVEL_X_2015-06-23-2300/unitTestLogs/FWCore/Integration

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @davidlt for CMSSW_7_6_X.

ClassFiller.cc: do not use obsolete edmplugin::PluginCapabilities

It involves the following packages:

IOPool/Streamer

@cmsbuild, @Dr15Jones can you please review it and eventually sign? Thanks.
@wddgit, @wmtan this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@smuzaffar
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

@wmtan
Copy link
Contributor

wmtan commented Jun 24, 2015

Merging this PR is optional, as #9872 is a superset of this PR.

@davidlt davidlt closed this Jun 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants