From a2edecd492386841d341190df0519be2c10056d6 Mon Sep 17 00:00:00 2001
From: Michael Davidsaver <mdavidsaver@gmail.com>
Date: Wed, 19 Jul 2023 16:18:39 -0700
Subject: [PATCH] consolidate version_information

---
 src/describe.cpp  |  8 ++++++++
 src/pvxs/util.h   | 11 +++++++++++
 tools/call.cpp    |  4 +---
 tools/get.cpp     |  4 +---
 tools/info.cpp    |  4 +---
 tools/list.cpp    |  4 +---
 tools/monitor.cpp |  4 +---
 tools/mshim.cpp   |  4 +---
 tools/put.cpp     |  4 +---
 tools/pvxvct.cpp  |  5 ++---
 10 files changed, 28 insertions(+), 24 deletions(-)

diff --git a/src/describe.cpp b/src/describe.cpp
index 91b30fdc2..66bd96cea 100644
--- a/src/describe.cpp
+++ b/src/describe.cpp
@@ -132,4 +132,12 @@ std::ostream& target_information(std::ostream& strm)
     return strm;
 }
 
+std::ostream& version_information(std::ostream& strm)
+{
+    strm<<version_str()<<"\n";
+    strm<<EPICS_VERSION_STRING<<"\n";
+    strm<<"libevent "<<event_get_version()<<"\n";
+    return strm;
+}
+
 } // namespace pvxs
diff --git a/src/pvxs/util.h b/src/pvxs/util.h
index 3af76719a..0de5a17b2 100644
--- a/src/pvxs/util.h
+++ b/src/pvxs/util.h
@@ -179,6 +179,17 @@ struct PVXS_API Detailed {
 PVXS_API
 std::ostream& target_information(std::ostream&);
 
+/** Print version information for PVXS library and dependencies
+ *
+ * As shown by eg. ``pvxget -V``
+ *
+ * @returns The same ostream passed as argument.
+ *
+ * @since UNRELEASED
+ */
+PVXS_API
+std::ostream& version_information(std::ostream&);
+
 /** Thread-safe, bounded, multi-producer, multi-consumer FIFO queue.
  *
  * Queue value_type must be movable.  If T is also copy constructable,
diff --git a/tools/call.cpp b/tools/call.cpp
index 6279086c4..fa171ef14 100644
--- a/tools/call.cpp
+++ b/tools/call.cpp
@@ -52,9 +52,7 @@ int main(int argc, char *argv[])
                     usage(argv[0]);
                     return 0;
                 case 'V':
-                    std::cout<<version_str()<<"\n";
-                    std::cout<<EPICS_VERSION_STRING<<"\n";
-                    std::cout<<"libevent "<<event_get_version()<<"\n";
+                    std::cout<<pvxs::version_information;
                     return 0;
                 case 'v':
                     verbose = true;
diff --git a/tools/get.cpp b/tools/get.cpp
index 85d3fe0ec..e79135263 100644
--- a/tools/get.cpp
+++ b/tools/get.cpp
@@ -60,9 +60,7 @@ int main(int argc, char *argv[])
                     usage(argv[0]);
                     return 0;
                 case 'V':
-                    std::cout<<version_str()<<"\n";
-                    std::cout<<EPICS_VERSION_STRING<<"\n";
-                    std::cout<<"libevent "<<event_get_version()<<"\n";
+                    std::cout<<pvxs::version_information;
                     return 0;
                 case 'v':
                     verbose = true;
diff --git a/tools/info.cpp b/tools/info.cpp
index c511ce13c..07330c74f 100644
--- a/tools/info.cpp
+++ b/tools/info.cpp
@@ -51,9 +51,7 @@ int main(int argc, char *argv[])
                     usage(argv[0]);
                     return 0;
                 case 'V':
-                    std::cout<<version_str()<<"\n";
-                    std::cout<<EPICS_VERSION_STRING<<"\n";
-                    std::cout<<"libevent "<<event_get_version()<<"\n";
+                    std::cout<<pvxs::version_information;
                     return 0;
                 case 'v':
                     verbose = true;
diff --git a/tools/list.cpp b/tools/list.cpp
index 24c55b039..fce555bd9 100644
--- a/tools/list.cpp
+++ b/tools/list.cpp
@@ -78,9 +78,7 @@ int main(int argc, char *argv[])
                     usage(argv[0]);
                     return 0;
                 case 'V':
-                    std::cout<<version_str()<<"\n";
-                    std::cout<<EPICS_VERSION_STRING<<"\n";
-                    std::cout<<"libevent "<<event_get_version()<<"\n";
+                    std::cout<<pvxs::version_information;
                     return 0;
                 case 'A':
                     active = true;
diff --git a/tools/monitor.cpp b/tools/monitor.cpp
index a941acfa5..87241900a 100644
--- a/tools/monitor.cpp
+++ b/tools/monitor.cpp
@@ -60,9 +60,7 @@ int main(int argc, char *argv[])
                     usage(argv[0]);
                     return 0;
                 case 'V':
-                    std::cout<<version_str()<<"\n";
-                    std::cout<<EPICS_VERSION_STRING<<"\n";
-                    std::cout<<"libevent "<<event_get_version()<<"\n";
+                    std::cout<<pvxs::version_information;
                     return 0;
                 case 'v':
                     verbose = true;
diff --git a/tools/mshim.cpp b/tools/mshim.cpp
index 7d7ab6876..41a070a26 100644
--- a/tools/mshim.cpp
+++ b/tools/mshim.cpp
@@ -266,9 +266,7 @@ int main(int argc, char *argv[])
                     usage(argv[0]);
                     return 0;
                 case 'V':
-                    std::cout<<version_str()<<"\n";
-                    std::cout<<EPICS_VERSION_STRING<<"\n";
-                    std::cout<<"libevent "<<event_get_version()<<std::endl;
+                    std::cout<<pvxs::version_information;
                     return 0;
                 default:
                     usage(argv[0]);
diff --git a/tools/put.cpp b/tools/put.cpp
index 5f1bff30c..5ee201fda 100644
--- a/tools/put.cpp
+++ b/tools/put.cpp
@@ -52,9 +52,7 @@ int main(int argc, char *argv[])
                     usage(argv[0]);
                     return 0;
                 case 'V':
-                    std::cout<<version_str()<<"\n";
-                    std::cout<<EPICS_VERSION_STRING<<"\n";
-                    std::cout<<"libevent "<<event_get_version()<<"\n";
+                    std::cout<<pvxs::version_information;
                     return 0;
                 case 'v':
                     verbose = true;
diff --git a/tools/pvxvct.cpp b/tools/pvxvct.cpp
index 7ca5a3388..7498ce6f7 100644
--- a/tools/pvxvct.cpp
+++ b/tools/pvxvct.cpp
@@ -21,6 +21,7 @@
 #include <osiSock.h>
 
 #include <pvxs/log.h>
+#include <pvxs/util.h>
 
 #include <udp_collector.h>
 #include <utilpvt.h>
@@ -133,9 +134,7 @@ int main(int argc, char *argv[])
                     usage(argv[0]);
                     return 0;
                 case 'V':
-                    std::cout<<pva::version_str()<<"\n";
-                    std::cout<<EPICS_VERSION_STRING<<"\n";
-                    std::cout<<"libevent "<<event_get_version()<<"\n";
+                    std::cout<<pvxs::version_information;
                     return 0;
                 default:
                     usage(argv[0]);