Skip to content

Fix typo in ElectronPhotonFourMomentumCorrection CMakeLists.txt

Oliver Majersky requested to merge omajersk/athena:fix-ElectronPhotonID-CMake into master

Title says it all, there is a typo INCUDE instead of INCLUDE in the CMakeLists of the ElectronPhotonFourMomentumCorrection package. When we try to build AnalysisTop packages, we get CMake configuration errors, because our pacakges (among other things) depend on this package, which has a broken configuration.

What I find worrying is that, this typo was not picked up by CI jobs. Not an expert on CI, to my limited understanding it does some caching of built packages to shorted the build time. However this suggests that changes of CMakeLists.txt do not properly retrigger the build process.

Thanks to @bmondal for spotting the typo.

Edited by Oliver Majersky

Merge request reports