search

cms-sw / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
1.08k stars 4.29k forks source link

Consider moving JobReport class out of FWCore/MessageLogger #45609

Open Dr15Jones opened 2 months ago

Dr15Jones commented 2 months ago

Nothing in the MessageLogger system uses JobReport and the two are logically distinct. Also, JobReport depends on tinyxml which is not needed by any part of the MessageLogger. Since DataFormats are allowed to depend on FWCore/MessageLogger, those packages inherit the unnecessary dependency on tinyxml.

Dr15Jones commented 2 months ago

assign core

cmsbuild commented 2 months ago

New categories assigned: core

@Dr15Jones,@makortel,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks

cmsbuild commented 2 months ago

cms-bot internal usage

cmsbuild commented 2 months ago

A new Issue was created by @Dr15Jones.

@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

Dr15Jones commented 2 months ago

In addition JobReport depends on DataFormats/Provenance while nothing in the MessageLogger system needs that dependency.