search

epics-modules / autosave

APS BCDA synApps module: autosave
https://epics-modules.github.io/autosave/
Other
8 stars 31 forks source link

iocsh scripts should pass P macro instead of PREFIX to create_monitor_set calls #20

Closed kmpeters closed 5 years ago

kmpeters commented 6 years ago

The calls to create_monitorset for built*.req: https://github.com/epics-modules/autosave/blob/master/iocsh/autosaveBuild.iocsh#L21

And the calls to create_montiorset for the user-specified req files (usually auto*.req): https://github.com/epics-modules/autosave/blob/master/iocsh/save_restore.iocsh#L27

All set PREFIX=$(PREFIX), but they should very likely be setting P=$(PREFIX), since the P is the standard prefix macro for most EPICS databases.

kmpeters commented 6 years ago

I'm not sure if this is actually necessary, but I think it eliminates an error message.

keenanlang commented 5 years ago

fixed in 227ad60