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

Failure in building a _DBG release #33734

Closed
Dr15Jones opened this issue May 14, 2021 · 17 comments
Closed

Failure in building a _DBG release #33734

Dr15Jones opened this issue May 14, 2021 · 17 comments

Comments

@Dr15Jones
Copy link
Contributor

CMSSW_12_0_DBG_X_2021-05-13-2300 had several compilation failures. This issue is meant to track progress on fixing them.

The packages with compilation failures are

  • CondTools/Ecal
  • L1Trigger/CSCTrackFinder
  • L1Trigger/DTTriggerPhase2
  • SimTransport/PPSProtonTransport
  • Validation/HGCalValidation
@cmsbuild
Copy link
Contributor

A new Issue was created by @Dr15Jones Chris Jones.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@Dr15Jones
Copy link
Contributor Author

#33719 fixes Validation/HGCalValidation

@Dr15Jones
Copy link
Contributor Author

assign core

@cmsbuild
Copy link
Contributor

New categories assigned: core

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

@Dr15Jones
Copy link
Contributor Author

I'll look into the build failure of CondTools/Ecal.

@gartung
Copy link
Member

gartung commented May 14, 2021

Copied from TBB issue
Validation/HGCalValidation #33719
RecoPixelVertexing/PixelVertexFinding #33718

@Dr15Jones
Copy link
Contributor Author

@smuzaffar @mrodozov So I created a CMSSW_12_0_DBG_X_2021-05-13-2300 work area (at FNAL), checked out CondTools/Ecal and it compiled without issue. Why didn't this fail like the IB?

@Dr15Jones
Copy link
Contributor Author

@smuzaffar @mrodozov I can answer my own question, doing scram b -v I see that debugging is actually NOT turned on implicitly when using that build. What are all the environment variable I must set to get the same build environment?

@Dr15Jones
Copy link
Contributor Author

So I discovered that compiling using USER_CXXFLAGS set to -O0 -g3 compiled fine but using -O3 -g3 -DEDM_ML_DEBUG lead to the compilation failure.

@mrodozov
Copy link
Contributor

mrodozov commented May 14, 2021

I had that issue already yesterday, sorry for not mentioning it, I've used the USER_CXXFLAGS="-g -O3 _DEDM_ML_DEBUG" in front of scram commands like:
USER_CXXFLAGS="-g -O3 _DEDM_ML_DEBUG" scram b -j 100
or the same as in the additional build options for the _DBG_ build:
https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_12_0_X/master/cmssw-queue-override.file#L17

@dan131riley
Copy link

Best to use -g -O0 -DEDM_ML_DEBUG, as some of the failures are from conditional debug code, and some are from code that doesn't compile without the optimizer.

@Dr15Jones
Copy link
Contributor Author

See #33737

@Dr15Jones
Copy link
Contributor Author

Interesting, the use of -O3 -g actually allowed the compiler to see that the original code would read outside of the array. Using -O0 or -O2 did not find the problem.

@smuzaffar
Copy link
Contributor

@Dr15Jones , you need to use

USER_CXXFLAGS="-g -O3 -DEDM_ML_DEBUG"  scram build

Once back to work, I will update the configuratin to make these flags defaults for the _DBG IBs

@Dr15Jones
Copy link
Contributor Author

See #33739 #33740 and #33741

@smuzaffar
Copy link
Contributor

+core
build errors for dbg Ibs are fixed now

@cmsbuild
Copy link
Contributor

This issue is fully signed and ready to be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants