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

adjust to new interface, remove obsolete externals #1197

Merged
merged 1 commit into from
Oct 28, 2013

Conversation

argiro
Copy link
Contributor

@argiro argiro commented Oct 28, 2013

Get rid of external "elementtree" and update scripts for new DB interface

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @argiro for CMSSW_7_0_X.

adjust to new interface, remove obsolete externals

It involves the following packages:

CondTools/Ecal

@ggovi, @smuzaffar, @apfeiffer1, @nclopezo can you please review it and eventually sign? Thanks.
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.
@ktf you are the release manager for this.

@@ -1,7 +1,7 @@
#
# Misc functions to manipulate Ecal records
# author: Stefano Argiro
# id: $Id: EcalCondTools.py,v 1.12 2009/11/03 17:40:07 yma Exp $
# id: $Id: EcalCondTools.py,v 1.14 2013/05/06 08:29:44 argiro Exp $
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, drop CVS keywords next time.

@ktf
Copy link
Contributor

ktf commented Oct 28, 2013

No way to really check this. Trusting Stefano.

ktf added a commit that referenced this pull request Oct 28, 2013
DB updates -- adjust to new interface, remove obsolete externals (element tree).
@ktf ktf merged commit edff3dd into cms-sw:CMSSW_7_0_X Oct 28, 2013
EmyrClement pushed a commit to EmyrClement/cmssw that referenced this pull request Feb 14, 2024
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.

3 participants