From 203017f5540f801c25f8345dd8ca30f0bc4de2f8 Mon Sep 17 00:00:00 2001
From: Justin Clarke Casey
Date: Thu, 22 May 2008 00:32:04 +0000
Subject: * Refactor: Move enhancement of version string with operating system
 information from Scene to OpenSimMain * This also means the operating system
 info will show up in the region console (and hence the logs)

---
 OpenSim/Region/Application/OpenSimMain.cs          | 28 ++++++++++++++++++----
 .../Region/ClientStack/LindenUDP/LLClientView.cs   |  3 +--
 OpenSim/Region/Environment/Scenes/Scene.cs         | 17 +------------
 3 files changed, 25 insertions(+), 23 deletions(-)

diff --git a/OpenSim/Region/Application/OpenSimMain.cs b/OpenSim/Region/Application/OpenSimMain.cs
index 4031634..eb44dca 100644
--- a/OpenSim/Region/Application/OpenSimMain.cs
+++ b/OpenSim/Region/Application/OpenSimMain.cs
@@ -346,12 +346,11 @@ namespace OpenSim
         }
 
         /// <summary>
-        /// Update the version string with extra information if it's available.  
-        /// This currently means adding a subversion number if the root .svn/entries file is present.
+        /// Enhance the version string with extra information if it's available.  
         /// </summary>
-        protected void updateAvailableVersionInformation()
+        protected void enhanceVersionInformation()
         {
-            // Set BuildVersion String for Show version command
+            // Add subversion revision information if available
             string svnFileName = "../.svn/entries";
             string inputLine;
             int strcmp;
@@ -380,7 +379,25 @@ namespace OpenSim
             if (!string.IsNullOrEmpty(buildVersion))
             {
                 VersionInfo.Version += ", SVN build r" + buildVersion;
+            }            
+
+            // Add operating system information if available
+            string OSString = "";
+
+            if (System.Environment.OSVersion.Platform != PlatformID.Unix)
+            {
+                OSString = System.Environment.OSVersion.ToString();
             }
+            else
+            {
+                OSString = Util.ReadEtcIssue();
+            }
+            if (OSString.Length > 45)
+            {
+                OSString = OSString.Substring(0, 45);
+            }   
+            
+            VersionInfo.Version += " on " + OSString;          
         }
 
         /// <summary>
@@ -388,7 +405,8 @@ namespace OpenSim
         /// </summary>
         protected void InternalStartUp()
         {
-            updateAvailableVersionInformation();
+            enhanceVersionInformation();
+                    
             m_log.Info("[STARTUP]: OpenSim version: " + VersionInfo.Version + "\n");
 
             m_stats = StatsManager.StartCollectingSimExtraStats();
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
index e5fc384..435ed04 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
@@ -350,11 +350,10 @@ namespace OpenSim.Region.ClientStack.LindenUDP
         {
             m_moneyBalance = 1000;
 
-            m_channelVersion = Helpers.StringToField(scene.GetSimulatorVersion());
+            m_channelVersion = Helpers.StringToField("OpenSimulator Server " + scene.GetSimulatorVersion());
 
             InitDefaultAnimations();
 
-
             m_scene = scene;
             m_assetCache = assetCache;
 
diff --git a/OpenSim/Region/Environment/Scenes/Scene.cs b/OpenSim/Region/Environment/Scenes/Scene.cs
index a980ce1..231c0dc 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.cs
@@ -293,22 +293,7 @@ namespace OpenSim.Region.Environment.Scenes
 
             m_statsReporter.SetObjectCapacity(objectCapacity);
 
-            string OSString = "";
-
-            if (System.Environment.OSVersion.Platform != PlatformID.Unix)
-            {
-                OSString = System.Environment.OSVersion.ToString();
-            }
-            else
-            {
-                OSString = Util.ReadEtcIssue();
-            }
-            if (OSString.Length > 45)
-            {
-                OSString = OSString.Substring(0, 45);
-            }
-
-            m_simulatorVersion = simulatorVersion + " on " + OSString 
+            m_simulatorVersion = simulatorVersion
                 + " ChilTasks:" + m_seeIntoRegionFromNeighbor.ToString() 
                 + " PhysPrim:" + m_physicalPrim.ToString();
         }
-- 
cgit v1.1