Update to latest Wixtoolset.Data and simplify Project/Package reference condition

author: Rob Mensching <rob@firegiant.com> 2018-01-01 16:35:30 -0800
committer: Rob Mensching <rob@firegiant.com> 2018-01-01 16:35:30 -0800
commit: 58f9e097a445dbdf3139832ac471374e68678fc6 (patch)
tree: 641e3b0cff1287459ac29eb26a7f810f1d4afb7f /src
parent: 651ad904724f32f5e993fa73aa11a611d95a1a10 (diff)
download: wix-58f9e097a445dbdf3139832ac471374e68678fc6.tar.gz
wix-58f9e097a445dbdf3139832ac471374e68678fc6.tar.bz2
wix-58f9e097a445dbdf3139832ac471374e68678fc6.zip
3 files changed, 22 insertions, 47 deletions
diff --git a/src/WixToolset.Core.TestPackage/WixToolset.Core.TestPackage.csproj b/src/WixToolset.Core.TestPackage/WixToolset.Core.TestPackage.csproj
index 4c0fe92d..d9683030 100644
--- a/src/WixToolset.Core.TestPackage/WixToolset.Core.TestPackage.csproj
+++ b/src/WixToolset.Core.TestPackage/WixToolset.Core.TestPackage.csproj
@@ -23,9 +23,14 @@
  </ItemGroup>
  <ItemGroup>
-    <PackageReference Include="WixToolset.Data" Version="4.0.*" />
+    <ProjectReference Include="$(WixToolsetRootFolder)\Data\src\WixToolset.Data\WixToolset.Data.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Data\README.md') " />
-    <PackageReference Include="WixToolset.Extensibility" Version="4.0.*" />
+    <PackageReference Include="WixToolset.Data" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Data\README.md') " />
-    <PackageReference Include="WixToolset.Core.Native" Version="4.0.*" />
+    <ProjectReference Include="$(WixToolsetRootFolder)\Extensibility\src\WixToolset.Extensibility\WixToolset.Extensibility.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Extensibility\README.md') " />
+    <PackageReference Include="WixToolset.Extensibility" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Extensibility\README.md') " />
+    <ProjectReference Include="$(WixToolsetRootFolder)\Core.Native\src\WixToolset.Core.Native\WixToolset.Core.Native.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Core.Native\README.md') " />
+    <PackageReference Include="WixToolset.Core.Native" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Core.Native\README.md') " />
  </ItemGroup>
  <ItemGroup>
diff --git a/src/WixToolset.Core/Compiler.cs b/src/WixToolset.Core/Compiler.cs
index c71a8ba4..cd51fed6 100644
--- a/src/WixToolset.Core/Compiler.cs
+++ b/src/WixToolset.Core/Compiler.cs
@@ -690,8 +690,7 @@ namespace WixToolset.Core
                if (YesNoType.Yes == suppressModularization)
                {
-                    var wixSuppressModularizationRow = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.WixSuppressModularization);
+                    var wixSuppressModularizationRow = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.WixSuppressModularization, id);
-                    wixSuppressModularizationRow.Set(0, id);
                }
            }
@@ -2474,8 +2473,7 @@ namespace WixToolset.Core
                if (multiInstance)
                {
-                    var instanceComponentRow = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.WixInstanceComponent);
+                    var instanceComponentRow = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.WixInstanceComponent, id);
-                    instanceComponentRow.Set(0, id);
                }
                if (0 < symbols.Count)
@@ -2488,8 +2486,7 @@ namespace WixToolset.Core
                // Complus
                if (CompilerConstants.IntegerNotSet != comPlusBits)
                {
-                    row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.Complus);
+                    row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.Complus, id);
-                    row.Set(0, id);
                    row.Set(1, comPlusBits);
                }
@@ -4832,7 +4829,7 @@ namespace WixToolset.Core
            if (!this.Core.EncounteredError)
            {
                var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.Feature, id);
-                row.Set(1, null); // this column is set in the linker
+                // row.Set(1, null); - this column is set in the linker
                row.Set(2, title);
                row.Set(3, description);
                if (0 < display.Length)
@@ -9543,7 +9540,6 @@ namespace WixToolset.Core
                }
            }
            if (!this.Core.EncounteredError)
            {
                var patchIdRow = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.WixPatchId);
@@ -9555,7 +9551,6 @@ namespace WixToolset.Core
                if (allowRemoval)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "AllowRemoval");
                    row.Set(2, allowRemoval ? "1" : "0");
                }
@@ -9563,7 +9558,6 @@ namespace WixToolset.Core
                if (null != classification)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "Classification");
                    row.Set(2, classification);
                }
@@ -9571,7 +9565,6 @@ namespace WixToolset.Core
                // always generate the CreationTimeUTC
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "CreationTimeUTC");
                    row.Set(2, DateTime.UtcNow.ToString("MM-dd-yy HH:mm", CultureInfo.InvariantCulture));
                }
@@ -9579,7 +9572,6 @@ namespace WixToolset.Core
                if (null != description)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "Description");
                    row.Set(2, description);
                }
@@ -9587,7 +9579,6 @@ namespace WixToolset.Core
                if (null != displayName)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "DisplayName");
                    row.Set(2, displayName);
                }
@@ -9595,7 +9586,6 @@ namespace WixToolset.Core
                if (null != manufacturer)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "ManufacturerName");
                    row.Set(2, manufacturer);
                }
@@ -9603,7 +9593,6 @@ namespace WixToolset.Core
                if (YesNoType.NotSet != minorUpdateTargetRTM)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "MinorUpdateTargetRTM");
                    row.Set(2, YesNoType.Yes == minorUpdateTargetRTM ? "1" : "0");
                }
@@ -9611,7 +9600,6 @@ namespace WixToolset.Core
                if (null != moreInfoUrl)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "MoreInfoURL");
                    row.Set(2, moreInfoUrl);
                }
@@ -9619,7 +9607,6 @@ namespace WixToolset.Core
                if (CompilerConstants.IntegerNotSet != optimizeCA)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "OptimizeCA");
                    row.Set(2, optimizeCA.ToString(CultureInfo.InvariantCulture));
                }
@@ -9627,7 +9614,6 @@ namespace WixToolset.Core
                if (YesNoType.NotSet != optimizedInstallMode)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "OptimizedInstallMode");
                    row.Set(2, YesNoType.Yes == optimizedInstallMode ? "1" : "0");
                }
@@ -9635,7 +9621,6 @@ namespace WixToolset.Core
                if (null != targetProductName)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiPatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "TargetProductName");
                    row.Set(2, targetProductName);
                }
@@ -11190,7 +11175,6 @@ namespace WixToolset.Core
                if (YesNoType.NotSet != allowRemoval)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "AllowRemoval");
                    row.Set(2, YesNoType.Yes == allowRemoval ? "1" : "0");
                }
@@ -11198,7 +11182,6 @@ namespace WixToolset.Core
                if (null != classification)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "Classification");
                    row.Set(2, classification);
                }
@@ -11206,7 +11189,6 @@ namespace WixToolset.Core
                if (null != creationTimeUtc)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "CreationTimeUTC");
                    row.Set(2, creationTimeUtc);
                }
@@ -11214,7 +11196,6 @@ namespace WixToolset.Core
                if (null != description)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "Description");
                    row.Set(2, description);
                }
@@ -11222,7 +11203,6 @@ namespace WixToolset.Core
                if (null != displayName)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "DisplayName");
                    row.Set(2, displayName);
                }
@@ -11230,7 +11210,6 @@ namespace WixToolset.Core
                if (null != manufacturerName)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "ManufacturerName");
                    row.Set(2, manufacturerName);
                }
@@ -11238,7 +11217,6 @@ namespace WixToolset.Core
                if (null != minorUpdateTargetRTM)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "MinorUpdateTargetRTM");
                    row.Set(2, minorUpdateTargetRTM);
                }
@@ -11246,7 +11224,6 @@ namespace WixToolset.Core
                if (null != moreInfoUrl)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "MoreInfoURL");
                    row.Set(2, moreInfoUrl);
                }
@@ -11254,7 +11231,6 @@ namespace WixToolset.Core
                if (CompilerConstants.IntegerNotSet != optimizeCA)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "OptimizeCA");
                    row.Set(2, optimizeCA.ToString(CultureInfo.InvariantCulture));
                }
@@ -11262,7 +11238,6 @@ namespace WixToolset.Core
                if (YesNoType.NotSet != optimizedInstallMode)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "OptimizedInstallMode");
                    row.Set(2, YesNoType.Yes == optimizedInstallMode ? "1" : "0");
                }
@@ -11270,7 +11245,6 @@ namespace WixToolset.Core
                if (null != targetProductName)
                {
                    var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.PatchMetadata);
-                    row.Set(0, null);
                    row.Set(1, "TargetProductName");
                    row.Set(2, targetProductName);
                }
@@ -12587,7 +12561,7 @@ namespace WixToolset.Core
                row.Set(1, root);
                row.Set(2, key);
                row.Set(3, name);
-                row.Set(4, null);
+                //row.Set(4, null);
                row.Set(5, componentId);
            }
@@ -12914,7 +12888,7 @@ namespace WixToolset.Core
                row.Set(3, name);
                if (Wix.RemoveRegistryKey.ActionType.removeOnUninstall == actionType) // Registry table
                {
-                    row.Set(4, null);
+                    //row.Set(4, null);
                    row.Set(5, componentId);
                }
                else // RemoveRegistry table
@@ -13206,7 +13180,7 @@ namespace WixToolset.Core
            {
                var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.RemoveFile, id);
                row.Set(1, componentId);
-                row.Set(2, null);
+                //row.Set(2, null);
                if (null != directory)
                {
                    row.Set(3, directory);
@@ -16848,7 +16822,7 @@ namespace WixToolset.Core
                var row = this.Core.CreateRow(sourceLineNumbers, TupleDefinitionType.MsiEmbeddedUI, id);
                row.Set(1, name);
                row.Set(2, 0); // embedded UI resources always set this to 0
-                row.Set(3, null);
+                //row.Set(3, null);
                row.Set(4, sourceFile);
            }
        }
@@ -19611,10 +19585,6 @@ namespace WixToolset.Core
                {
                    sourceFile = Path.Combine("SourceDir", name);
                }
-                else
-                {
-                    sourceFile = String.Empty;  // SourceFile is required it cannot be null.
-                }
            }
            else if (null != remotePayload)
            {
diff --git a/src/WixToolset.Core/WixToolset.Core.csproj b/src/WixToolset.Core/WixToolset.Core.csproj
index bfa84aa8..07cc692c 100644
--- a/src/WixToolset.Core/WixToolset.Core.csproj
+++ b/src/WixToolset.Core/WixToolset.Core.csproj
@@ -9,14 +9,14 @@
  </PropertyGroup>
  <ItemGroup>
-    <ProjectReference Include="$(WixToolsetRootFolder)\Data\src\WixToolset.Data\WixToolset.Data.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Data\src\WixToolset.Data\WixToolset.Data.csproj') " />
+    <ProjectReference Include="$(WixToolsetRootFolder)\Data\src\WixToolset.Data\WixToolset.Data.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Data\README.md') " />
-    <PackageReference Include="WixToolset.Data" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Data\src\WixToolset.Data\WixToolset.Data.csproj') " />
+    <PackageReference Include="WixToolset.Data" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Data\README.md') " />
-    <ProjectReference Include="$(WixToolsetRootFolder)\Extensibility\src\WixToolset.Extensibility\WixToolset.Extensibility.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Extensibility\src\WixToolset.Extensibility\WixToolset.Extensibility.csproj') " />
+    <ProjectReference Include="$(WixToolsetRootFolder)\Extensibility\src\WixToolset.Extensibility\WixToolset.Extensibility.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Extensibility\README.md') " />
-    <PackageReference Include="WixToolset.Extensibility" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Extensibility\src\WixToolset.Extensibility\WixToolset.Extensibility.csproj') " />
+    <PackageReference Include="WixToolset.Extensibility" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Extensibility\README.md') " />
-    <ProjectReference Include="$(WixToolsetRootFolder)\Core.Native\src\WixToolset.Core.Native\WixToolset.Core.Native.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Core.Native\src\WixToolset.Core.Native\WixToolset.Core.Native.csproj') " />
+    <ProjectReference Include="$(WixToolsetRootFolder)\Core.Native\src\WixToolset.Core.Native\WixToolset.Core.Native.csproj" Condition=" '$(Configuration)' == 'Debug' And Exists('$(WixToolsetRootFolder)\Core.Native\README.md') " />
-    <PackageReference Include="WixToolset.Core.Native" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Core.Native\src\WixToolset.Core.Native\WixToolset.Core.Native.csproj') " />
+    <PackageReference Include="WixToolset.Core.Native" Version="4.0.*" Condition=" '$(Configuration)' == 'Release' Or !Exists('$(WixToolsetRootFolder)\Core.Native\README.md') " />
  </ItemGroup>
  <ItemGroup>
author	Rob Mensching <rob@firegiant.com>	2018-01-01 16:35:30 -0800
committer	Rob Mensching <rob@firegiant.com>	2018-01-01 16:35:30 -0800
commit	58f9e097a445dbdf3139832ac471374e68678fc6 (patch)
tree	641e3b0cff1287459ac29eb26a7f810f1d4afb7f /src
parent	651ad904724f32f5e993fa73aa11a611d95a1a10 (diff)
download	wix-58f9e097a445dbdf3139832ac471374e68678fc6.tar.gz wix-58f9e097a445dbdf3139832ac471374e68678fc6.tar.bz2 wix-58f9e097a445dbdf3139832ac471374e68678fc6.zip