Always generate ThisAssembly members for assembly metadata

Author: AArnott

src/NerdBank.GitVersioning.Tests/BuildIntegrationTests.cs

@@ -648,21 +648,10 @@ public async Task AssemblyInfo(bool isVB, bool includeNonVersionAttributes)
         Assert.Equal(result.AssemblyName, thisAssemblyClass.GetField("AssemblyName", fieldFlags).GetValue(null));
         Assert.Equal(result.RootNamespace, thisAssemblyClass.GetField("RootNamespace", fieldFlags).GetValue(null));
         Assert.Equal(result.AssemblyConfiguration, thisAssemblyClass.GetField("AssemblyConfiguration", fieldFlags).GetValue(null));
-        if (includeNonVersionAttributes)
-        {
-
-            Assert.Equal(result.AssemblyTitle, thisAssemblyClass.GetField("AssemblyTitle", fieldFlags).GetValue(null));
-            Assert.Equal(result.AssemblyProduct, thisAssemblyClass.GetField("AssemblyProduct", fieldFlags).GetValue(null));
-            Assert.Equal(result.AssemblyCompany, thisAssemblyClass.GetField("AssemblyCompany", fieldFlags).GetValue(null));
-            Assert.Equal(result.AssemblyCopyright, thisAssemblyClass.GetField("AssemblyCopyright", fieldFlags).GetValue(null));
-        }
-        else
-        {
-            Assert.Null(thisAssemblyClass.GetField("AssemblyTitle", fieldFlags));
-            Assert.Null(thisAssemblyClass.GetField("AssemblyProduct", fieldFlags));
-            Assert.Null(thisAssemblyClass.GetField("AssemblyCompany", fieldFlags));
-            Assert.Null(thisAssemblyClass.GetField("AssemblyCopyright", fieldFlags));
-        }
+        Assert.Equal(result.AssemblyTitle, thisAssemblyClass.GetField("AssemblyTitle", fieldFlags)?.GetValue(null));
+        Assert.Equal(result.AssemblyProduct, thisAssemblyClass.GetField("AssemblyProduct", fieldFlags)?.GetValue(null));
+        Assert.Equal(result.AssemblyCompany, thisAssemblyClass.GetField("AssemblyCompany", fieldFlags)?.GetValue(null));
+        Assert.Equal(result.AssemblyCopyright, thisAssemblyClass.GetField("AssemblyCopyright", fieldFlags)?.GetValue(null));
 
         // Verify that it doesn't have key fields
         Assert.Null(thisAssemblyClass.GetField("PublicKey", fieldFlags));

src/Nerdbank.GitVersioning.NuGet/build/NerdBank.GitVersioning.targets

@@ -43,12 +43,6 @@
     <_NBGV_BuildingRef>$(_NBGV_BuildingTag)</_NBGV_BuildingRef>
     <_NBGV_BuildingRef Condition=" '$(_NBGV_BuildingRef)' == '' ">$(_NBGV_BuildingBranch)</_NBGV_BuildingRef>
   </PropertyGroup>
-  <PropertyGroup Condition=" '$(NBGV_EmitNonVersionCustomAttributes)' == 'true' ">
-    <_NBGV_AssemblyTitle>$(AssemblyTitle)</_NBGV_AssemblyTitle>
-    <_NBGV_AssemblyProduct>$(AssemblyProduct)</_NBGV_AssemblyProduct>
-    <_NBGV_AssemblyCopyright>$(AssemblyCopyright)</_NBGV_AssemblyCopyright>
-    <_NBGV_AssemblyCompany>$(AssemblyCompany)</_NBGV_AssemblyCompany>
-  </PropertyGroup>
 
   <Target Name="GetBuildVersion" Returns="$(BuildVersion)">
     <Nerdbank.GitVersioning.Tasks.GetBuildVersion
@@ -132,11 +126,13 @@
                   AssemblyName="$(AssemblyName)"
                   RootNamespace="$(RootNamespace)"
                   AssemblyOriginatorKeyFile="$(AssemblyOriginatorKeyFile)"
-                  AssemblyTitle="$(_NBGV_AssemblyTitle)"
-                  AssemblyProduct="$(_NBGV_AssemblyProduct)"
-                  AssemblyCopyright="$(_NBGV_AssemblyCopyright)"
-                  AssemblyCompany="$(_NBGV_AssemblyCompany)"
-                  AssemblyConfiguration="$(Configuration)"/>
+                  AssemblyTitle="$(AssemblyTitle)"
+                  AssemblyProduct="$(AssemblyProduct)"
+                  AssemblyCopyright="$(AssemblyCopyright)"
+                  AssemblyCompany="$(AssemblyCompany)"
+                  AssemblyConfiguration="$(Configuration)"
+                  EmitNonVersionCustomAttributes="$(NBGV_EmitNonVersionCustomAttributes)"
+                  />
     <!-- Avoid applying the newly generated AssemblyVersionInfo.cs file to the build 
          unless it has changed in order to allow for incremental building. -->
     <Nerdbank.GitVersioning.Tasks.CompareFiles OriginalItems="$(VersionSourceFile)" NewItems="$(NewVersionSourceFile)">

src/Nerdbank.GitVersioning.Tasks/AssemblyVersionInfo.cs

@@ -29,6 +29,8 @@ public class AssemblyVersionInfo : Task
         [Required]
         public string OutputFile { get; set; }
 
+        public bool EmitNonVersionCustomAttributes { get; set; }
+
         public string AssemblyName { get; set; }
 
         public string AssemblyVersion { get; set; }
@@ -169,14 +171,17 @@ private IEnumerable<CodeAttributeDeclaration> CreateAssemblyAttributes()
             yield return DeclareAttribute(typeof(AssemblyVersionAttribute), this.AssemblyVersion);
             yield return DeclareAttribute(typeof(AssemblyFileVersionAttribute), this.AssemblyFileVersion);
             yield return DeclareAttribute(typeof(AssemblyInformationalVersionAttribute), this.AssemblyInformationalVersion);
-            if (!string.IsNullOrEmpty(this.AssemblyTitle))
-                yield return DeclareAttribute(typeof(AssemblyTitleAttribute), this.AssemblyTitle);
-            if (!string.IsNullOrEmpty(this.AssemblyProduct))
-                yield return DeclareAttribute(typeof(AssemblyProductAttribute), this.AssemblyProduct);
-            if (!string.IsNullOrEmpty(this.AssemblyCompany))
-                yield return DeclareAttribute(typeof(AssemblyCompanyAttribute), this.AssemblyCompany);
-            if (!string.IsNullOrEmpty(this.AssemblyCopyright))
-                yield return DeclareAttribute(typeof(AssemblyCopyrightAttribute), this.AssemblyCopyright);
+            if (this.EmitNonVersionCustomAttributes)
+            {
+                if (!string.IsNullOrEmpty(this.AssemblyTitle))
+                    yield return DeclareAttribute(typeof(AssemblyTitleAttribute), this.AssemblyTitle);
+                if (!string.IsNullOrEmpty(this.AssemblyProduct))
+                    yield return DeclareAttribute(typeof(AssemblyProductAttribute), this.AssemblyProduct);
+                if (!string.IsNullOrEmpty(this.AssemblyCompany))
+                    yield return DeclareAttribute(typeof(AssemblyCompanyAttribute), this.AssemblyCompany);
+                if (!string.IsNullOrEmpty(this.AssemblyCopyright))
+                    yield return DeclareAttribute(typeof(AssemblyCopyrightAttribute), this.AssemblyCopyright);
+            }
         }
 
         private static IEnumerable<CodeMemberField> CreateFields(IReadOnlyDictionary<string, string> namesAndValues)