Code review comment for ~anj/epics-base/+git/base-7.0:typed-dsets

Revision history for this message
Andrew Johnson (anj) wrote :

Added documentation to the Release Notes including how to modify an external device support that must remain compatible with older versions of Base. Rather than using one of the new and probably unfamiliar record types for that I converted the ai record and all the internal device supports that use it.

There is an unfortunate issue with the lso, lsi and printf records which were already publishing their dsets, so they need a slightly different approach to remain compatible with older Base versions, which I also showed. The alternative would have been to rename the dset for these record types, but I didn't want to break the naming pattern if I could avoid it.

I have tested these approaches with Kay's ether_ip driver, and will send him a PR for that code once this branch gets merged.

« Back to merge proposal