Skip to content

Use full path in sub files for default lalapps_MakeSFTDAG options

Evan Goetz requested to merge evan-goetz/lalsuite:add-fullpath into master

Description

Condor needs the full path to executables listed in .sub files but the default behaviour of lalapps_MakeSFTDAG only lists the program name. This fix changes the default executable path to /usr/bin which is the default location for the two programs needed in .sub files: gw_data_find and lalapps_MakeSFTs.

Unit test was also updated with this change.

API Changes and Justification

Backwards Compatible Changes

  • This change introduces no API changes
  • This change adds new API calls

Backwards Incompatible Changes

  • This change modifies an existing API
  • This change removes an existing API

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

@david-keitel or @karl-wette might be good to have a look at this since they've worked on some of this interface before.

Merge request reports