cms-gem-daq-project / ctp7_modules

0 stars 13 forks source link

amc.h does not include definition of an amc class called in src/calibration_routines.cpp #145

Closed vetens closed 5 years ago

vetens commented 5 years ago

Brief summary of issue

When src/calibration_routines.cpp is called, which in turn calls a function amc::OH_PER_AMC, I receive an error stating that 'amc' has not been declared.

Types of issue

Expected Behavior

Current Behavior

error raised when attempting make rpm

Steps to Reproduce (for bugs)

  1. run make rpm

Possible Solution (for bugs)

There is an amc.h in include that does not have the definition of an amc class in it so I suspect there is a newer version somewhere with that definition that just hasn't been merged yet

Context (for feature requests)

Your Environment

jsturdy commented 5 years ago

Can you add the git hash you're compiling against to the Your Environment section?

vetens commented 5 years ago

updated: 2c5d65f

jsturdy commented 5 years ago

Thanks! Indeed, unfortunately the backport hotfix (#125) didn't take into account that these had moved around... If you want to do a patch of this you will need to simply add:

#include "hw_constants.h"

to the includes in calibration_routines.cpp

vetens commented 5 years ago

Thanks! That worked.