diff options
author | Bob Arnson <bob@firegiant.com> | 2020-10-28 17:10:38 -0400 |
---|---|---|
committer | Bob Arnson <bob@firegiant.com> | 2020-10-28 17:15:50 -0400 |
commit | f28de8b2b38a09af130bc128b6226a2010ab535d (patch) | |
tree | f1b43011c2d965c5cec6d3592d7a891b14204401 | |
parent | 20cc160df97aa67a4bfc6452943c4d1b86bf45e2 (diff) | |
download | wix-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.cs | 1 | ||||
-rw-r--r-- | src/WixToolset.Core/Compiler.cs | 4 | ||||
-rw-r--r-- | src/WixToolset.Core/Linker.cs | 8 |
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 | ||