aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBob Arnson <bob@firegiant.com>2020-10-28 17:10:38 -0400
committerBob Arnson <bob@firegiant.com>2020-10-28 17:15:50 -0400
commitf28de8b2b38a09af130bc128b6226a2010ab535d (patch)
treef1b43011c2d965c5cec6d3592d7a891b14204401
parent20cc160df97aa67a4bfc6452943c4d1b86bf45e2 (diff)
downloadwix-f28de8b2b38a09af130bc128b6226a2010ab535d.tar.gz
wix-f28de8b2b38a09af130bc128b6226a2010ab535d.tar.bz2
wix-f28de8b2b38a09af130bc128b6226a2010ab535d.zip
A bit of cleanup on now-defunct Product element references.
-rw-r--r--src/WixToolset.Core.WindowsInstaller/WindowsInstallerBackendFactory.cs1
-rw-r--r--src/WixToolset.Core/Compiler.cs4
-rw-r--r--src/WixToolset.Core/Linker.cs8
3 files changed, 7 insertions, 6 deletions
diff --git a/src/WixToolset.Core.WindowsInstaller/WindowsInstallerBackendFactory.cs b/src/WixToolset.Core.WindowsInstaller/WindowsInstallerBackendFactory.cs
index f9cf4492..f72acb21 100644
--- a/src/WixToolset.Core.WindowsInstaller/WindowsInstallerBackendFactory.cs
+++ b/src/WixToolset.Core.WindowsInstaller/WindowsInstallerBackendFactory.cs
@@ -23,6 +23,7 @@ namespace WixToolset.Core.WindowsInstaller
23 return true; 23 return true;
24 24
25 case "msipackage": 25 case "msipackage":
26 case "package":
26 case "product": 27 case "product":
27 case ".msi": 28 case ".msi":
28 backend = new MsiBackend(); 29 backend = new MsiBackend();
diff --git a/src/WixToolset.Core/Compiler.cs b/src/WixToolset.Core/Compiler.cs
index a575bb95..3dfe36c2 100644
--- a/src/WixToolset.Core/Compiler.cs
+++ b/src/WixToolset.Core/Compiler.cs
@@ -6962,13 +6962,13 @@ namespace WixToolset.Core
6962 var upgradeCode = contextValues["UpgradeCode"]; 6962 var upgradeCode = contextValues["UpgradeCode"];
6963 if (String.IsNullOrEmpty(upgradeCode)) 6963 if (String.IsNullOrEmpty(upgradeCode))
6964 { 6964 {
6965 this.Core.Write(ErrorMessages.ParentElementAttributeRequired(sourceLineNumbers, "Product", "UpgradeCode", node.Name.LocalName)); 6965 this.Core.Write(ErrorMessages.ParentElementAttributeRequired(sourceLineNumbers, "Package", "UpgradeCode", node.Name.LocalName));
6966 } 6966 }
6967 6967
6968 var productVersion = contextValues["ProductVersion"]; 6968 var productVersion = contextValues["ProductVersion"];
6969 if (String.IsNullOrEmpty(productVersion)) 6969 if (String.IsNullOrEmpty(productVersion))
6970 { 6970 {
6971 this.Core.Write(ErrorMessages.ParentElementAttributeRequired(sourceLineNumbers, "Product", "Version", node.Name.LocalName)); 6971 this.Core.Write(ErrorMessages.ParentElementAttributeRequired(sourceLineNumbers, "Package", "Version", node.Name.LocalName));
6972 } 6972 }
6973 6973
6974 var productLanguage = contextValues["ProductLanguage"]; 6974 var productLanguage = contextValues["ProductLanguage"];
diff --git a/src/WixToolset.Core/Linker.cs b/src/WixToolset.Core/Linker.cs
index 1cfd085d..86eb7a3d 100644
--- a/src/WixToolset.Core/Linker.cs
+++ b/src/WixToolset.Core/Linker.cs
@@ -793,7 +793,7 @@ namespace WixToolset.Core
793 { 793 {
794 if (connection.IsExplicitPrimaryFeature) 794 if (connection.IsExplicitPrimaryFeature)
795 { 795 {
796 this.Messaging.Write(ErrorMessages.MultiplePrimaryReferences(wixComplexReferenceRow.SourceLineNumbers, wixComplexReferenceRow.ChildType.ToString(), wixComplexReferenceRow.Child, wixComplexReferenceRow.ParentType.ToString(), wixComplexReferenceRow.Parent, (null != connection.PrimaryFeature ? "Feature" : "Product"), connection.PrimaryFeature ?? resolvedSection.Id)); 796 this.Messaging.Write(ErrorMessages.MultiplePrimaryReferences(wixComplexReferenceRow.SourceLineNumbers, wixComplexReferenceRow.ChildType.ToString(), wixComplexReferenceRow.Child, wixComplexReferenceRow.ParentType.ToString(), wixComplexReferenceRow.Parent, (null != connection.PrimaryFeature ? "Feature" : "Package"), connection.PrimaryFeature ?? resolvedSection.Id));
797 continue; 797 continue;
798 } 798 }
799 else 799 else
@@ -825,7 +825,7 @@ namespace WixToolset.Core
825 connection = featuresToFeatures[wixComplexReferenceRow.Child]; 825 connection = featuresToFeatures[wixComplexReferenceRow.Child];
826 if (null != connection) 826 if (null != connection)
827 { 827 {
828 this.Messaging.Write(ErrorMessages.MultiplePrimaryReferences(wixComplexReferenceRow.SourceLineNumbers, wixComplexReferenceRow.ChildType.ToString(), wixComplexReferenceRow.Child, wixComplexReferenceRow.ParentType.ToString(), wixComplexReferenceRow.Parent, (null != connection.PrimaryFeature ? "Feature" : "Product"), (null != connection.PrimaryFeature ? connection.PrimaryFeature : resolvedSection.Id))); 828 this.Messaging.Write(ErrorMessages.MultiplePrimaryReferences(wixComplexReferenceRow.SourceLineNumbers, wixComplexReferenceRow.ChildType.ToString(), wixComplexReferenceRow.Child, wixComplexReferenceRow.ParentType.ToString(), wixComplexReferenceRow.Parent, (null != connection.PrimaryFeature ? "Feature" : "Package"), (null != connection.PrimaryFeature ? connection.PrimaryFeature : resolvedSection.Id)));
829 continue; 829 continue;
830 } 830 }
831 831
@@ -842,7 +842,7 @@ namespace WixToolset.Core
842 { 842 {
843 if (connection.IsExplicitPrimaryFeature) 843 if (connection.IsExplicitPrimaryFeature)
844 { 844 {
845 this.Messaging.Write(ErrorMessages.MultiplePrimaryReferences(wixComplexReferenceRow.SourceLineNumbers, wixComplexReferenceRow.ChildType.ToString(), wixComplexReferenceRow.Child, wixComplexReferenceRow.ParentType.ToString(), wixComplexReferenceRow.Parent, (null != connection.PrimaryFeature ? "Feature" : "Product"), (null != connection.PrimaryFeature ? connection.PrimaryFeature : resolvedSection.Id))); 845 this.Messaging.Write(ErrorMessages.MultiplePrimaryReferences(wixComplexReferenceRow.SourceLineNumbers, wixComplexReferenceRow.ChildType.ToString(), wixComplexReferenceRow.Child, wixComplexReferenceRow.ParentType.ToString(), wixComplexReferenceRow.Parent, (null != connection.PrimaryFeature ? "Feature" : "Package"), (null != connection.PrimaryFeature ? connection.PrimaryFeature : resolvedSection.Id)));
846 continue; 846 continue;
847 } 847 }
848 else 848 else
@@ -915,7 +915,7 @@ namespace WixToolset.Core
915 connection = featuresToFeatures[wixComplexReferenceRow.Child]; 915 connection = featuresToFeatures[wixComplexReferenceRow.Child];
916 if (null != connection) 916 if (null != connection)
917 { 917 {
918 this.Messaging.Write(ErrorMessages.MultiplePrimaryReferences(wixComplexReferenceRow.SourceLineNumbers, wixComplexReferenceRow.ChildType.ToString(), wixComplexReferenceRow.Child, wixComplexReferenceRow.ParentType.ToString(), wixComplexReferenceRow.Parent, (null != connection.PrimaryFeature ? "Feature" : "Product"), (null != connection.PrimaryFeature ? connection.PrimaryFeature : resolvedSection.Id))); 918 this.Messaging.Write(ErrorMessages.MultiplePrimaryReferences(wixComplexReferenceRow.SourceLineNumbers, wixComplexReferenceRow.ChildType.ToString(), wixComplexReferenceRow.Child, wixComplexReferenceRow.ParentType.ToString(), wixComplexReferenceRow.Parent, (null != connection.PrimaryFeature ? "Feature" : "Package"), (null != connection.PrimaryFeature ? connection.PrimaryFeature : resolvedSection.Id)));
919 continue; 919 continue;
920 } 920 }
921 921