Problem 1208 - Trajectory drawing does not work with cmake-build Geant 4.9.4_p01
Summary: Trajectory drawing does not work with cmake-build Geant 4.9.4_p01
Status: CLOSED FIXED
Alias: None
Product: Geant4
Classification: Unclassified
Component: cmake (show other problems)
Version: 9.4
Hardware: All All
: P2 major
Assignee: Ben Morgan
URL:
Depends on:
Blocks:
 
Reported: 2011-04-28 11:12 CEST by Matthias Nagl
Modified: 2011-05-17 18:05 CEST (History)
0 users

See Also:

Attachments
Fix for the missing G4_NO_STORE_TRAJECTORY definition (390 bytes, patch)
2011-04-28 11:12 CEST, Matthias Nagl 		Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this problem.
Description Matthias Nagl 2011-04-28 11:12:18 CEST 
Created attachment 107 [details]
Fix for the missing G4_NO_STORE_TRAJECTORY definition

Cmake-build installs of Geant 4.9.4_p01 do not support the drawing of trajectories with any visualisation driver. The problem results from a missing define (G4_STORE_TRAJECTORY). The old build system defines this in config/architecture.gmk:

ifndef G4_NO_STORE_TRAJECTORY
  CPPFLAGS += -DG4_STORE_TRAJECTORY
endif

A simple fix (without the G4_NO_STORE_TRAJECTORY functionality) is added as attachment.
Comment 1 Ben Morgan 2011-04-28 23:11:58 CEST 
Thanks for the fix - it'll be in the summer beta, but we'll probably wrap it in an option(...) command so it behaves as in the current system.

Ben.
Comment 2 Ben Morgan 2011-05-17 18:05:06 CEST 
Fixed in SVN : Revision #49877, log:

BUGFIX : Bug #1208 : Added G4_STORE_TRAJECTORY to global compile definitions

Also added a CMake advanced option so that users can choose not to store
trajectories, but by default this option is set to ON. This replicates the
behaviour in the current Geant4 makefiles. The option is marked as advanced
because most users will not need to adjust this performance tweak.