[mono/monodevelop] [5 commits] b0c1cf71: [Core] Only send to Raygun when we're not in debug mode.

"Cody Russell ([email protected])" <[email protected]>
Newsgroups gmane.comp.gnome.mono.patches
Message-ID <00000141e0d089b0-3455d817-53bf-4f2d-9438-21b72a8a72da-000000@email.amazonses.com>
   Branch: refs/heads/license-sync
     Home: https://github.com/mono/monodevelop
  Compare: https://github.com/mono/monodevelop/compare/3da143671a5c...9f46237afc73

   Commit: b0c1cf71aa9b0f28cab7493350f121dfb432816f
   Author: Cody Russell <[email protected]> (bratsche)
     Date: 2013-10-22 15:31:48 GMT
      URL: https://github.com/mono/monodevelop/commit/b0c1cf71aa9b0f28cab7493350f121dfb432816f

[Core] Only send to Raygun when we're not in debug mode.

Changed paths:
  M main/src/core/MonoDevelop.Core/MonoDevelop.Core/LoggingService.cs

Modified: main/src/core/MonoDevelop.Core/MonoDevelop.Core/LoggingService.cs
===================================================================
@@ -50,7 +50,7 @@ public static class LoggingService
 
 		public static readonly FilePath CrashLogDirectory = UserProfile.Current.LogDir.Combine ("LogAgent");
 
-		static RaygunClient raygunClient;
+		static RaygunClient raygunClient = null;
 		static List<ILogger> loggers = new List<ILogger> ();
 		static RemoteLogger remoteLogger;
 		static DateTime timestamp;
@@ -100,10 +100,12 @@ static LoggingService ()
 
 			timestamp = DateTime.Now;
 
+#if !DEBUG
 			string raygunKey = BrandingService.GetString ("RaygunApiKey");
 			if (raygunKey != null) {
 				raygunClient = new RaygunClient (raygunKey);
 			}
+#endif
 
 			//remove the default trace listener on .NET, it throws up horrible dialog boxes for asserts
 			System.Diagnostics.Debug.Listeners.Clear ();

   Commit: 58248785ba24feb45306891d129c096611430f8b
   Author: Cody Russell <[email protected]> (bratsche)
     Date: 2013-10-22 15:31:56 GMT
      URL: https://github.com/mono/monodevelop/commit/58248785ba24feb45306891d129c096611430f8b

[Core] Add ENABLE_RAYGUN constant.

Changed paths:
  M main/src/core/MonoDevelop.Core/MonoDevelop.Core.csproj
  M main/src/core/MonoDevelop.Core/MonoDevelop.Core/LoggingService.cs

Modified: main/src/core/MonoDevelop.Core/MonoDevelop.Core.csproj
===================================================================
@@ -12,6 +12,9 @@
     <BuildInfo>..\..\..\build\bin\buildinfo</BuildInfo>
     <VcRevision>..\..\..\vcrevision</VcRevision>
   </PropertyGroup>
+  <PropertyGroup Condition="'$(BUILD_REVISION)' != ''">
+      <DefineConstants>$(DefineConstants);ENABLE_RAYGUN</DefineConstants>
+  </PropertyGroup>
   <Choose>
     <When Condition=" Exists('c:\Program Files\Git\bin\git.exe') ">
       <PropertyGroup>

Modified: main/src/core/MonoDevelop.Core/MonoDevelop.Core/LoggingService.cs
===================================================================
@@ -100,7 +100,7 @@ static LoggingService ()
 
 			timestamp = DateTime.Now;
 
-#if !DEBUG
+#if ENABLE_RAYGUN
 			string raygunKey = BrandingService.GetString ("RaygunApiKey");
 			if (raygunKey != null) {
 				raygunClient = new RaygunClient (raygunKey);

   Commit: 57f37fc3c6ea2d8efa8184fb38282d3cb1a146db
   Author: Cody Russell <[email protected]> (bratsche)
     Date: 2013-10-22 15:32:04 GMT
      URL: https://github.com/mono/monodevelop/commit/57f37fc3c6ea2d8efa8184fb38282d3cb1a146db

Add environment variables section to README.

Changed paths:
  M README

Modified: README
===================================================================
@@ -77,7 +77,14 @@ Dependencies
 	Gtk# >= 2.12.8
 	monodoc >= 1.0
 	mono-addins >= 0.6
-	
+
+Special Environment Variables
+-----------------------------
+
+BUILD_REVISION
+        If this environment variable exists we assume we are compiling inside wrench
+
+
 References
 ----------
 

   Commit: f8867e3f3acfe082eec823898e65c454d27d3c7a
   Author: Cody Russell <[email protected]> (bratsche)
     Date: 2013-10-22 15:32:17 GMT
      URL: https://github.com/mono/monodevelop/commit/f8867e3f3acfe082eec823898e65c454d27d3c7a

[Core] Specify the version to Raygun rather than let it use the assembly version.

Changed paths:
  M main/src/core/MonoDevelop.Core/MonoDevelop.Core/LoggingService.cs

Modified: main/src/core/MonoDevelop.Core/MonoDevelop.Core/LoggingService.cs
===================================================================
@@ -196,7 +196,7 @@ internal static void ReportUnhandledException (Exception ex, bool willShutDown,
 
 				if (raygunClient != null) {
 					ThreadPool.QueueUserWorkItem (delegate {
-						raygunClient.Send (ex, tags);
+						raygunClient.Send (ex, tags, BuildInfo.Version);
 					});
 				}
 

   Commit: 9f46237afc73a13df88109e0bd56d14c24ac92b5
   Author: Cody Russell <[email protected]> (bratsche)
     Date: 2013-10-22 15:32:29 GMT
      URL: https://github.com/mono/monodevelop/commit/9f46237afc73a13df88109e0bd56d14c24ac92b5

[Core] Send full system information to Raygun in a custom data attribute.

Changed paths:
  M main/src/core/MonoDevelop.Core/MonoDevelop.Core/LoggingService.cs

Modified: main/src/core/MonoDevelop.Core/MonoDevelop.Core/LoggingService.cs
===================================================================
@@ -27,6 +27,7 @@
 //
 
 using System;
+using System.Collections;
 using System.Collections.Generic;
 using System.IO;
 using System.Net;
@@ -194,9 +195,12 @@ internal static void ReportUnhandledException (Exception ex, bool willShutDown,
 					data = stream.ToArray ();
 				}
 
+				var customData = new Hashtable ();
+				customData["SystemInformation"] = SystemInformation.GetTextDescription ();
+
 				if (raygunClient != null) {
 					ThreadPool.QueueUserWorkItem (delegate {
-						raygunClient.Send (ex, tags, BuildInfo.Version);
+						raygunClient.Send (ex, tags, customData, BuildInfo.Version);
 					});
 				}
 


_______________________________________________
Mono-patches maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.