epics-modules / motor

APS BCDA synApps module: motor
https://epics-modules.github.io/motor/
20 stars 47 forks source link

Proposal: Comment out SUPPORT in RELEASE by default #159

Closed kmpeters closed 4 years ago

kmpeters commented 4 years ago

The definition of SUPPORT is not needed in the RELEASE file when motor is configured in synApps using make release or when a RELEASE.local file is used.

The inclusion of a dummy definition of SUPPORT can cause checkRelease problems (see #158).

kmpeters commented 4 years ago

This may not be possible unless synApps build system changes.

synApps/configure/RELEASE defines module paths using SUPPORT:

https://github.com/EPICS-synApps/configure/blob/72ff4547999c22184f8c86e7ca5b50ac38848cad/RELEASE#L41

synApps/configure/makeReleaseConsistent.pl, the script that is called when make release is issued, copies these definitions to the synApps modules, which are required to define SUPPORT the same way that synApps/configure/RELEASE does.