Skip to content

Update reference screens to use new record names#80

Merged
simon-ess merged 2 commits into
epics-modules:masterfrom
nariox:EPICS_TZ
Apr 29, 2026
Merged

Update reference screens to use new record names#80
simon-ess merged 2 commits into
epics-modules:masterfrom
nariox:EPICS_TZ

Conversation

@nariox
Copy link
Copy Markdown
Contributor

@nariox nariox commented Apr 8, 2026

Hello fellow EPICS engineers,

This PR updates the reference screens to follow the new standard implemented in release 4.0.0. The new standard uses the actual environment variable names (as defined in CONFIG_ENV and CONFIG_SITE_ENV) for the record names. Although most of these screens continue to work due to the compatibility aliases (with the exception of EPICS_TIMEZONE, more on that on the second paragraph), this PR updates the names so the screens follow the new standard and will continue to work after the compatibility aliases are removed from the inclusion by default.

The EPICS_TIMEZONE env var was deprecated with the release of EPICS 3.15.7 (back in 2019). Nevertheless, it wasn't until this commit (8715ab4) in 2023 that the EPICS_TIMEZONE record was removed from the generated db files (and replaced by the EPICS_TZ variable). In the meantime (for those using the latest EPICS and iocStats versions), the record continued to exist, but was not functional. My understanding is that other than for vxWorks, this variable doesn't do much, which is probably why this went unnoticed until now.

P.S.: I don't have access to some of these tools, so I could not test all of them, I simply replaced the record names in those. I believe there shouldn't be any issues from that, but caveat emptor.

P.S.: Also, since screens continue to (for most intents and purposes) work fine, I didn't think this deserved to have an issue in addition to the PR. But if it'd be best to have an issue for internal tracking/changelog purposes, I would be glad to file it as well.

@simon-ess
Copy link
Copy Markdown
Contributor

Thanks for creating this, I will take a look!

@nariox
Copy link
Copy Markdown
Contributor Author

nariox commented Apr 10, 2026

It's not super urgent, but I thought it's something I could contribute.

By the way, I wonder if we should also remove the edl screens from iocAdmin/srcDisplay in favor of the ones in op/edl/autoconvert as they haven't been updated in a long time and it is not immediately clear which ones are being maintained.

P.S.: I had accidentally run my script twice, so the records that only got appended (i.e., CA_AUTO_ADDR ->CA_AUTO_ADDR_LIST and CA_MAX_ARRAY -> CA_MAX_ARRAY_BYTES) had duplicated suffixes. I fixed that now.

@lucmaga
Copy link
Copy Markdown

lucmaga commented Apr 13, 2026

LGTM.

@simon-ess
Copy link
Copy Markdown
Contributor

LGTM as well. Merging.

@simon-ess simon-ess merged commit 76d12af into epics-modules:master Apr 29, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants