Closed ChristianTackeGSI closed 5 months ago
tests/ci/slurm-submit.sh (2)
[info] 64-64: Double quote to prevent globbing and word splitting. --- [info] 66-66: Double quote to prevent globbing and word splitting.
tests/ci/slurm-submit.sh (3)
`3-5`: The updated argument check and error message are clear and correctly enforce the new requirement for three arguments. --- `9-11`: Variable assignments for `type`, `label`, and `jobsh` are correctly updated to match the new argument structure. --- `30-31`: The switch case logic is correctly updated to check the `type` variable.
Not leaking fGeoLoader on FairRunSim seems to cause some memory issues (probbaly a double free/delete). Until we find it, let's leak the Geoloader, as before.
See: https://github.com/FairRootGroup/FairRoot/issues/1514 See: ede7137bd698df6ba9a6b4e6807e8225e68e121b
Checklist: