Revert "DPL Analysis: make BindingNodes constexpr"

[martenole]

Reverts #12661

Hi @ktf and @aalkin would you be OK to revert this? It is requires c++20 features which we don't yet have employed at P2 and therefore our daily builds are failing, e.g. https://alijenkins.cern.ch/job/DailyBuilds/job/DailyO2epn/1252/console
Next week's FLP suite should contain the bump of the O2 defaults and then this could be merged

[ktf]

In principle we can make this work with both after lunch. Would that work?

[aalkin]

#12674 works both for 20 and 17

[alibuild]

Error while checking build/O2/fullCI for b53fd0d at 2024-02-08 12:27:

## sw/BUILD/O2-latest/log
c++: error: unrecognized command-line option '--rtlib=compiler-rt'
c++: error: unrecognized command-line option '--rtlib=compiler-rt'


## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -v clang-diagnostic-error error-log.txt
++ grep ' error:'
/sw/SOURCES/O2/12673-slc8_x86-64/0/Framework/DataInspector/src/DataInspectorService.cxx:208:11: error: namespace 'o2::framework::DataInspector' does not follow the underscore convention [aliceO2-namespace-naming]
/sw/SOURCES/O2/12673-slc8_x86-64/0/Framework/DataInspector/src/DataInspectorService.cxx:241:36: error: namespace 'o2::framework::DataInspector' does not follow the underscore convention [aliceO2-namespace-naming]
/sw/SOURCES/O2/12673-slc8_x86-64/0/Framework/DataInspector/src/DataInspector.cxx:47:26: error: namespace 'o2::framework::DataInspector' does not follow the underscore convention [aliceO2-namespace-naming]
++ [[ 0 == 0 ]]
++ exit 1
--

Full log here.

[martenole]

Thanks! Then I'll close this in favor of #12674