-
Notifications
You must be signed in to change notification settings - Fork 9
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
Compiling under CMSSW_5_3_X broken since commit 7ece9bf8 #4
Comments
Hi all! There are also compilation errors for the CMSSW_7 receipt (see below). Without looking who made the last commit can this be fixed by the guilty one, please!? ;-) Thanks and cheers, Arno [lxplus0056] ~/scratch1/CMSSW_7_0_7_patch1/src $ scramv1 b -j8
On 01.12.2014, at 05:21, Frank Meier <notifications@github.commailto:notifications@github.com> wrote: I tried to test the instructions on https://twiki.cern.ch/twiki/bin/view/CMS/Bmm4HowtoAnalyze and did so in CMSSW_5_3_22_patch1, what I found to be the latest available release on t3.unl.eduhttp://t3.unl.edu. Unfortunately the last command scramv1 b -j8 failed with output as follows: ----- snip ----
With git blame HFDumpUtilities.cchttp://HFDumpUtilities.cc I identified commit 7ece9bfhttps://github.com/Bmm4/Bmm/commit/7ece9bf83051ef08424fa66e121d2a440e68018c as culprit. Reverting it using git revert 7ece9bf Did the trick. I did not commit such a change, I only did it locally to test. � |
Hi all, without checking I think that CMSSW_7_3_0_preX should compile. It seems to Cheers, On Mon, Dec 1, 2014 at 9:38 AM, heistera notifications@github.com wrote:
|
Arno, Frank, Best regards, Danek Bohdan Kotlinski On 1 Dec 2014, at 09:38, Arno Heister Arno.Heister@cern.ch wrote:
|
Hi Urs! Yes, CMSSW_7_3_0_pre3 works. I have changed the twiki instructions accordingly. THanks! Cheers, Arno On 01.12.2014, at 09:40, Urs Langenegger notifications@github.com wrote:
|
Hi all, I changed the GT in test.py and I verified that I can run on /RelValBsToMuMu_13/CMSSW_7_3_0_pre2-MCRUN2_73_V1-v1/GEN-SIM-RECO and ancient files /Summer12_DR53X/BuToJPsiK_K2MuPtEtaEtaFilter_8TeV-pythia6-evtgen/AODSIM/PU_S10_START53_V7A-v2 At the moment I think we can live off one HEAD in 7_3_X Cheers, On Mon, Dec 1, 2014 at 9:50 AM, heistera notifications@github.com wrote:
|
In today's meeting we decided to deprecate the use of old release. Therefore I close this issue. I've added a warning to https://twiki.cern.ch/twiki/bin/save/CMS/Bmm4HowtoAnalyze (but did not remove the 53X example, as there might still be some use). |
Hello, Now in CandAna::triggerSelection() only those triggers are considered Best regards, |
Sorry, wrong list! On 11 Sep 2015, at 12:07, heistera <notifications@github.commailto:notifications@github.com> wrote: Hello, Now in CandAna::triggerSelection() only those triggers are considered Best regards, |
I tried to test the instructions on https://twiki.cern.ch/twiki/bin/view/CMS/Bmm4HowtoAnalyze and did so in
CMSSW_5_3_22_patch1
, what I found to be the latest available release on t3.unl.edu. Unfortunately the last commandscramv1 b -j8
failed with output as follows:With
git blame HFDumpUtilities.cc
I identified commit 7ece9bf as culprit. Reverting it usinggit revert 7ece9bf8
Did the trick. I did not commit such a change, I only did it locally to test.
But: What to do? Do we need to support 5_3_X? If yes, then likely we'll need a separate branch. If not, then forget about.
The text was updated successfully, but these errors were encountered: