search

ControlSystemStudio / cs-studio

Control System Studio is an Eclipse-based collections of tools to monitor and operate large scale control systems, such as the ones in the accelerator community.
https://controlsystemstudio.org/
Eclipse Public License 1.0
112 stars 96 forks source link

remove utility.pv #899

Open kasemir opened 9 years ago

kasemir commented 9 years ago

alarm server, archive engine, scan server, PV table, EPICS PV Tree use vtype.pv. opibuilder, probe use (via simplepv) either utility.pv or PVManager.

Remove utility.pv; have simplepv select between vtype.pv or PVManager.

kasemir commented 9 years ago

On the bug899_util_pv branch, utility.pv dependencies have been removed from the opibuilder (via simplepv).

The older o.c.diag.probe plugin has also been updated to use vtype.pv instead of util.pv, but otherwise left as is since products tend to include o.c.diag.probe.pvmanager anyway.

The only code that still depends on the utility.pv.* plugins is now org.csstudio.archive.reader.archiverecord. It doesn't seem worth updating, however, since it doesn't return any data. Its ArchiveRecordReader class is where Jan left it 2 years ago with a TODO to return VType values.

@berryma4 : Exclude archiverecord from pom.xml so it's not built, then remove all utility.pv.* plugins?

berryma4 commented 9 years ago

That sounds fine with me. Thank you!

On Mon, Jan 26, 2015 at 4:53 PM, kasemir notifications@github.com wrote:

On the bug899_util_pv branch, utility.pv dependencies have been removed from the opibuilder (via simplepv).

The older o.c.diag.probe plugin has also been updated to use vtype.pv instead of util.pv, but otherwise left as is since products tend to include o.c.diag.probe.pvmanager anyway.

The only code that still depends on the utility.pv.* plugins is now org.csstudio.archive.reader.archiverecord. It doesn't seem worth updating, however, since it doesn't return any data. Its ArchiveRecordReader class is where Jan left it 2 years ago with a TODO to return VType values.

@berryma4 https://github.com/berryma4 : Exclude archiverecord from pom.xml so it's not built, then remove all utility.pv.* plugins?

— Reply to this email directly or view it on GitHub https://github.com/ControlSystemStudio/cs-studio/issues/899#issuecomment-71546227 .