search

ralphlange / procServ

Wrapper to start arbitrary interactive commands in the background, with telnet or Unix domain socket access to stdin/stdout
GNU General Public License v3.0
23 stars 23 forks source link

Makefile.Epics should define PROD_CMD too #56

Closed dirk-zimoch closed 1 year ago

dirk-zimoch commented 1 year ago

https://github.com/ralphlange/procServ/blob/cd68a34da12ec156c4126932b62947ee195b6210/Makefile.Epics.in#L12 In EPICS R7.0.5, a new VALID_BUILDS choice Command has been introduced for architectures that need command line tools but not all the build tools like antelope. Thus, some Architectures may now use Command but no longer Host in their VALID_BUILDS. On those architectures, procServ is erroneously not built because procServ uses PROD_HOST only.

Specifying both, PROD_HOST = procServ and PROD_CMD = procServ ensures that proServ is built for all architectures, that use the old Host or the new Command in their VALID_BUILDS.

ralphlange commented 1 year ago

Sounds reasonable.

dirk-zimoch commented 1 year ago

Fixed the quite consistent typo: I wrote PROC instead of PROD.

ralphlange commented 1 year ago

Don't worry. The commit was valid. 😉