Merge pull request #424 from JakeGinnivan/fixLoggingNotSetupEarlyEnough

JakeGinnivan · JakeGinnivan · commit 9bcb8670089c · 2015-04-23T20:38:46.000+01:00
Logging was not initialised early enough causing a crash
diff --git a/GitVersionExe/Program.cs b/GitVersionExe/Program.cs
@@ -55,6 +55,7 @@ static int VerifyArgumentsAndRun()
                     return 0;
                 }
 
+                ConfigureLogging(arguments);
                 if (arguments.Init)
                 {
                     ConfigurationProvider.WriteSample(arguments.TargetPath, fileSystem);
@@ -71,7 +72,6 @@ static int VerifyArgumentsAndRun()
                     arguments.Output = OutputType.BuildServer;
                 }
 
-                ConfigureLogging(arguments);
                 Logger.WriteInfo("Working directory: " + arguments.TargetPath);
 
                 SpecifiedArgumentRunner.Run(arguments, fileSystem);

Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,7 @@ static int VerifyArgumentsAndRun()`
`55`	`55`	`return 0;`
`56`	`56`	`}`
`57`	`57`
	`58`	`+ ConfigureLogging(arguments);`
`58`	`59`	`if (arguments.Init)`
`59`	`60`	`{`
`60`	`61`	`ConfigurationProvider.WriteSample(arguments.TargetPath, fileSystem);`
`@@ -71,7 +72,6 @@ static int VerifyArgumentsAndRun()`
`71`	`72`	`arguments.Output = OutputType.BuildServer;`
`72`	`73`	`}`
`73`	`74`
`74`		`- ConfigureLogging(arguments);`
`75`	`75`	`Logger.WriteInfo("Working directory: " + arguments.TargetPath);`
`76`	`76`
`77`	`77`	`SpecifiedArgumentRunner.Run(arguments, fileSystem);`