Chris Pinkenburg

Please do not use github to merge your PR with the master, this just triggers a jenkins run and klogs up our system. Merge/rebase the master in your local fork...

> I could be misremembering, but I thought we discussed this last week: the previous Jenkins report indicated nothing wrong with the module here, I've confirmed that it compiles and...

I added the line laser package to the build and made small changes to the Makefile.am and configure.ac to trigger a rerun of jenkins

This PR introduces reading files from /sphenix/user : cc = new caen_correction("/sphenix/user/llegnosky/calib/calib_24974_5G.dat"); cc = new caen_correction("/sphenix/user/llegnosky/calib/calib_24974_2G.dat"); cc = new caen_correction("/sphenix/user/llegnosky/calib/calib_24974_1G.dat"); I overlooked this when letting this PR through. This needs...

The recursive is just a warning (it is a possible infinite loop if you call it the wrong way and clang-tidy assumes the worst). If you add this // NOLINTNEXTLINE(misc-no-recursion)...