diff options
author | Rob Mensching <rob@firegiant.com> | 2023-07-19 10:20:19 -0700 |
---|---|---|
committer | Rob Mensching <rob@firegiant.com> | 2023-07-19 11:55:19 -0700 |
commit | 42118ef58ba698280b5c28c69f86029c9152072f (patch) | |
tree | 5c947a305785e5a277839aa749d428c72625f16a /src | |
parent | 7a603e4d5a9c68f721464df15da2619b084fdb18 (diff) | |
download | wix-42118ef58ba698280b5c28c69f86029c9152072f.tar.gz wix-42118ef58ba698280b5c28c69f86029c9152072f.tar.bz2 wix-42118ef58ba698280b5c28c69f86029c9152072f.zip |
Fix out of date references to Product in warnings
Fixes 7631
Diffstat (limited to 'src')
-rw-r--r-- | src/api/wix/WixToolset.Data/WarningMessages.cs | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/src/api/wix/WixToolset.Data/WarningMessages.cs b/src/api/wix/WixToolset.Data/WarningMessages.cs index d54990e1..790291e4 100644 --- a/src/api/wix/WixToolset.Data/WarningMessages.cs +++ b/src/api/wix/WixToolset.Data/WarningMessages.cs | |||
@@ -57,9 +57,9 @@ namespace WixToolset.Data | |||
57 | return Message(sourceLineNumbers, Ids.CannotUpdateCabCache, "Cannot update the timestamp of cached cabinet: '{0}'. If the timestamp is not updated, the build may rebuild more than is necessary. To fix the issue, ensure that the cabinet file is writable, error: {1}", cabinetPath, detail); | 57 | return Message(sourceLineNumbers, Ids.CannotUpdateCabCache, "Cannot update the timestamp of cached cabinet: '{0}'. If the timestamp is not updated, the build may rebuild more than is necessary. To fix the issue, ensure that the cabinet file is writable, error: {1}", cabinetPath, detail); |
58 | } | 58 | } |
59 | 59 | ||
60 | public static Message ColumnsIncompatibleWithInstallerVersion(SourceLineNumber sourceLineNumbers, string tableName, int productInstallerVersion) | 60 | public static Message ColumnsIncompatibleWithInstallerVersion(SourceLineNumber sourceLineNumbers, string tableName, int packageInstallerVersion) |
61 | { | 61 | { |
62 | return Message(sourceLineNumbers, Ids.ColumnsIncompatibleWithInstallerVersion, "Table '{0}' uses columns that require a version of Windows Installer greater than specified in your package ('{1}').", tableName, productInstallerVersion); | 62 | return Message(sourceLineNumbers, Ids.ColumnsIncompatibleWithInstallerVersion, "Table '{0}' uses columns that require a version of Windows Installer greater than specified in your package ('{1}').", tableName, packageInstallerVersion); |
63 | } | 63 | } |
64 | 64 | ||
65 | public static Message CopyFileFileIdUseless(SourceLineNumber sourceLineNumbers) | 65 | public static Message CopyFileFileIdUseless(SourceLineNumber sourceLineNumbers) |
@@ -199,7 +199,7 @@ namespace WixToolset.Data | |||
199 | 199 | ||
200 | public static Message DeprecatedUpgradeProperty(SourceLineNumber sourceLineNumbers) | 200 | public static Message DeprecatedUpgradeProperty(SourceLineNumber sourceLineNumbers) |
201 | { | 201 | { |
202 | return Message(sourceLineNumbers, Ids.DeprecatedUpgradeProperty, "Specifying a Property element as a child of an Upgrade element has been deprecated. Please specify this Property element as a child of a different element such as Product or Fragment."); | 202 | return Message(sourceLineNumbers, Ids.DeprecatedUpgradeProperty, "Specifying a Property element as a child of an Upgrade element has been deprecated. Please specify this Property element as a child of a different element such as Package or Fragment."); |
203 | } | 203 | } |
204 | 204 | ||
205 | public static Message DirectoryInUse(SourceLineNumber sourceLineNumbers, string filePath) | 205 | public static Message DirectoryInUse(SourceLineNumber sourceLineNumbers, string filePath) |
@@ -367,9 +367,9 @@ namespace WixToolset.Data | |||
367 | return Message(sourceLineNumbers, Ids.InvalidAttributeCombination, "It is invalid to combine attributes {0} and {1}. The decompiled output will set attribute {2} to {3}.", attrib1, attrib2, name, value); | 367 | return Message(sourceLineNumbers, Ids.InvalidAttributeCombination, "It is invalid to combine attributes {0} and {1}. The decompiled output will set attribute {2} to {3}.", attrib1, attrib2, name, value); |
368 | } | 368 | } |
369 | 369 | ||
370 | public static Message InvalidHigherInstallerVersionInModule(SourceLineNumber sourceLineNumbers, string moduleId, int moduleInstallerVersion, int productInstallerVersion) | 370 | public static Message InvalidHigherInstallerVersionInModule(SourceLineNumber sourceLineNumbers, string moduleId, int moduleInstallerVersion, int packageInstallerVersion) |
371 | { | 371 | { |
372 | return Message(sourceLineNumbers, Ids.InvalidHigherInstallerVersionInModule, "Merge module '{0}' has an installer version of {1} which is greater than the product's installer version of {2}. Merging a module with a higher installer version than the product it is being merged into can result in invalid values in the resulting msi. You must set the Package/@InstallerVersion attribute to {1} or greater to merge this merge module into your product.", moduleId, moduleInstallerVersion, productInstallerVersion); | 372 | return Message(sourceLineNumbers, Ids.InvalidHigherInstallerVersionInModule, "Merge module '{0}' has an installer version of {1} which is greater than the package's installer version of {2}. Merging a module with a higher installer version than the package it is being merged into can result in invalid values in the resulting msi. You must set the Package/@InstallerVersion attribute to {1} or greater to merge this merge module into your package.", moduleId, moduleInstallerVersion, packageInstallerVersion); |
373 | } | 373 | } |
374 | 374 | ||
375 | public static Message InvalidFourPartVersion(SourceLineNumber sourceLineNumbers, string elementName, string version) | 375 | public static Message InvalidFourPartVersion(SourceLineNumber sourceLineNumbers, string elementName, string version) |
@@ -384,7 +384,7 @@ namespace WixToolset.Data | |||
384 | 384 | ||
385 | public static Message MajorUpgradePatchNotRecommended() | 385 | public static Message MajorUpgradePatchNotRecommended() |
386 | { | 386 | { |
387 | return Message(null, Ids.MajorUpgradePatchNotRecommended, "Changing the ProductCode in a patch is not recommended because the patch cannot be uninstalled nor can it be sequenced along with other patches for the target product. See https://learn.microsoft.com/en-us/windows/win32/msi/applying-major-upgrades-by-patching-the-local-installation-of-the-product for more information."); | 387 | return Message(null, Ids.MajorUpgradePatchNotRecommended, "Changing the ProductCode in a patch is not recommended because the patch cannot be uninstalled nor can it be sequenced along with other patches for the target package. See https://learn.microsoft.com/en-us/windows/win32/msi/applying-major-upgrades-by-patching-the-local-installation-of-the-product for more information."); |
388 | } | 388 | } |
389 | 389 | ||
390 | public static Message MediaExternalCabinetFilenameIllegal(SourceLineNumber sourceLineNumbers, string elementName, string attributeName, string value) | 390 | public static Message MediaExternalCabinetFilenameIllegal(SourceLineNumber sourceLineNumbers, string elementName, string attributeName, string value) |
@@ -404,7 +404,7 @@ namespace WixToolset.Data | |||
404 | 404 | ||
405 | public static Message MissingUpgradeCode(SourceLineNumber sourceLineNumbers) | 405 | public static Message MissingUpgradeCode(SourceLineNumber sourceLineNumbers) |
406 | { | 406 | { |
407 | return Message(sourceLineNumbers, Ids.MissingUpgradeCode, "The Product/@UpgradeCode attribute was not found; it is strongly recommended to ensure that this product can be upgraded."); | 407 | return Message(sourceLineNumbers, Ids.MissingUpgradeCode, "The Package/@UpgradeCode attribute was not found; it is strongly recommended to ensure that this package can be upgraded."); |
408 | } | 408 | } |
409 | 409 | ||
410 | public static Message MsiTransactionLimitations(SourceLineNumber sourceLineNumbers) | 410 | public static Message MsiTransactionLimitations(SourceLineNumber sourceLineNumbers) |
@@ -572,9 +572,9 @@ namespace WixToolset.Data | |||
572 | return Message(null, Ids.SuppressMergedAction, "The merged action '{0}' in the {1} table is being suppressed.", action, sequenceName); | 572 | return Message(null, Ids.SuppressMergedAction, "The merged action '{0}' in the {1} table is being suppressed.", action, sequenceName); |
573 | } | 573 | } |
574 | 574 | ||
575 | public static Message TableIncompatibleWithInstallerVersion(SourceLineNumber sourceLineNumbers, string tableName, int productInstallerVersion) | 575 | public static Message TableIncompatibleWithInstallerVersion(SourceLineNumber sourceLineNumbers, string tableName, int packageInstallerVersion) |
576 | { | 576 | { |
577 | return Message(sourceLineNumbers, Ids.TableIncompatibleWithInstallerVersion, "Using table '{0}' requires a version of Windows Installer greater than specified in your package ('{1}').", tableName, productInstallerVersion); | 577 | return Message(sourceLineNumbers, Ids.TableIncompatibleWithInstallerVersion, "Using table '{0}' requires a version of Windows Installer greater than specified in your package ('{1}').", tableName, packageInstallerVersion); |
578 | } | 578 | } |
579 | 579 | ||
580 | public static Message TargetDirCorrectedDefaultDir() | 580 | public static Message TargetDirCorrectedDefaultDir() |
@@ -620,7 +620,7 @@ namespace WixToolset.Data | |||
620 | 620 | ||
621 | public static Message UnexpectedTableInProduct(SourceLineNumber sourceLineNumbers, string tableName) | 621 | public static Message UnexpectedTableInProduct(SourceLineNumber sourceLineNumbers, string tableName) |
622 | { | 622 | { |
623 | return Message(sourceLineNumbers, Ids.UnexpectedTableInProduct, "An unexpected row in the '{0}' table was found in this product. Products should not contain the '{0}' table.", tableName); | 623 | return Message(sourceLineNumbers, Ids.UnexpectedTableInProduct, "An unexpected row in the '{0}' table was found in this package. Packages should not contain the '{0}' table.", tableName); |
624 | } | 624 | } |
625 | 625 | ||
626 | public static Message UnknownAction(SourceLineNumber sourceLineNumbers, string sequenceTableName, string actionName) | 626 | public static Message UnknownAction(SourceLineNumber sourceLineNumbers, string sequenceTableName, string actionName) |
@@ -675,14 +675,14 @@ namespace WixToolset.Data | |||
675 | 675 | ||
676 | public static Message InvalidMsiProductVersion(SourceLineNumber sourceLineNumbers, string version, string package) | 676 | public static Message InvalidMsiProductVersion(SourceLineNumber sourceLineNumbers, string version, string package) |
677 | { | 677 | { |
678 | return Message(sourceLineNumbers, Ids.InvalidMsiProductVersion, "Invalid product version '{0}' in MSI package '{1}'. Product version should have a major version less than 256, a minor version less than 256, and a build version less than 65536. The bundle may incorrectly detect upgrades of this package.", version, package); | 678 | return Message(sourceLineNumbers, Ids.InvalidMsiProductVersion, "Invalid package version '{0}' in MSI package '{1}'. Package version should have a major version less than 256, a minor version less than 256, and a build version less than 65536. The bundle may incorrectly detect upgrades of this package.", version, package); |
679 | } | 679 | } |
680 | 680 | ||
681 | public static Message InvalidMsiProductVersion(SourceLineNumber sourceLineNumbers, string version) | 681 | public static Message InvalidMsiProductVersion(SourceLineNumber sourceLineNumbers, string version) |
682 | { | 682 | { |
683 | return Message(sourceLineNumbers, Ids.InvalidMsiProductVersion, | 683 | return Message(sourceLineNumbers, Ids.InvalidMsiProductVersion, |
684 | "Invalid MSI package version: '{0}'. " + | 684 | "Invalid MSI package version: '{0}'. " + |
685 | "The Windows Installer SDK says that MSI product versions must have a major version less than 256, a minor version less than 256, and a build version less than 65536. " + | 685 | "The Windows Installer SDK says that MSI package versions must have a major version less than 256, a minor version less than 256, and a build version less than 65536. " + |
686 | "The revision value is ignored but version labels and metadata are not allowed. " + | 686 | "The revision value is ignored but version labels and metadata are not allowed. " + |
687 | "Violating the MSI rules sometimes works as expected but the behavior is unpredictable and undefined. "+ | 687 | "Violating the MSI rules sometimes works as expected but the behavior is unpredictable and undefined. "+ |
688 | "Future versions of WiX might treat invalid package versions as an error.", | 688 | "Future versions of WiX might treat invalid package versions as an error.", |