Merge pull request #482 from FireHost/feature/Fix_ShowVariable_Output

Update ArgumentParser to use variables from VersionVariables

Author: JakeGinnivan

GitVersionCore/OutputVariables/VersionVariables.cs

@@ -50,7 +50,7 @@ public VersionVariables(string major, string minor, string patch, string buildMe
         public string NuGetVersionV2 { get { return LegacySemVerPadded.ToLower(); } }
         public string NuGetVersion { get { return NuGetVersionV2; } }
 
-        public IEnumerable<string> AvailableVariables
+        public static IEnumerable<string> AvailableVariables
         {
             get { return typeof(VersionVariables).GetProperties().Select(p => p.Name).OrderBy(a => a); }
         }

GitVersionExe/ArgumentParser.cs

@@ -4,18 +4,11 @@ namespace GitVersion
     using System.Collections.Generic;
     using System.Collections.Specialized;
     using System.Linq;
-    using System.Reflection;
     using System.Text.RegularExpressions;
 
 
     public class ArgumentParser
     {
-        static ArgumentParser()
-        {
-            var fields = typeof(VariableProvider).GetFields(BindingFlags.Public | BindingFlags.Static);
-            VersionParts = fields.Select(x => x.Name.ToLower()).ToArray();
-        }
-
         public static Arguments ParseArguments(string commandLineArguments)
         {
             return ParseArguments(commandLineArguments.Split(new[] { ' ' }, StringSplitOptions.RemoveEmptyEntries).ToList());
@@ -185,9 +178,23 @@ public static Arguments ParseArguments(List<string> commandLineArguments)
                     throw new WarningException("assemblyversionformat switch removed, use AssemblyVersioningScheme configuration value instead");
                 }
 
-                if ((IsSwitch("v", name)) && VersionParts.Contains(value.ToLower()))
+                if (IsSwitch("v", name) || IsSwitch("showvariable", name))
                 {
-                    arguments.ShowVariable = value.ToLower();
+                    string versionVariable = null;
+
+                    if (!string.IsNullOrWhiteSpace(value))
+                    {
+                        versionVariable = VersionVariables.AvailableVariables.SingleOrDefault(av => av.Equals(value.Replace("'", ""), StringComparison.CurrentCultureIgnoreCase));
+                    }
+                    
+                    if (versionVariable == null)
+                    {
+                        var messageFormat = "{0} requires a valid version variable.  Available variables are:\n{1}";
+                        var message = string.Format(messageFormat, name, String.Join(", ", VersionVariables.AvailableVariables.Select(x=>string.Concat("'", x, "'"))));
+                        throw new WarningException(message);
+                    }
+
+                    arguments.ShowVariable = versionVariable;
                     continue;
                 }
 
@@ -297,7 +304,5 @@ static bool IsHelp(string singleArgument)
                 IsSwitch("help", singleArgument) ||
                 IsSwitch("?", singleArgument);
         }
-
-        static string[] VersionParts;
     }
 }

GitVersionExe/Program.cs

@@ -42,6 +42,19 @@ static int VerifyArgumentsAndRun()
                 {
                     arguments = ArgumentParser.ParseArguments(argumentsWithoutExeName);
                 }
+                catch (WarningException ex)
+                {
+                    Console.WriteLine("Failed to parse arguments: {0}", string.Join(" ", argumentsWithoutExeName));
+                    if (!string.IsNullOrWhiteSpace(ex.Message))
+                    {
+                        Console.WriteLine();
+                        Console.WriteLine(ex.Message);
+                        Console.WriteLine();
+                    }
+
+                    HelpWriter.Write();
+                    return 1;
+                }
                 catch (Exception)
                 {
                     Console.WriteLine("Failed to parse arguments: {0}", string.Join(" ", argumentsWithoutExeName));