Adding verbose as obsolete with warning message (#347)

SetCurrentVersion.cmd

@@ -1,9 +1,9 @@
 set MAJOR_PREVIOUS=1
 set MINOR_PREVIOUS=7
-set PATCH_PREVIOUS=2
+set PATCH_PREVIOUS=3
 set PRERELEASE_PREVIOUS=
 
 set MAJOR=1
 set MINOR=7
-set PATCH=3
+set PATCH=4
 set PRERELEASE=

src/BinSkim.Driver/AnalyzeCommand.cs

@@ -33,6 +33,13 @@ protected override BinaryAnalyzerContext CreateContext(AnalyzeOptions options, I
             binaryAnalyzerContext.TracePdbLoads = options.Traces.Contains(nameof(Traces.PdbLoad));
             binaryAnalyzerContext.LocalSymbolDirectories = options.LocalSymbolDirectories;
 
+#pragma warning disable CS0618 // Type or member is obsolete
+            if (options.Verbose)
+#pragma warning restore CS0618 // Type or member is obsolete
+            {
+                Warnings.LogObsoleteOption(binaryAnalyzerContext, "--verbose", Sdk.SdkResources.Verbose_ReplaceWithLevelAndKind);
+            }
+
             return binaryAnalyzerContext;
         }
 
@@ -48,12 +55,20 @@ public override int Run(AnalyzeOptions analyzeOptions)
                 analyzeOptions.SarifOutputVersion = s_UnitTestOutputVersion;
             }
 
+#pragma warning disable CS0618 // Type or member is obsolete
+            if (analyzeOptions.Verbose)
+#pragma warning restore CS0618 // Type or member is obsolete
+            {
+                analyzeOptions.Level = new List<FailureLevel> { FailureLevel.Error, FailureLevel.Warning, FailureLevel.Note };
+                analyzeOptions.Kind = new List<ResultKind> { ResultKind.Fail, ResultKind.NotApplicable, ResultKind.Pass };
+            }
+
             int result = base.Run(analyzeOptions);
 
             // In BinSkim, no rule is ever applicable to every target type. For example,
             // we have checks that are only relevant to either 32-bit or 64-bit binaries.
             // Because of this, the return code bit for RuleNotApplicableToTarget is not
-            // interesting (it will always be set). 
+            // interesting (it will always be set).
 
             return analyzeOptions.RichReturnCode
                 ? (int)((uint)result & ~(uint)RuntimeConditions.RuleNotApplicableToTarget)

src/BinSkim.Driver/AnalyzeOptions.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft. All rights reserved.
 // Licensed under the MIT license. See LICENSE file in the project root for full license information.
 
+using System;
 using System.Collections.Generic;
 
 using CommandLine;
@@ -36,5 +37,11 @@ public class AnalyzeOptions : AnalyzeOptionsBase
             "local-symbol-directories",
             HelpText = "A set of semicolon-delimited local directory paths that will be examined when attempting to locate PDBs.")]
         public string LocalSymbolDirectories { get; internal set; }
+
+        [Option(
+            "verbose",
+            HelpText = "Emit verbose output. The resulting comprehensive report is designed to provide appropriate evidence for compliance scenarios.")]
+        [Obsolete("Use --level and --kind instead.")]
+        public bool Verbose { get; set; }
     }
 }

src/BinSkim.Sdk/SdkResources.resx

@@ -219,4 +219,7 @@
   <data name="MetadataCondition_ImageIsDotNetNativeBootstrapExe" xml:space="preserve">
     <value>image is a .NET native bootstrap exe</value>
   </data>
+  <data name="Verbose_ReplaceWithLevelAndKind" xml:space="preserve">
+    <value>use --level and --kind</value>
+  </data>
 </root>