Improve error reporting wrt ExePackage and MsuPackage attributes

For example, DetectCondition is required when RepairArguments or
UninstallArguments present and always recommended. Also, non-permanent
ExePackages need UninstallArguments. The code was refactored to make
it easier to reason over the different requirements for different
package types.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/ext/Bal/test/examples/LatestCoreBundleFDD/FrameworkDependentBundle.wxs b/src/ext/Bal/test/examples/LatestCoreBundleFDD/FrameworkDependentBundle.wxs
index 78ac8220..b29363dd 100644
--- a/src/ext/Bal/test/examples/LatestCoreBundleFDD/FrameworkDependentBundle.wxs
+++ b/src/ext/Bal/test/examples/LatestCoreBundleFDD/FrameworkDependentBundle.wxs
@@ -9,7 +9,7 @@
             <bal:WixDotNetCoreBootstrapperApplicationHost />
         </BootstrapperApplication>
         <Chain>
-            <ExePackage DetectCondition="none" SourceFile="..\.data\notanexe.exe" bal:PrereqPackage="yes" />
+            <ExePackage DetectCondition="none" UninstallArguments="-foo" SourceFile="..\.data\notanexe.exe" bal:PrereqPackage="yes" />
         </Chain>
     </Bundle>
 </Wix>