when the value is used as 'half-length'
replacing GetNegativeEndZ by GetPositiveEndZ fixes the problem
because GetPositiveEndZ is the user value that corresponds to half-length

I hope this will work fine. If the list of headers or linkdefs changes, then cmake should re-run anyway and that will call the file(Generate) to recreate the script, and if only the file content of headers or linkdefs changes, we only have to re-run the shell script.

because add_custom_command does not know the COMPILE_LANGUAGE we are not allowed to treat definitions with that genex inside.
We cannot use add_custom_command(TARGET because that fails with the same error, even though it maybe shouldn't

For compatibility with ninja(?) we need to use these $<CONFIG>$<COMPILE_LANGUAGE> shenanigans

add USE_COMMAND_FOR_GENERATION to fall back to old way

Adape documentation for Tube(...) constructor to reality. (Issue https://github.com/AIDASoft/DD4hep/issues/1236)

Fix DD4hep_DetectorDump plugin to avoid crash. Add test to check if DetElements have proper placement set

LCIO now exports LCIO::lcio, so we could use that if LCIO is new enough

Explicity set namespace to TrackClassification in
TestStackingAction::classifyNewTrack

Co-authored-by: Andre Sailer <andre.philippe.sailer@cern.ch>

Co-authored-by: Andre Sailer <andre.philippe.sailer@cern.ch>

Co-authored-by: Andre Sailer <andre.philippe.sailer@cern.ch>