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
Clean BuildFiles in DataFormats #31459
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,4 @@ | ||
<use name="FWCore/Framework"/> | ||
<use name="DataFormats/CTPPSDetId"/> | ||
<use name="DataFormats/Common"/> | ||
<export> | ||
<lib name="1"/> | ||
</export> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,5 @@ | ||
<use name="DataFormats/CaloTowers"/> | ||
<use name="FWCore/Framework"/> | ||
<use name="FWCore/ParameterSet"/> | ||
<library file="CaloTowersDump.cc" name="CaloTowersDump"> | ||
<flags EDM_PLUGIN="1"/> | ||
</library> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,5 @@ | ||
<use name="FWCore/Framework"/> | ||
<use name="DataFormats/DTDigi"/> | ||
<use name="DataFormats/DetId"/> | ||
<bin name="testDTDigis" file="testDTDigis.cpp"> | ||
<use name="cppunit"/> | ||
<use name="DataFormats/FEDRawData"/> | ||
</bin> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,6 @@ | ||
<use name="root"/> | ||
<use name="DataFormats/Common"/> | ||
<use name="DataFormats/Candidate"/> | ||
<use name="CondFormats/HIObjects"/> | ||
<use name="CondFormats/DataRecord"/> | ||
<export> | ||
<lib name="1"/> | ||
</export> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,5 @@ | ||
<use name="DataFormats/CaloRecHit"/> | ||
<use name="DataFormats/Common"/> | ||
<use name="DataFormats/EgammaCandidates "/> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. hi @guitargeek - I don't understand this change. DataFormats/EgammaCandidates is used in 3 headers (all of which are then needed to build srcs in this package. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hello! My script is grepping for the exact dependency name, which was here "DataFormats/EgammaCandidates " with a whitespace in the end. By eye this whitespace is not so easy to see when scrolling through the diffs, so I was not prepared for this problem. I'll make sure to strip the witespaces before grepping next time! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ah..I also missed the whitespace.. that explains it:) Thanks. |
||
<use name="DataFormats/EgammaReco "/> | ||
<use name="DataFormats/MuonReco"/> | ||
<use name="DataFormats/TrackReco"/> | ||
<use name="DataFormats/TrajectorySeed"/> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,3 @@ | ||
<use name="FWCore/Utilities"/> | ||
<use name="DataFormats/Common"/> | ||
<export> | ||
<lib name="1"/> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,4 @@ | ||
<use name="DataFormats/Provenance"/> | ||
<use name="DataFormats/StdDictionaries"/> | ||
<export> | ||
<lib name="1"/> | ||
</export> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,5 @@ | ||
<use name="DataFormats/CaloRecHit"/> | ||
<use name="DataFormats/Common"/> | ||
<use name="DataFormats/Math"/> | ||
<use name="DataFormats/Candidate"/> | ||
<use name="DataFormats/VertexReco"/> | ||
<use name="FWCore/Utilities"/> | ||
<export> | ||
<lib name="1"/> | ||
</export> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,5 @@ | ||
<use name="DataFormats/Common"/> | ||
<use name="DataFormats/TrackReco"/> | ||
<use name="FWCore/Utilities"/> | ||
<export> | ||
<lib name="1"/> | ||
</export> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@smuzaffar I get the following complaint from scram in CMSSW_11_2_X_2020-09-18-1100
Are we missing something more?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you get this in latest IB?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, in CMSSW_11_2_X_2020-09-22-1100 there is no such complaint. Thanks!