diff options
| author | Sean Hall <r.sean.hall@gmail.com> | 2022-04-01 15:44:34 -0500 |
|---|---|---|
| committer | Sean Hall <r.sean.hall@gmail.com> | 2022-04-01 22:06:11 -0500 |
| commit | 39b9a6112c2ff97f31f195749e2142538e47a2eb (patch) | |
| tree | 8b2337b589fa5f52fabce89c99d3fca0ef1c8fc0 /src | |
| parent | 386a3578413ba16b3c0615d47870ee44a0e461f6 (diff) | |
| download | wix-39b9a6112c2ff97f31f195749e2142538e47a2eb.tar.gz wix-39b9a6112c2ff97f31f195749e2142538e47a2eb.tar.bz2 wix-39b9a6112c2ff97f31f195749e2142538e47a2eb.zip | |
Detect related bundles for BundlePackages.
Diffstat (limited to 'src')
38 files changed, 773 insertions, 238 deletions
diff --git a/src/api/burn/WixToolset.BootstrapperCore.Native/inc/BootstrapperApplication.h b/src/api/burn/WixToolset.BootstrapperCore.Native/inc/BootstrapperApplication.h index 943f5ead..fbbd10ee 100644 --- a/src/api/burn/WixToolset.BootstrapperCore.Native/inc/BootstrapperApplication.h +++ b/src/api/burn/WixToolset.BootstrapperCore.Native/inc/BootstrapperApplication.h | |||
| @@ -226,6 +226,7 @@ enum BOOTSTRAPPER_APPLICATION_MESSAGE | |||
| 226 | BOOTSTRAPPER_APPLICATION_MESSAGE_ONPLANRELATEDBUNDLETYPE, | 226 | BOOTSTRAPPER_APPLICATION_MESSAGE_ONPLANRELATEDBUNDLETYPE, |
| 227 | BOOTSTRAPPER_APPLICATION_MESSAGE_ONAPPLYDOWNGRADE, | 227 | BOOTSTRAPPER_APPLICATION_MESSAGE_ONAPPLYDOWNGRADE, |
| 228 | BOOTSTRAPPER_APPLICATION_MESSAGE_ONEXECUTEPROCESSCANCEL, | 228 | BOOTSTRAPPER_APPLICATION_MESSAGE_ONEXECUTEPROCESSCANCEL, |
| 229 | BOOTSTRAPPER_APPLICATION_MESSAGE_ONDETECTRELATEDBUNDLEPACKAGE, | ||
| 229 | }; | 230 | }; |
| 230 | 231 | ||
| 231 | enum BOOTSTRAPPER_APPLYCOMPLETE_ACTION | 232 | enum BOOTSTRAPPER_APPLYCOMPLETE_ACTION |
| @@ -793,6 +794,22 @@ struct BA_ONDETECTRELATEDBUNDLE_RESULTS | |||
| 793 | BOOL fCancel; | 794 | BOOL fCancel; |
| 794 | }; | 795 | }; |
| 795 | 796 | ||
| 797 | struct BA_ONDETECTRELATEDBUNDLEPACKAGE_ARGS | ||
| 798 | { | ||
| 799 | DWORD cbSize; | ||
| 800 | LPCWSTR wzPackageId; | ||
| 801 | LPCWSTR wzBundleId; | ||
| 802 | BOOTSTRAPPER_RELATION_TYPE relationType; | ||
| 803 | BOOL fPerMachine; | ||
| 804 | LPCWSTR wzVersion; | ||
| 805 | }; | ||
| 806 | |||
| 807 | struct BA_ONDETECTRELATEDBUNDLEPACKAGE_RESULTS | ||
| 808 | { | ||
| 809 | DWORD cbSize; | ||
| 810 | BOOL fCancel; | ||
| 811 | }; | ||
| 812 | |||
| 796 | struct BA_ONDETECTRELATEDMSIPACKAGE_ARGS | 813 | struct BA_ONDETECTRELATEDMSIPACKAGE_ARGS |
| 797 | { | 814 | { |
| 798 | DWORD cbSize; | 815 | DWORD cbSize; |
diff --git a/src/api/burn/WixToolset.Mba.Core/BootstrapperApplication.cs b/src/api/burn/WixToolset.Mba.Core/BootstrapperApplication.cs index 5ed064fa..41738bf6 100644 --- a/src/api/burn/WixToolset.Mba.Core/BootstrapperApplication.cs +++ b/src/api/burn/WixToolset.Mba.Core/BootstrapperApplication.cs | |||
| @@ -283,6 +283,9 @@ namespace WixToolset.Mba.Core | |||
| 283 | /// <inheritdoc/> | 283 | /// <inheritdoc/> |
| 284 | public event EventHandler<ExecuteProcessCancelEventArgs> ExecuteProcessCancel; | 284 | public event EventHandler<ExecuteProcessCancelEventArgs> ExecuteProcessCancel; |
| 285 | 285 | ||
| 286 | /// <inheritdoc/> | ||
| 287 | public event EventHandler<DetectRelatedBundlePackageEventArgs> DetectRelatedBundlePackage; | ||
| 288 | |||
| 286 | /// <summary> | 289 | /// <summary> |
| 287 | /// Entry point that is called when the bootstrapper application is ready to run. | 290 | /// Entry point that is called when the bootstrapper application is ready to run. |
| 288 | /// </summary> | 291 | /// </summary> |
| @@ -1385,6 +1388,19 @@ namespace WixToolset.Mba.Core | |||
| 1385 | } | 1388 | } |
| 1386 | } | 1389 | } |
| 1387 | 1390 | ||
| 1391 | /// <summary> | ||
| 1392 | /// Called by the engine, raises the <see cref="DetectRelatedBundlePackage"/> event. | ||
| 1393 | /// </summary> | ||
| 1394 | /// <param name="args">Additional arguments for this event.</param> | ||
| 1395 | protected virtual void OnDetectRelatedBundlePackage(DetectRelatedBundlePackageEventArgs args) | ||
| 1396 | { | ||
| 1397 | EventHandler<DetectRelatedBundlePackageEventArgs> handler = this.DetectRelatedBundlePackage; | ||
| 1398 | if (null != handler) | ||
| 1399 | { | ||
| 1400 | handler(this, args); | ||
| 1401 | } | ||
| 1402 | } | ||
| 1403 | |||
| 1388 | #region IBootstrapperApplication Members | 1404 | #region IBootstrapperApplication Members |
| 1389 | 1405 | ||
| 1390 | int IBootstrapperApplication.BAProc(int message, IntPtr pvArgs, IntPtr pvResults, IntPtr pvContext) | 1406 | int IBootstrapperApplication.BAProc(int message, IntPtr pvArgs, IntPtr pvResults, IntPtr pvContext) |
| @@ -2144,6 +2160,15 @@ namespace WixToolset.Mba.Core | |||
| 2144 | return args.HResult; | 2160 | return args.HResult; |
| 2145 | } | 2161 | } |
| 2146 | 2162 | ||
| 2163 | int IBootstrapperApplication.OnDetectRelatedBundlePackage(string wzPackageId, string wzProductCode, RelationType relationType, bool fPerMachine, string wzVersion, ref bool fCancel) | ||
| 2164 | { | ||
| 2165 | DetectRelatedBundlePackageEventArgs args = new DetectRelatedBundlePackageEventArgs(wzPackageId, wzProductCode, relationType, fPerMachine, wzVersion, fCancel); | ||
| 2166 | this.OnDetectRelatedBundlePackage(args); | ||
| 2167 | |||
| 2168 | fCancel = args.Cancel; | ||
| 2169 | return args.HResult; | ||
| 2170 | } | ||
| 2171 | |||
| 2147 | #endregion | 2172 | #endregion |
| 2148 | } | 2173 | } |
| 2149 | } | 2174 | } |
diff --git a/src/api/burn/WixToolset.Mba.Core/EventArgs.cs b/src/api/burn/WixToolset.Mba.Core/EventArgs.cs index c2c73067..07b14b3d 100644 --- a/src/api/burn/WixToolset.Mba.Core/EventArgs.cs +++ b/src/api/burn/WixToolset.Mba.Core/EventArgs.cs | |||
| @@ -2030,10 +2030,7 @@ namespace WixToolset.Mba.Core | |||
| 2030 | [Serializable] | 2030 | [Serializable] |
| 2031 | public class LaunchApprovedExeBeginEventArgs : CancellableHResultEventArgs | 2031 | public class LaunchApprovedExeBeginEventArgs : CancellableHResultEventArgs |
| 2032 | { | 2032 | { |
| 2033 | /// <summary> | 2033 | /// <summary /> |
| 2034 | /// | ||
| 2035 | /// </summary> | ||
| 2036 | /// <param name="cancelRecommendation"></param> | ||
| 2037 | public LaunchApprovedExeBeginEventArgs(bool cancelRecommendation) | 2034 | public LaunchApprovedExeBeginEventArgs(bool cancelRecommendation) |
| 2038 | : base(cancelRecommendation) | 2035 | : base(cancelRecommendation) |
| 2039 | { | 2036 | { |
| @@ -2046,27 +2043,18 @@ namespace WixToolset.Mba.Core | |||
| 2046 | [Serializable] | 2043 | [Serializable] |
| 2047 | public class LaunchApprovedExeCompleteEventArgs : StatusEventArgs | 2044 | public class LaunchApprovedExeCompleteEventArgs : StatusEventArgs |
| 2048 | { | 2045 | { |
| 2049 | private int processId; | 2046 | /// <summary /> |
| 2050 | |||
| 2051 | /// <summary> | ||
| 2052 | /// | ||
| 2053 | /// </summary> | ||
| 2054 | /// <param name="hrStatus"></param> | ||
| 2055 | /// <param name="processId"></param> | ||
| 2056 | public LaunchApprovedExeCompleteEventArgs(int hrStatus, int processId) | 2047 | public LaunchApprovedExeCompleteEventArgs(int hrStatus, int processId) |
| 2057 | : base(hrStatus) | 2048 | : base(hrStatus) |
| 2058 | { | 2049 | { |
| 2059 | this.processId = processId; | 2050 | this.ProcessId = processId; |
| 2060 | } | 2051 | } |
| 2061 | 2052 | ||
| 2062 | /// <summary> | 2053 | /// <summary> |
| 2063 | /// Gets the ProcessId of the process that was launched. | 2054 | /// Gets the ProcessId of the process that was launched. |
| 2064 | /// This is only valid if the status reports success. | 2055 | /// This is only valid if the status reports success. |
| 2065 | /// </summary> | 2056 | /// </summary> |
| 2066 | public int ProcessId | 2057 | public int ProcessId { get; private set; } |
| 2067 | { | ||
| 2068 | get { return this.processId; } | ||
| 2069 | } | ||
| 2070 | } | 2058 | } |
| 2071 | 2059 | ||
| 2072 | /// <summary> | 2060 | /// <summary> |
| @@ -2075,26 +2063,17 @@ namespace WixToolset.Mba.Core | |||
| 2075 | [Serializable] | 2063 | [Serializable] |
| 2076 | public class BeginMsiTransactionBeginEventArgs : CancellableHResultEventArgs | 2064 | public class BeginMsiTransactionBeginEventArgs : CancellableHResultEventArgs |
| 2077 | { | 2065 | { |
| 2078 | private string transactionId; | 2066 | /// <summary /> |
| 2079 | |||
| 2080 | /// <summary> | ||
| 2081 | /// | ||
| 2082 | /// </summary> | ||
| 2083 | /// <param name="transactionId"></param> | ||
| 2084 | /// <param name="cancelRecommendation"></param> | ||
| 2085 | public BeginMsiTransactionBeginEventArgs(string transactionId, bool cancelRecommendation) | 2067 | public BeginMsiTransactionBeginEventArgs(string transactionId, bool cancelRecommendation) |
| 2086 | : base(cancelRecommendation) | 2068 | : base(cancelRecommendation) |
| 2087 | { | 2069 | { |
| 2088 | this.transactionId = transactionId; | 2070 | this.TransactionId = transactionId; |
| 2089 | } | 2071 | } |
| 2090 | 2072 | ||
| 2091 | /// <summary> | 2073 | /// <summary> |
| 2092 | /// Gets the MSI transaction Id. | 2074 | /// Gets the MSI transaction Id. |
| 2093 | /// </summary> | 2075 | /// </summary> |
| 2094 | public string TransactionId | 2076 | public string TransactionId { get; private set; } |
| 2095 | { | ||
| 2096 | get { return this.transactionId; } | ||
| 2097 | } | ||
| 2098 | } | 2077 | } |
| 2099 | 2078 | ||
| 2100 | /// <summary> | 2079 | /// <summary> |
| @@ -2103,26 +2082,17 @@ namespace WixToolset.Mba.Core | |||
| 2103 | [Serializable] | 2082 | [Serializable] |
| 2104 | public class BeginMsiTransactionCompleteEventArgs : StatusEventArgs | 2083 | public class BeginMsiTransactionCompleteEventArgs : StatusEventArgs |
| 2105 | { | 2084 | { |
| 2106 | private string transactionId; | 2085 | /// <summary /> |
| 2107 | |||
| 2108 | /// <summary> | ||
| 2109 | /// | ||
| 2110 | /// </summary> | ||
| 2111 | /// <param name="transactionId"></param> | ||
| 2112 | /// <param name="hrStatus"></param> | ||
| 2113 | public BeginMsiTransactionCompleteEventArgs(string transactionId, int hrStatus) | 2086 | public BeginMsiTransactionCompleteEventArgs(string transactionId, int hrStatus) |
| 2114 | : base(hrStatus) | 2087 | : base(hrStatus) |
| 2115 | { | 2088 | { |
| 2116 | this.transactionId = transactionId; | 2089 | this.TransactionId = transactionId; |
| 2117 | } | 2090 | } |
| 2118 | 2091 | ||
| 2119 | /// <summary> | 2092 | /// <summary> |
| 2120 | /// Gets the MSI transaction Id. | 2093 | /// Gets the MSI transaction Id. |
| 2121 | /// </summary> | 2094 | /// </summary> |
| 2122 | public string TransactionId | 2095 | public string TransactionId { get; private set; } |
| 2123 | { | ||
| 2124 | get { return this.transactionId; } | ||
| 2125 | } | ||
| 2126 | } | 2096 | } |
| 2127 | 2097 | ||
| 2128 | /// <summary> | 2098 | /// <summary> |
| @@ -2131,26 +2101,17 @@ namespace WixToolset.Mba.Core | |||
| 2131 | [Serializable] | 2101 | [Serializable] |
| 2132 | public class CommitMsiTransactionBeginEventArgs : CancellableHResultEventArgs | 2102 | public class CommitMsiTransactionBeginEventArgs : CancellableHResultEventArgs |
| 2133 | { | 2103 | { |
| 2134 | private string transactionId; | 2104 | /// <summary /> |
| 2135 | |||
| 2136 | /// <summary> | ||
| 2137 | /// | ||
| 2138 | /// </summary> | ||
| 2139 | /// <param name="transactionId"></param> | ||
| 2140 | /// <param name="cancelRecommendation"></param> | ||
| 2141 | public CommitMsiTransactionBeginEventArgs(string transactionId, bool cancelRecommendation) | 2105 | public CommitMsiTransactionBeginEventArgs(string transactionId, bool cancelRecommendation) |
| 2142 | : base(cancelRecommendation) | 2106 | : base(cancelRecommendation) |
| 2143 | { | 2107 | { |
| 2144 | this.transactionId = transactionId; | 2108 | this.TransactionId = transactionId; |
| 2145 | } | 2109 | } |
| 2146 | 2110 | ||
| 2147 | /// <summary> | 2111 | /// <summary> |
| 2148 | /// Gets the MSI transaction Id. | 2112 | /// Gets the MSI transaction Id. |
| 2149 | /// </summary> | 2113 | /// </summary> |
| 2150 | public string TransactionId | 2114 | public string TransactionId { get; private set; } |
| 2151 | { | ||
| 2152 | get { return this.transactionId; } | ||
| 2153 | } | ||
| 2154 | } | 2115 | } |
| 2155 | 2116 | ||
| 2156 | /// <summary> | 2117 | /// <summary> |
| @@ -2159,26 +2120,17 @@ namespace WixToolset.Mba.Core | |||
| 2159 | [Serializable] | 2120 | [Serializable] |
| 2160 | public class CommitMsiTransactionCompleteEventArgs : StatusEventArgs | 2121 | public class CommitMsiTransactionCompleteEventArgs : StatusEventArgs |
| 2161 | { | 2122 | { |
| 2162 | private string transactionId; | 2123 | /// <summary /> |
| 2163 | |||
| 2164 | /// <summary> | ||
| 2165 | /// | ||
| 2166 | /// </summary> | ||
| 2167 | /// <param name="transactionId"></param> | ||
| 2168 | /// <param name="hrStatus"></param> | ||
| 2169 | public CommitMsiTransactionCompleteEventArgs(string transactionId, int hrStatus) | 2124 | public CommitMsiTransactionCompleteEventArgs(string transactionId, int hrStatus) |
| 2170 | : base(hrStatus) | 2125 | : base(hrStatus) |
| 2171 | { | 2126 | { |
| 2172 | this.transactionId = transactionId; | 2127 | this.TransactionId = transactionId; |
| 2173 | } | 2128 | } |
| 2174 | 2129 | ||
| 2175 | /// <summary> | 2130 | /// <summary> |
| 2176 | /// Gets the MSI transaction Id. | 2131 | /// Gets the MSI transaction Id. |
| 2177 | /// </summary> | 2132 | /// </summary> |
| 2178 | public string TransactionId | 2133 | public string TransactionId { get; private set; } |
| 2179 | { | ||
| 2180 | get { return this.transactionId; } | ||
| 2181 | } | ||
| 2182 | } | 2134 | } |
| 2183 | 2135 | ||
| 2184 | /// <summary> | 2136 | /// <summary> |
| @@ -2187,24 +2139,16 @@ namespace WixToolset.Mba.Core | |||
| 2187 | [Serializable] | 2139 | [Serializable] |
| 2188 | public class RollbackMsiTransactionBeginEventArgs : HResultEventArgs | 2140 | public class RollbackMsiTransactionBeginEventArgs : HResultEventArgs |
| 2189 | { | 2141 | { |
| 2190 | private string transactionId; | 2142 | /// <summary /> |
| 2191 | |||
| 2192 | /// <summary> | ||
| 2193 | /// | ||
| 2194 | /// </summary> | ||
| 2195 | /// <param name="transactionId"></param> | ||
| 2196 | public RollbackMsiTransactionBeginEventArgs(string transactionId) | 2143 | public RollbackMsiTransactionBeginEventArgs(string transactionId) |
| 2197 | { | 2144 | { |
| 2198 | this.transactionId = transactionId; | 2145 | this.TransactionId = transactionId; |
| 2199 | } | 2146 | } |
| 2200 | 2147 | ||
| 2201 | /// <summary> | 2148 | /// <summary> |
| 2202 | /// Gets the MSI transaction Id. | 2149 | /// Gets the MSI transaction Id. |
| 2203 | /// </summary> | 2150 | /// </summary> |
| 2204 | public string TransactionId | 2151 | public string TransactionId { get; private set; } |
| 2205 | { | ||
| 2206 | get { return this.transactionId; } | ||
| 2207 | } | ||
| 2208 | } | 2152 | } |
| 2209 | 2153 | ||
| 2210 | /// <summary> | 2154 | /// <summary> |
| @@ -2213,26 +2157,17 @@ namespace WixToolset.Mba.Core | |||
| 2213 | [Serializable] | 2157 | [Serializable] |
| 2214 | public class RollbackMsiTransactionCompleteEventArgs : StatusEventArgs | 2158 | public class RollbackMsiTransactionCompleteEventArgs : StatusEventArgs |
| 2215 | { | 2159 | { |
| 2216 | private string transactionId; | 2160 | /// <summary /> |
| 2217 | |||
| 2218 | /// <summary> | ||
| 2219 | /// | ||
| 2220 | /// </summary> | ||
| 2221 | /// <param name="transactionId"></param> | ||
| 2222 | /// <param name="hrStatus"></param> | ||
| 2223 | public RollbackMsiTransactionCompleteEventArgs(string transactionId, int hrStatus) | 2161 | public RollbackMsiTransactionCompleteEventArgs(string transactionId, int hrStatus) |
| 2224 | : base(hrStatus) | 2162 | : base(hrStatus) |
| 2225 | { | 2163 | { |
| 2226 | this.transactionId = transactionId; | 2164 | this.TransactionId = transactionId; |
| 2227 | } | 2165 | } |
| 2228 | 2166 | ||
| 2229 | /// <summary> | 2167 | /// <summary> |
| 2230 | /// Gets the MSI transaction Id. | 2168 | /// Gets the MSI transaction Id. |
| 2231 | /// </summary> | 2169 | /// </summary> |
| 2232 | public string TransactionId | 2170 | public string TransactionId { get; private set; } |
| 2233 | { | ||
| 2234 | get { return this.transactionId; } | ||
| 2235 | } | ||
| 2236 | } | 2171 | } |
| 2237 | 2172 | ||
| 2238 | /// <summary> | 2173 | /// <summary> |
| @@ -2524,4 +2459,47 @@ namespace WixToolset.Mba.Core | |||
| 2524 | /// </summary> | 2459 | /// </summary> |
| 2525 | public BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION Action { get; set; } | 2460 | public BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION Action { get; set; } |
| 2526 | } | 2461 | } |
| 2462 | |||
| 2463 | /// <summary> | ||
| 2464 | /// Event arguments for <see cref="IDefaultBootstrapperApplication.DetectRelatedBundlePackage"/> | ||
| 2465 | /// </summary> | ||
| 2466 | [Serializable] | ||
| 2467 | public class DetectRelatedBundlePackageEventArgs : CancellableHResultEventArgs | ||
| 2468 | { | ||
| 2469 | /// <summary /> | ||
| 2470 | public DetectRelatedBundlePackageEventArgs(string packageId, string productCode, RelationType relationType, bool perMachine, string version, bool cancelRecommendation) | ||
| 2471 | : base(cancelRecommendation) | ||
| 2472 | { | ||
| 2473 | this.PackageId = packageId; | ||
| 2474 | this.ProductCode = productCode; | ||
| 2475 | this.RelationType = relationType; | ||
| 2476 | this.PerMachine = perMachine; | ||
| 2477 | this.Version = version; | ||
| 2478 | } | ||
| 2479 | |||
| 2480 | /// <summary> | ||
| 2481 | /// Gets the identity of the product's package detected. | ||
| 2482 | /// </summary> | ||
| 2483 | public string PackageId { get; private set; } | ||
| 2484 | |||
| 2485 | /// <summary> | ||
| 2486 | /// Gets the identity of the related bundle detected. | ||
| 2487 | /// </summary> | ||
| 2488 | public string ProductCode { get; private set; } | ||
| 2489 | |||
| 2490 | /// <summary> | ||
| 2491 | /// Gets the relationship type of the related bundle. | ||
| 2492 | /// </summary> | ||
| 2493 | public RelationType RelationType { get; private set; } | ||
| 2494 | |||
| 2495 | /// <summary> | ||
| 2496 | /// Gets whether the detected bundle is per machine. | ||
| 2497 | /// </summary> | ||
| 2498 | public bool PerMachine { get; private set; } | ||
| 2499 | |||
| 2500 | /// <summary> | ||
| 2501 | /// Gets the version of the related bundle detected. | ||
| 2502 | /// </summary> | ||
| 2503 | public string Version { get; private set; } | ||
| 2504 | } | ||
| 2527 | } | 2505 | } |
diff --git a/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplication.cs b/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplication.cs index 7cf0957a..ae642474 100644 --- a/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplication.cs +++ b/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplication.cs | |||
| @@ -1182,6 +1182,20 @@ namespace WixToolset.Mba.Core | |||
| 1182 | [MarshalAs(UnmanagedType.I4)] BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION recommendation, | 1182 | [MarshalAs(UnmanagedType.I4)] BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION recommendation, |
| 1183 | [MarshalAs(UnmanagedType.I4)] ref BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION pAction | 1183 | [MarshalAs(UnmanagedType.I4)] ref BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION pAction |
| 1184 | ); | 1184 | ); |
| 1185 | |||
| 1186 | /// <summary> | ||
| 1187 | /// See <see cref="IDefaultBootstrapperApplication.DetectRelatedBundlePackage"/>. | ||
| 1188 | /// </summary> | ||
| 1189 | [PreserveSig] | ||
| 1190 | [return: MarshalAs(UnmanagedType.I4)] | ||
| 1191 | int OnDetectRelatedBundlePackage( | ||
| 1192 | [MarshalAs(UnmanagedType.LPWStr)] string wzPackageId, | ||
| 1193 | [MarshalAs(UnmanagedType.LPWStr)] string wzBundleId, | ||
| 1194 | [MarshalAs(UnmanagedType.U4)] RelationType relationType, | ||
| 1195 | [MarshalAs(UnmanagedType.Bool)] bool fPerMachine, | ||
| 1196 | [MarshalAs(UnmanagedType.LPWStr)] string wzVersion, | ||
| 1197 | [MarshalAs(UnmanagedType.Bool)] ref bool fCancel | ||
| 1198 | ); | ||
| 1185 | } | 1199 | } |
| 1186 | 1200 | ||
| 1187 | /// <summary> | 1201 | /// <summary> |
diff --git a/src/api/burn/WixToolset.Mba.Core/IDefaultBootstrapperApplication.cs b/src/api/burn/WixToolset.Mba.Core/IDefaultBootstrapperApplication.cs index 21d99b32..77089e83 100644 --- a/src/api/burn/WixToolset.Mba.Core/IDefaultBootstrapperApplication.cs +++ b/src/api/burn/WixToolset.Mba.Core/IDefaultBootstrapperApplication.cs | |||
| @@ -36,15 +36,15 @@ namespace WixToolset.Mba.Core | |||
| 36 | 36 | ||
| 37 | /// <summary> | 37 | /// <summary> |
| 38 | /// Fired when the engine has begun acquiring the payload or container. | 38 | /// Fired when the engine has begun acquiring the payload or container. |
| 39 | /// The BA can change the source using <see cref="IEngine.SetLocalSource(string, string, string)"/> | 39 | /// The BA can change the source using <see cref="IEngine.SetLocalSource(String, String, String)"/> |
| 40 | /// or <see cref="IEngine.SetDownloadSource(string, string, string, string, string)"/>. | 40 | /// or <see cref="IEngine.SetDownloadSource(String, String, String, String, String)"/>. |
| 41 | /// </summary> | 41 | /// </summary> |
| 42 | event EventHandler<CacheAcquireBeginEventArgs> CacheAcquireBegin; | 42 | event EventHandler<CacheAcquireBeginEventArgs> CacheAcquireBegin; |
| 43 | 43 | ||
| 44 | /// <summary> | 44 | /// <summary> |
| 45 | /// Fired when the engine has completed the acquisition of the payload or container. | 45 | /// Fired when the engine has completed the acquisition of the payload or container. |
| 46 | /// The BA can change the source using <see cref="IEngine.SetLocalSource(string, string, string)"/> | 46 | /// The BA can change the source using <see cref="IEngine.SetLocalSource(String, String, String)"/> |
| 47 | /// or <see cref="IEngine.SetDownloadSource(string, string, string, string, string)"/>. | 47 | /// or <see cref="IEngine.SetDownloadSource(String, String, String, String, String)"/>. |
| 48 | /// </summary> | 48 | /// </summary> |
| 49 | event EventHandler<CacheAcquireCompleteEventArgs> CacheAcquireComplete; | 49 | event EventHandler<CacheAcquireCompleteEventArgs> CacheAcquireComplete; |
| 50 | 50 | ||
| @@ -179,6 +179,11 @@ namespace WixToolset.Mba.Core | |||
| 179 | event EventHandler<DetectRelatedBundleEventArgs> DetectRelatedBundle; | 179 | event EventHandler<DetectRelatedBundleEventArgs> DetectRelatedBundle; |
| 180 | 180 | ||
| 181 | /// <summary> | 181 | /// <summary> |
| 182 | /// Fired when a related bundle has been detected for a bundle package. | ||
| 183 | /// </summary> | ||
| 184 | event EventHandler<DetectRelatedBundlePackageEventArgs> DetectRelatedBundlePackage; | ||
| 185 | |||
| 186 | /// <summary> | ||
| 182 | /// Fired when a related MSI package has been detected for a package. | 187 | /// Fired when a related MSI package has been detected for a package. |
| 183 | /// </summary> | 188 | /// </summary> |
| 184 | event EventHandler<DetectRelatedMsiPackageEventArgs> DetectRelatedMsiPackage; | 189 | event EventHandler<DetectRelatedMsiPackageEventArgs> DetectRelatedMsiPackage; |
diff --git a/src/api/burn/WixToolset.Mba.Core/PackageInfo.cs b/src/api/burn/WixToolset.Mba.Core/PackageInfo.cs index 3681a497..39fe4d73 100644 --- a/src/api/burn/WixToolset.Mba.Core/PackageInfo.cs +++ b/src/api/burn/WixToolset.Mba.Core/PackageInfo.cs | |||
| @@ -56,6 +56,11 @@ namespace WixToolset.Mba.Core | |||
| 56 | /// | 56 | /// |
| 57 | /// </summary> | 57 | /// </summary> |
| 58 | UpdateBundle, | 58 | UpdateBundle, |
| 59 | |||
| 60 | /// <summary> | ||
| 61 | /// | ||
| 62 | /// </summary> | ||
| 63 | ChainBundle, | ||
| 59 | } | 64 | } |
| 60 | 65 | ||
| 61 | /// <summary> | 66 | /// <summary> |
| @@ -220,7 +225,11 @@ namespace WixToolset.Mba.Core | |||
| 220 | return null; | 225 | return null; |
| 221 | } | 226 | } |
| 222 | 227 | ||
| 223 | if (attributeValue.Equals("Exe", StringComparison.InvariantCulture)) | 228 | if (attributeValue.Equals("Bundle", StringComparison.InvariantCulture)) |
| 229 | { | ||
| 230 | return PackageType.ChainBundle; | ||
| 231 | } | ||
| 232 | else if (attributeValue.Equals("Exe", StringComparison.InvariantCulture)) | ||
| 224 | { | 233 | { |
| 225 | return PackageType.Exe; | 234 | return PackageType.Exe; |
| 226 | } | 235 | } |
| @@ -268,7 +277,7 @@ namespace WixToolset.Mba.Core | |||
| 268 | package.Type = PackageType.UpgradeBundle; | 277 | package.Type = PackageType.UpgradeBundle; |
| 269 | break; | 278 | break; |
| 270 | default: | 279 | default: |
| 271 | throw new Exception(string.Format("Unknown related bundle type: {0}", relationType)); | 280 | throw new Exception(String.Format("Unknown related bundle type: {0}", relationType)); |
| 272 | } | 281 | } |
| 273 | 282 | ||
| 274 | return package; | 283 | return package; |
| @@ -302,7 +311,7 @@ namespace WixToolset.Mba.Core | |||
| 302 | 311 | ||
| 303 | if (!packagesById.TryGetValue(id, out var ipackage)) | 312 | if (!packagesById.TryGetValue(id, out var ipackage)) |
| 304 | { | 313 | { |
| 305 | throw new Exception(string.Format("Failed to find package specified in WixBalPackageInfo: {0}", id)); | 314 | throw new Exception(String.Format("Failed to find package specified in WixBalPackageInfo: {0}", id)); |
| 306 | } | 315 | } |
| 307 | 316 | ||
| 308 | var package = (PackageInfo)ipackage; | 317 | var package = (PackageInfo)ipackage; |
| @@ -322,7 +331,7 @@ namespace WixToolset.Mba.Core | |||
| 322 | 331 | ||
| 323 | if (!packagesById.TryGetValue(id, out var ipackage)) | 332 | if (!packagesById.TryGetValue(id, out var ipackage)) |
| 324 | { | 333 | { |
| 325 | throw new Exception(string.Format("Failed to find package specified in WixMbaPrereqInformation: {0}", id)); | 334 | throw new Exception(String.Format("Failed to find package specified in WixMbaPrereqInformation: {0}", id)); |
| 326 | } | 335 | } |
| 327 | 336 | ||
| 328 | var package = (PackageInfo)ipackage; | 337 | var package = (PackageInfo)ipackage; |
diff --git a/src/api/burn/balutil/balinfo.cpp b/src/api/burn/balutil/balinfo.cpp index d9cc9b76..f0eb9904 100644 --- a/src/api/burn/balutil/balinfo.cpp +++ b/src/api/burn/balutil/balinfo.cpp | |||
| @@ -433,7 +433,11 @@ static HRESULT ParsePackagesFromXml( | |||
| 433 | hr = XmlGetAttributeEx(pNode, L"PackageType", &scz); | 433 | hr = XmlGetAttributeEx(pNode, L"PackageType", &scz); |
| 434 | ExitOnFailure(hr, "Failed to get package type for package."); | 434 | ExitOnFailure(hr, "Failed to get package type for package."); |
| 435 | 435 | ||
| 436 | if (CSTR_EQUAL == ::CompareStringW(LOCALE_NEUTRAL, 0, L"Exe", -1, scz, -1)) | 436 | if (CSTR_EQUAL == ::CompareStringW(LOCALE_NEUTRAL, 0, L"Bundle", -1, scz, -1)) |
| 437 | { | ||
| 438 | prgPackages[iPackage].type = BAL_INFO_PACKAGE_TYPE_BUNDLE_CHAIN; | ||
| 439 | } | ||
| 440 | else if (CSTR_EQUAL == ::CompareStringW(LOCALE_NEUTRAL, 0, L"Exe", -1, scz, -1)) | ||
| 437 | { | 441 | { |
| 438 | prgPackages[iPackage].type = BAL_INFO_PACKAGE_TYPE_EXE; | 442 | prgPackages[iPackage].type = BAL_INFO_PACKAGE_TYPE_EXE; |
| 439 | } | 443 | } |
diff --git a/src/api/burn/balutil/inc/BAFunctions.h b/src/api/burn/balutil/inc/BAFunctions.h index 158e65b5..9be3f62f 100644 --- a/src/api/burn/balutil/inc/BAFunctions.h +++ b/src/api/burn/balutil/inc/BAFunctions.h | |||
| @@ -92,6 +92,7 @@ enum BA_FUNCTIONS_MESSAGE | |||
| 92 | BA_FUNCTIONS_MESSAGE_ONPLANRELATEDBUNDLETYPE = BOOTSTRAPPER_APPLICATION_MESSAGE_ONPLANRELATEDBUNDLETYPE, | 92 | BA_FUNCTIONS_MESSAGE_ONPLANRELATEDBUNDLETYPE = BOOTSTRAPPER_APPLICATION_MESSAGE_ONPLANRELATEDBUNDLETYPE, |
| 93 | BA_FUNCTIONS_MESSAGE_ONAPPLYDOWNGRADE = BOOTSTRAPPER_APPLICATION_MESSAGE_ONAPPLYDOWNGRADE, | 93 | BA_FUNCTIONS_MESSAGE_ONAPPLYDOWNGRADE = BOOTSTRAPPER_APPLICATION_MESSAGE_ONAPPLYDOWNGRADE, |
| 94 | BA_FUNCTIONS_MESSAGE_ONEXECUTEPROCESSCANCEL = BOOTSTRAPPER_APPLICATION_MESSAGE_ONEXECUTEPROCESSCANCEL, | 94 | BA_FUNCTIONS_MESSAGE_ONEXECUTEPROCESSCANCEL = BOOTSTRAPPER_APPLICATION_MESSAGE_ONEXECUTEPROCESSCANCEL, |
| 95 | BA_FUNCTIONS_MESSAGE_ONDETECTRELATEDBUNDLEPACKAGE = BOOTSTRAPPER_APPLICATION_MESSAGE_ONDETECTRELATEDBUNDLEPACKAGE, | ||
| 95 | 96 | ||
| 96 | BA_FUNCTIONS_MESSAGE_ONTHEMELOADED = 1024, | 97 | BA_FUNCTIONS_MESSAGE_ONTHEMELOADED = 1024, |
| 97 | BA_FUNCTIONS_MESSAGE_WNDPROC, | 98 | BA_FUNCTIONS_MESSAGE_WNDPROC, |
diff --git a/src/api/burn/balutil/inc/BalBaseBAFunctions.h b/src/api/burn/balutil/inc/BalBaseBAFunctions.h index 614d4bcf..6bde05d2 100644 --- a/src/api/burn/balutil/inc/BalBaseBAFunctions.h +++ b/src/api/burn/balutil/inc/BalBaseBAFunctions.h | |||
| @@ -887,6 +887,18 @@ public: // IBootstrapperApplication | |||
| 887 | return S_OK; | 887 | return S_OK; |
| 888 | } | 888 | } |
| 889 | 889 | ||
| 890 | virtual STDMETHODIMP OnDetectRelatedBundlePackage( | ||
| 891 | __in_z LPCWSTR /*wzPackageId*/, | ||
| 892 | __in_z LPCWSTR /*wzBundleId*/, | ||
| 893 | __in BOOTSTRAPPER_RELATION_TYPE /*relationType*/, | ||
| 894 | __in BOOL /*fPerMachine*/, | ||
| 895 | __in LPCWSTR /*wzVersion*/, | ||
| 896 | __inout BOOL* /*pfCancel*/ | ||
| 897 | ) | ||
| 898 | { | ||
| 899 | return S_OK; | ||
| 900 | } | ||
| 901 | |||
| 890 | public: // IBAFunctions | 902 | public: // IBAFunctions |
| 891 | virtual STDMETHODIMP OnPlan( | 903 | virtual STDMETHODIMP OnPlan( |
| 892 | ) | 904 | ) |
diff --git a/src/api/burn/balutil/inc/BalBaseBAFunctionsProc.h b/src/api/burn/balutil/inc/BalBaseBAFunctionsProc.h index b96a180c..4564ad0c 100644 --- a/src/api/burn/balutil/inc/BalBaseBAFunctionsProc.h +++ b/src/api/burn/balutil/inc/BalBaseBAFunctionsProc.h | |||
| @@ -163,6 +163,7 @@ static HRESULT WINAPI BalBaseBAFunctionsProc( | |||
| 163 | case BA_FUNCTIONS_MESSAGE_ONPLANRELATEDBUNDLETYPE: | 163 | case BA_FUNCTIONS_MESSAGE_ONPLANRELATEDBUNDLETYPE: |
| 164 | case BA_FUNCTIONS_MESSAGE_ONAPPLYDOWNGRADE: | 164 | case BA_FUNCTIONS_MESSAGE_ONAPPLYDOWNGRADE: |
| 165 | case BA_FUNCTIONS_MESSAGE_ONEXECUTEPROCESSCANCEL: | 165 | case BA_FUNCTIONS_MESSAGE_ONEXECUTEPROCESSCANCEL: |
| 166 | case BA_FUNCTIONS_MESSAGE_ONDETECTRELATEDBUNDLEPACKAGE: | ||
| 166 | hr = BalBaseBootstrapperApplicationProc((BOOTSTRAPPER_APPLICATION_MESSAGE)message, pvArgs, pvResults, pvContext); | 167 | hr = BalBaseBootstrapperApplicationProc((BOOTSTRAPPER_APPLICATION_MESSAGE)message, pvArgs, pvResults, pvContext); |
| 167 | break; | 168 | break; |
| 168 | case BA_FUNCTIONS_MESSAGE_ONTHEMELOADED: | 169 | case BA_FUNCTIONS_MESSAGE_ONTHEMELOADED: |
diff --git a/src/api/burn/balutil/inc/BalBaseBootstrapperApplication.h b/src/api/burn/balutil/inc/BalBaseBootstrapperApplication.h index 25570ffd..b661c7c9 100644 --- a/src/api/burn/balutil/inc/BalBaseBootstrapperApplication.h +++ b/src/api/burn/balutil/inc/BalBaseBootstrapperApplication.h | |||
| @@ -1087,6 +1087,19 @@ public: // IBootstrapperApplication | |||
| 1087 | return S_OK; | 1087 | return S_OK; |
| 1088 | } | 1088 | } |
| 1089 | 1089 | ||
| 1090 | virtual STDMETHODIMP OnDetectRelatedBundlePackage( | ||
| 1091 | __in_z LPCWSTR /*wzPackageId*/, | ||
| 1092 | __in_z LPCWSTR /*wzBundleId*/, | ||
| 1093 | __in BOOTSTRAPPER_RELATION_TYPE /*relationType*/, | ||
| 1094 | __in BOOL /*fPerMachine*/, | ||
| 1095 | __in LPCWSTR /*wzVersion*/, | ||
| 1096 | __inout BOOL* pfCancel | ||
| 1097 | ) | ||
| 1098 | { | ||
| 1099 | *pfCancel |= CheckCanceled(); | ||
| 1100 | return S_OK; | ||
| 1101 | } | ||
| 1102 | |||
| 1090 | public: //CBalBaseBootstrapperApplication | 1103 | public: //CBalBaseBootstrapperApplication |
| 1091 | virtual STDMETHODIMP Initialize( | 1104 | virtual STDMETHODIMP Initialize( |
| 1092 | __in const BOOTSTRAPPER_CREATE_ARGS* pCreateArgs | 1105 | __in const BOOTSTRAPPER_CREATE_ARGS* pCreateArgs |
diff --git a/src/api/burn/balutil/inc/BalBaseBootstrapperApplicationProc.h b/src/api/burn/balutil/inc/BalBaseBootstrapperApplicationProc.h index b196d183..4ef7bac5 100644 --- a/src/api/burn/balutil/inc/BalBaseBootstrapperApplicationProc.h +++ b/src/api/burn/balutil/inc/BalBaseBootstrapperApplicationProc.h | |||
| @@ -756,6 +756,15 @@ static HRESULT BalBaseBAProcOnApplyDowngrade( | |||
| 756 | return pBA->OnApplyDowngrade(pArgs->hrRecommended, &pResults->hrStatus); | 756 | return pBA->OnApplyDowngrade(pArgs->hrRecommended, &pResults->hrStatus); |
| 757 | } | 757 | } |
| 758 | 758 | ||
| 759 | static HRESULT BalBaseBAProcOnDetectRelatedBundlePackage( | ||
| 760 | __in IBootstrapperApplication* pBA, | ||
| 761 | __in BA_ONDETECTRELATEDBUNDLEPACKAGE_ARGS* pArgs, | ||
| 762 | __inout BA_ONDETECTRELATEDBUNDLEPACKAGE_RESULTS* pResults | ||
| 763 | ) | ||
| 764 | { | ||
| 765 | return pBA->OnDetectRelatedBundlePackage(pArgs->wzPackageId, pArgs->wzBundleId, pArgs->relationType, pArgs->fPerMachine, pArgs->wzVersion, &pResults->fCancel); | ||
| 766 | } | ||
| 767 | |||
| 759 | /******************************************************************* | 768 | /******************************************************************* |
| 760 | BalBaseBootstrapperApplicationProc - requires pvContext to be of type IBootstrapperApplication. | 769 | BalBaseBootstrapperApplicationProc - requires pvContext to be of type IBootstrapperApplication. |
| 761 | Provides a default mapping between the new message based BA interface and | 770 | Provides a default mapping between the new message based BA interface and |
| @@ -1024,6 +1033,9 @@ static HRESULT WINAPI BalBaseBootstrapperApplicationProc( | |||
| 1024 | case BOOTSTRAPPER_APPLICATION_MESSAGE_ONEXECUTEPROCESSCANCEL: | 1033 | case BOOTSTRAPPER_APPLICATION_MESSAGE_ONEXECUTEPROCESSCANCEL: |
| 1025 | hr = BalBaseBAProcOnExecuteProcessCancel(pBA, reinterpret_cast<BA_ONEXECUTEPROCESSCANCEL_ARGS*>(pvArgs), reinterpret_cast<BA_ONEXECUTEPROCESSCANCEL_RESULTS*>(pvResults)); | 1034 | hr = BalBaseBAProcOnExecuteProcessCancel(pBA, reinterpret_cast<BA_ONEXECUTEPROCESSCANCEL_ARGS*>(pvArgs), reinterpret_cast<BA_ONEXECUTEPROCESSCANCEL_RESULTS*>(pvResults)); |
| 1026 | break; | 1035 | break; |
| 1036 | case BOOTSTRAPPER_APPLICATION_MESSAGE_ONDETECTRELATEDBUNDLEPACKAGE: | ||
| 1037 | hr = BalBaseBAProcOnDetectRelatedBundlePackage(pBA, reinterpret_cast<BA_ONDETECTRELATEDBUNDLEPACKAGE_ARGS*>(pvArgs), reinterpret_cast<BA_ONDETECTRELATEDBUNDLEPACKAGE_RESULTS*>(pvResults)); | ||
| 1038 | break; | ||
| 1027 | } | 1039 | } |
| 1028 | } | 1040 | } |
| 1029 | 1041 | ||
diff --git a/src/api/burn/balutil/inc/IBootstrapperApplication.h b/src/api/burn/balutil/inc/IBootstrapperApplication.h index 6174c290..a4840228 100644 --- a/src/api/burn/balutil/inc/IBootstrapperApplication.h +++ b/src/api/burn/balutil/inc/IBootstrapperApplication.h | |||
| @@ -723,4 +723,14 @@ DECLARE_INTERFACE_IID_(IBootstrapperApplication, IUnknown, "53C31D56-49C0-426B-A | |||
| 723 | __in BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION recommendation, | 723 | __in BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION recommendation, |
| 724 | __inout BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION* pAction | 724 | __inout BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION* pAction |
| 725 | ) = 0; | 725 | ) = 0; |
| 726 | |||
| 727 | // OnDetectRelatedBundlePackage - called when the engine detects a related bundle for a BundlePackage. | ||
| 728 | STDMETHOD(OnDetectRelatedBundlePackage)( | ||
| 729 | __in_z LPCWSTR wzPackageId, | ||
| 730 | __in_z LPCWSTR wzBundleId, | ||
| 731 | __in BOOTSTRAPPER_RELATION_TYPE relationType, | ||
| 732 | __in BOOL fPerMachine, | ||
| 733 | __in_z LPCWSTR wzVersion, | ||
| 734 | __inout BOOL* pfCancel | ||
| 735 | ) = 0; | ||
| 726 | }; | 736 | }; |
diff --git a/src/api/burn/balutil/inc/balinfo.h b/src/api/burn/balutil/inc/balinfo.h index 8f61685f..0c7a5b93 100644 --- a/src/api/burn/balutil/inc/balinfo.h +++ b/src/api/burn/balutil/inc/balinfo.h | |||
| @@ -17,6 +17,7 @@ typedef enum BAL_INFO_PACKAGE_TYPE | |||
| 17 | BAL_INFO_PACKAGE_TYPE_BUNDLE_ADDON, | 17 | BAL_INFO_PACKAGE_TYPE_BUNDLE_ADDON, |
| 18 | BAL_INFO_PACKAGE_TYPE_BUNDLE_PATCH, | 18 | BAL_INFO_PACKAGE_TYPE_BUNDLE_PATCH, |
| 19 | BAL_INFO_PACKAGE_TYPE_BUNDLE_UPDATE, | 19 | BAL_INFO_PACKAGE_TYPE_BUNDLE_UPDATE, |
| 20 | BAL_INFO_PACKAGE_TYPE_BUNDLE_CHAIN, | ||
| 20 | } BAL_INFO_PACKAGE_TYPE; | 21 | } BAL_INFO_PACKAGE_TYPE; |
| 21 | 22 | ||
| 22 | typedef enum _BAL_INFO_RESTART | 23 | typedef enum _BAL_INFO_RESTART |
diff --git a/src/api/wix/WixToolset.Data/Symbols/SymbolDefinitions.cs b/src/api/wix/WixToolset.Data/Symbols/SymbolDefinitions.cs index d4a91343..1fd8ded1 100644 --- a/src/api/wix/WixToolset.Data/Symbols/SymbolDefinitions.cs +++ b/src/api/wix/WixToolset.Data/Symbols/SymbolDefinitions.cs | |||
| @@ -142,6 +142,7 @@ namespace WixToolset.Data | |||
| 142 | WixBundlePackageCommandLine, | 142 | WixBundlePackageCommandLine, |
| 143 | WixBundlePackageExitCode, | 143 | WixBundlePackageExitCode, |
| 144 | WixBundlePackageGroup, | 144 | WixBundlePackageGroup, |
| 145 | WixBundlePackageRelatedBundle, | ||
| 145 | WixBundlePatchTargetCode, | 146 | WixBundlePatchTargetCode, |
| 146 | WixBundlePayload, | 147 | WixBundlePayload, |
| 147 | WixBundlePayloadGroup, | 148 | WixBundlePayloadGroup, |
| @@ -618,6 +619,9 @@ namespace WixToolset.Data | |||
| 618 | case SymbolDefinitionType.WixBundlePackageGroup: | 619 | case SymbolDefinitionType.WixBundlePackageGroup: |
| 619 | return SymbolDefinitions.WixBundlePackageGroup; | 620 | return SymbolDefinitions.WixBundlePackageGroup; |
| 620 | 621 | ||
| 622 | case SymbolDefinitionType.WixBundlePackageRelatedBundle: | ||
| 623 | return SymbolDefinitions.WixBundlePackageRelatedBundle; | ||
| 624 | |||
| 621 | case SymbolDefinitionType.WixBundlePatchTargetCode: | 625 | case SymbolDefinitionType.WixBundlePatchTargetCode: |
| 622 | return SymbolDefinitions.WixBundlePatchTargetCode; | 626 | return SymbolDefinitions.WixBundlePatchTargetCode; |
| 623 | 627 | ||
diff --git a/src/api/wix/WixToolset.Data/Symbols/WixBundleBundlePackageSymbol.cs b/src/api/wix/WixToolset.Data/Symbols/WixBundleBundlePackageSymbol.cs index 36b9eb67..dcf59e28 100644 --- a/src/api/wix/WixToolset.Data/Symbols/WixBundleBundlePackageSymbol.cs +++ b/src/api/wix/WixToolset.Data/Symbols/WixBundleBundlePackageSymbol.cs | |||
| @@ -12,6 +12,7 @@ namespace WixToolset.Data | |||
| 12 | { | 12 | { |
| 13 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.Attributes), IntermediateFieldType.Number), | 13 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.Attributes), IntermediateFieldType.Number), |
| 14 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.BundleId), IntermediateFieldType.String), | 14 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.BundleId), IntermediateFieldType.String), |
| 15 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.Version), IntermediateFieldType.String), | ||
| 15 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.InstallCommand), IntermediateFieldType.String), | 16 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.InstallCommand), IntermediateFieldType.String), |
| 16 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.RepairCommand), IntermediateFieldType.String), | 17 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.RepairCommand), IntermediateFieldType.String), |
| 17 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.UninstallCommand), IntermediateFieldType.String), | 18 | new IntermediateFieldDefinition(nameof(WixBundleBundlePackageSymbolFields.UninstallCommand), IntermediateFieldType.String), |
| @@ -28,6 +29,7 @@ namespace WixToolset.Data.Symbols | |||
| 28 | { | 29 | { |
| 29 | Attributes, | 30 | Attributes, |
| 30 | BundleId, | 31 | BundleId, |
| 32 | Version, | ||
| 31 | InstallCommand, | 33 | InstallCommand, |
| 32 | RepairCommand, | 34 | RepairCommand, |
| 33 | UninstallCommand, | 35 | UninstallCommand, |
| @@ -65,6 +67,12 @@ namespace WixToolset.Data.Symbols | |||
| 65 | set => this.Set((int)WixBundleBundlePackageSymbolFields.BundleId, value); | 67 | set => this.Set((int)WixBundleBundlePackageSymbolFields.BundleId, value); |
| 66 | } | 68 | } |
| 67 | 69 | ||
| 70 | public string Version | ||
| 71 | { | ||
| 72 | get => (string)this.Fields[(int)WixBundleBundlePackageSymbolFields.Version]; | ||
| 73 | set => this.Set((int)WixBundleBundlePackageSymbolFields.Version, value); | ||
| 74 | } | ||
| 75 | |||
| 68 | public string InstallCommand | 76 | public string InstallCommand |
| 69 | { | 77 | { |
| 70 | get => (string)this.Fields[(int)WixBundleBundlePackageSymbolFields.InstallCommand]; | 78 | get => (string)this.Fields[(int)WixBundleBundlePackageSymbolFields.InstallCommand]; |
diff --git a/src/api/wix/WixToolset.Data/Symbols/WixBundlePackageRelatedBundleSymbol.cs b/src/api/wix/WixToolset.Data/Symbols/WixBundlePackageRelatedBundleSymbol.cs new file mode 100644 index 00000000..dfb48714 --- /dev/null +++ b/src/api/wix/WixToolset.Data/Symbols/WixBundlePackageRelatedBundleSymbol.cs | |||
| @@ -0,0 +1,60 @@ | |||
| 1 | // Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information. | ||
| 2 | |||
| 3 | namespace WixToolset.Data | ||
| 4 | { | ||
| 5 | using WixToolset.Data.Symbols; | ||
| 6 | |||
| 7 | public static partial class SymbolDefinitions | ||
| 8 | { | ||
| 9 | public static readonly IntermediateSymbolDefinition WixBundlePackageRelatedBundle = new IntermediateSymbolDefinition( | ||
| 10 | SymbolDefinitionType.WixBundlePackageRelatedBundle, | ||
| 11 | new[] | ||
| 12 | { | ||
| 13 | new IntermediateFieldDefinition(nameof(WixBundlePackageRelatedBundleSymbolFields.PackageRef), IntermediateFieldType.String), | ||
| 14 | new IntermediateFieldDefinition(nameof(WixBundlePackageRelatedBundleSymbolFields.BundleId), IntermediateFieldType.String), | ||
| 15 | new IntermediateFieldDefinition(nameof(WixBundlePackageRelatedBundleSymbolFields.Action), IntermediateFieldType.Number), | ||
| 16 | }, | ||
| 17 | typeof(WixBundlePackageRelatedBundleSymbol)); | ||
| 18 | } | ||
| 19 | } | ||
| 20 | |||
| 21 | namespace WixToolset.Data.Symbols | ||
| 22 | { | ||
| 23 | public enum WixBundlePackageRelatedBundleSymbolFields | ||
| 24 | { | ||
| 25 | PackageRef, | ||
| 26 | BundleId, | ||
| 27 | Action, | ||
| 28 | } | ||
| 29 | |||
| 30 | public class WixBundlePackageRelatedBundleSymbol : IntermediateSymbol | ||
| 31 | { | ||
| 32 | public WixBundlePackageRelatedBundleSymbol() : base(SymbolDefinitions.WixBundlePackageRelatedBundle, null, null) | ||
| 33 | { | ||
| 34 | } | ||
| 35 | |||
| 36 | public WixBundlePackageRelatedBundleSymbol(SourceLineNumber sourceLineNumber, Identifier id = null) : base(SymbolDefinitions.WixBundlePackageRelatedBundle, sourceLineNumber, id) | ||
| 37 | { | ||
| 38 | } | ||
| 39 | |||
| 40 | public IntermediateField this[WixBundlePackageRelatedBundleSymbolFields index] => this.Fields[(int)index]; | ||
| 41 | |||
| 42 | public string PackageRef | ||
| 43 | { | ||
| 44 | get => (string)this.Fields[(int)WixBundlePackageRelatedBundleSymbolFields.PackageRef]; | ||
| 45 | set => this.Set((int)WixBundlePackageRelatedBundleSymbolFields.PackageRef, value); | ||
| 46 | } | ||
| 47 | |||
| 48 | public string BundleId | ||
| 49 | { | ||
| 50 | get => (string)this.Fields[(int)WixBundlePackageRelatedBundleSymbolFields.BundleId]; | ||
| 51 | set => this.Set((int)WixBundlePackageRelatedBundleSymbolFields.BundleId, value); | ||
| 52 | } | ||
| 53 | |||
| 54 | public RelatedBundleActionType Action | ||
| 55 | { | ||
| 56 | get => (RelatedBundleActionType)this.Fields[(int)WixBundlePackageRelatedBundleSymbolFields.Action].AsNumber(); | ||
| 57 | set => this.Set((int)WixBundlePackageRelatedBundleSymbolFields.Action, (int)value); | ||
| 58 | } | ||
| 59 | } | ||
| 60 | } | ||
diff --git a/src/burn/engine/bundlepackageengine.cpp b/src/burn/engine/bundlepackageengine.cpp index f3badfc1..ef08d417 100644 --- a/src/burn/engine/bundlepackageengine.cpp +++ b/src/burn/engine/bundlepackageengine.cpp | |||
| @@ -2,6 +2,18 @@ | |||
| 2 | 2 | ||
| 3 | #include "precomp.h" | 3 | #include "precomp.h" |
| 4 | 4 | ||
| 5 | typedef struct _BUNDLE_QUERY_CONTEXT | ||
| 6 | { | ||
| 7 | BURN_PACKAGE* pPackage; | ||
| 8 | BURN_USER_EXPERIENCE* pUserExperience; | ||
| 9 | BOOL fSelfFound; | ||
| 10 | BOOL fNewerFound; | ||
| 11 | } BUNDLE_QUERY_CONTEXT; | ||
| 12 | |||
| 13 | static BUNDLE_QUERY_CALLBACK_RESULT CALLBACK QueryRelatedBundlesCallback( | ||
| 14 | __in const BUNDLE_QUERY_RELATED_BUNDLE_RESULT* pBundle, | ||
| 15 | __in_opt LPVOID pvContext | ||
| 16 | ); | ||
| 5 | static HRESULT ExecuteBundle( | 17 | static HRESULT ExecuteBundle( |
| 6 | __in BURN_CACHE* pCache, | 18 | __in BURN_CACHE* pCache, |
| 7 | __in BURN_VARIABLES* pVariables, | 19 | __in BURN_VARIABLES* pVariables, |
| @@ -35,6 +47,18 @@ extern "C" HRESULT BundlePackageEngineParsePackageFromXml( | |||
| 35 | hr = XmlGetAttributeEx(pixnBundlePackage, L"BundleId", &pPackage->Bundle.sczBundleId); | 47 | hr = XmlGetAttributeEx(pixnBundlePackage, L"BundleId", &pPackage->Bundle.sczBundleId); |
| 36 | ExitOnRequiredXmlQueryFailure(hr, "Failed to get @BundleId."); | 48 | ExitOnRequiredXmlQueryFailure(hr, "Failed to get @BundleId."); |
| 37 | 49 | ||
| 50 | // @Version | ||
| 51 | hr = XmlGetAttributeEx(pixnBundlePackage, L"Version", &scz); | ||
| 52 | ExitOnRequiredXmlQueryFailure(hr, "Failed to get @Version."); | ||
| 53 | |||
| 54 | hr = VerParseVersion(scz, 0, FALSE, &pPackage->Bundle.pVersion); | ||
| 55 | ExitOnFailure(hr, "Failed to parse @Version: %ls", scz); | ||
| 56 | |||
| 57 | if (pPackage->Bundle.pVersion->fInvalid) | ||
| 58 | { | ||
| 59 | LogId(REPORT_WARNING, MSG_MANIFEST_INVALID_VERSION, scz); | ||
| 60 | } | ||
| 61 | |||
| 38 | // @InstallArguments | 62 | // @InstallArguments |
| 39 | hr = XmlGetAttributeEx(pixnBundlePackage, L"InstallArguments", &pPackage->Bundle.sczInstallArguments); | 63 | hr = XmlGetAttributeEx(pixnBundlePackage, L"InstallArguments", &pPackage->Bundle.sczInstallArguments); |
| 40 | ExitOnOptionalXmlQueryFailure(hr, fFoundXml, "Failed to get @InstallArguments."); | 64 | ExitOnOptionalXmlQueryFailure(hr, fFoundXml, "Failed to get @InstallArguments."); |
| @@ -55,6 +79,9 @@ extern "C" HRESULT BundlePackageEngineParsePackageFromXml( | |||
| 55 | hr = XmlGetYesNoAttribute(pixnBundlePackage, L"Win64", &pPackage->Bundle.fWin64); | 79 | hr = XmlGetYesNoAttribute(pixnBundlePackage, L"Win64", &pPackage->Bundle.fWin64); |
| 56 | ExitOnRequiredXmlQueryFailure(hr, "Failed to get @Win64."); | 80 | ExitOnRequiredXmlQueryFailure(hr, "Failed to get @Win64."); |
| 57 | 81 | ||
| 82 | hr = BundlePackageEngineParseRelatedCodes(pixnBundlePackage, &pPackage->Bundle.rgsczDetectCodes, &pPackage->Bundle.cDetectCodes, &pPackage->Bundle.rgsczUpgradeCodes, &pPackage->Bundle.cUpgradeCodes, &pPackage->Bundle.rgsczAddonCodes, &pPackage->Bundle.cAddonCodes, &pPackage->Bundle.rgsczPatchCodes, &pPackage->Bundle.cPatchCodes); | ||
| 83 | ExitOnFailure(hr, "Failed to parse related codes."); | ||
| 84 | |||
| 58 | hr = ExeEngineParseExitCodesFromXml(pixnBundlePackage, &pPackage->Bundle.rgExitCodes, &pPackage->Bundle.cExitCodes); | 85 | hr = ExeEngineParseExitCodesFromXml(pixnBundlePackage, &pPackage->Bundle.rgExitCodes, &pPackage->Bundle.cExitCodes); |
| 59 | ExitOnFailure(hr, "Failed to parse exit codes."); | 86 | ExitOnFailure(hr, "Failed to parse exit codes."); |
| 60 | 87 | ||
| @@ -70,11 +97,101 @@ LExit: | |||
| 70 | return hr; | 97 | return hr; |
| 71 | } | 98 | } |
| 72 | 99 | ||
| 100 | |||
| 101 | extern "C" HRESULT BundlePackageEngineParseRelatedCodes( | ||
| 102 | __in IXMLDOMNode* pixnBundle, | ||
| 103 | __in LPWSTR** prgsczDetectCodes, | ||
| 104 | __in DWORD* pcDetectCodes, | ||
| 105 | __in LPWSTR** prgsczUpgradeCodes, | ||
| 106 | __in DWORD* pcUpgradeCodes, | ||
| 107 | __in LPWSTR** prgsczAddonCodes, | ||
| 108 | __in DWORD* pcAddonCodes, | ||
| 109 | __in LPWSTR** prgsczPatchCodes, | ||
| 110 | __in DWORD* pcPatchCodes | ||
| 111 | ) | ||
| 112 | { | ||
| 113 | HRESULT hr = S_OK; | ||
| 114 | IXMLDOMNodeList* pixnNodes = NULL; | ||
| 115 | IXMLDOMNode* pixnElement = NULL; | ||
| 116 | LPWSTR sczAction = NULL; | ||
| 117 | LPWSTR sczId = NULL; | ||
| 118 | DWORD cElements = 0; | ||
| 119 | |||
| 120 | hr = XmlSelectNodes(pixnBundle, L"RelatedBundle", &pixnNodes); | ||
| 121 | ExitOnFailure(hr, "Failed to get RelatedBundle nodes"); | ||
| 122 | |||
| 123 | hr = pixnNodes->get_length((long*)&cElements); | ||
| 124 | ExitOnFailure(hr, "Failed to get RelatedBundle element count."); | ||
| 125 | |||
| 126 | for (DWORD i = 0; i < cElements; ++i) | ||
| 127 | { | ||
| 128 | hr = XmlNextElement(pixnNodes, &pixnElement, NULL); | ||
| 129 | ExitOnFailure(hr, "Failed to get next RelatedBundle element."); | ||
| 130 | |||
| 131 | hr = XmlGetAttributeEx(pixnElement, L"Action", &sczAction); | ||
| 132 | ExitOnFailure(hr, "Failed to get @Action."); | ||
| 133 | |||
| 134 | hr = XmlGetAttributeEx(pixnElement, L"Id", &sczId); | ||
| 135 | ExitOnFailure(hr, "Failed to get @Id."); | ||
| 136 | |||
| 137 | if (CSTR_EQUAL == ::CompareStringW(LOCALE_INVARIANT, 0, sczAction, -1, L"Detect", -1)) | ||
| 138 | { | ||
| 139 | hr = MemEnsureArraySizeForNewItems(reinterpret_cast<LPVOID*>(prgsczDetectCodes), *pcDetectCodes, 1, sizeof(LPWSTR), 5); | ||
| 140 | ExitOnFailure(hr, "Failed to resize Detect code array"); | ||
| 141 | |||
| 142 | *prgsczDetectCodes[*pcDetectCodes] = sczId; | ||
| 143 | sczId = NULL; | ||
| 144 | *pcDetectCodes += 1; | ||
| 145 | } | ||
| 146 | else if (CSTR_EQUAL == ::CompareStringW(LOCALE_INVARIANT, 0, sczAction, -1, L"Upgrade", -1)) | ||
| 147 | { | ||
| 148 | hr = MemEnsureArraySizeForNewItems(reinterpret_cast<LPVOID*>(prgsczUpgradeCodes), *pcUpgradeCodes, 1, sizeof(LPWSTR), 5); | ||
| 149 | ExitOnFailure(hr, "Failed to resize Upgrade code array"); | ||
| 150 | |||
| 151 | *prgsczUpgradeCodes[*pcUpgradeCodes] = sczId; | ||
| 152 | sczId = NULL; | ||
| 153 | *pcUpgradeCodes += 1; | ||
| 154 | } | ||
| 155 | else if (CSTR_EQUAL == ::CompareStringW(LOCALE_INVARIANT, 0, sczAction, -1, L"Addon", -1)) | ||
| 156 | { | ||
| 157 | hr = MemEnsureArraySizeForNewItems(reinterpret_cast<LPVOID*>(prgsczAddonCodes), *pcAddonCodes, 1, sizeof(LPWSTR), 5); | ||
| 158 | ExitOnFailure(hr, "Failed to resize Addon code array"); | ||
| 159 | |||
| 160 | *prgsczAddonCodes[*pcAddonCodes] = sczId; | ||
| 161 | sczId = NULL; | ||
| 162 | *pcAddonCodes += 1; | ||
| 163 | } | ||
| 164 | else if (CSTR_EQUAL == ::CompareStringW(LOCALE_INVARIANT, 0, sczAction, -1, L"Patch", -1)) | ||
| 165 | { | ||
| 166 | hr = MemEnsureArraySizeForNewItems(reinterpret_cast<LPVOID*>(prgsczPatchCodes), *pcPatchCodes, 1, sizeof(LPWSTR), 5); | ||
| 167 | ExitOnFailure(hr, "Failed to resize Patch code array"); | ||
| 168 | |||
| 169 | *prgsczPatchCodes[*pcPatchCodes] = sczId; | ||
| 170 | sczId = NULL; | ||
| 171 | *pcPatchCodes += 1; | ||
| 172 | } | ||
| 173 | else | ||
| 174 | { | ||
| 175 | hr = E_INVALIDARG; | ||
| 176 | ExitOnFailure(hr, "Invalid value for @Action: %ls", sczAction); | ||
| 177 | } | ||
| 178 | } | ||
| 179 | |||
| 180 | LExit: | ||
| 181 | ReleaseObject(pixnNodes); | ||
| 182 | ReleaseObject(pixnElement); | ||
| 183 | ReleaseStr(sczAction); | ||
| 184 | ReleaseStr(sczId); | ||
| 185 | |||
| 186 | return hr; | ||
| 187 | } | ||
| 188 | |||
| 73 | extern "C" void BundlePackageEnginePackageUninitialize( | 189 | extern "C" void BundlePackageEnginePackageUninitialize( |
| 74 | __in BURN_PACKAGE* pPackage | 190 | __in BURN_PACKAGE* pPackage |
| 75 | ) | 191 | ) |
| 76 | { | 192 | { |
| 77 | ReleaseStr(pPackage->Bundle.sczBundleId); | 193 | ReleaseStr(pPackage->Bundle.sczBundleId); |
| 194 | ReleaseVerutilVersion(pPackage->Bundle.pVersion); | ||
| 78 | ReleaseStr(pPackage->Bundle.sczRegistrationKey); | 195 | ReleaseStr(pPackage->Bundle.sczRegistrationKey); |
| 79 | ReleaseStr(pPackage->Bundle.sczInstallArguments); | 196 | ReleaseStr(pPackage->Bundle.sczInstallArguments); |
| 80 | ReleaseStr(pPackage->Bundle.sczRepairArguments); | 197 | ReleaseStr(pPackage->Bundle.sczRepairArguments); |
| @@ -92,45 +209,95 @@ extern "C" void BundlePackageEnginePackageUninitialize( | |||
| 92 | MemFree(pPackage->Bundle.rgCommandLineArguments); | 209 | MemFree(pPackage->Bundle.rgCommandLineArguments); |
| 93 | } | 210 | } |
| 94 | 211 | ||
| 212 | for (DWORD i = 0; i < pPackage->Bundle.cDetectCodes; ++i) | ||
| 213 | { | ||
| 214 | ReleaseStr(pPackage->Bundle.rgsczDetectCodes[i]); | ||
| 215 | } | ||
| 216 | ReleaseMem(pPackage->Bundle.rgsczDetectCodes); | ||
| 217 | |||
| 218 | for (DWORD i = 0; i < pPackage->Bundle.cUpgradeCodes; ++i) | ||
| 219 | { | ||
| 220 | ReleaseStr(pPackage->Bundle.rgsczUpgradeCodes[i]); | ||
| 221 | } | ||
| 222 | ReleaseMem(pPackage->Bundle.rgsczUpgradeCodes); | ||
| 223 | |||
| 224 | for (DWORD i = 0; i < pPackage->Bundle.cAddonCodes; ++i) | ||
| 225 | { | ||
| 226 | ReleaseStr(pPackage->Bundle.rgsczAddonCodes[i]); | ||
| 227 | } | ||
| 228 | ReleaseMem(pPackage->Bundle.rgsczAddonCodes); | ||
| 229 | |||
| 230 | for (DWORD i = 0; i < pPackage->Bundle.cPatchCodes; ++i) | ||
| 231 | { | ||
| 232 | ReleaseStr(pPackage->Bundle.rgsczPatchCodes[i]); | ||
| 233 | } | ||
| 234 | ReleaseMem(pPackage->Bundle.rgsczPatchCodes); | ||
| 235 | |||
| 95 | // clear struct | 236 | // clear struct |
| 96 | memset(&pPackage->Bundle, 0, sizeof(pPackage->Bundle)); | 237 | memset(&pPackage->Bundle, 0, sizeof(pPackage->Bundle)); |
| 97 | } | 238 | } |
| 98 | 239 | ||
| 99 | extern "C" HRESULT BundlePackageEngineDetectPackage( | 240 | extern "C" HRESULT BundlePackageEngineDetectPackage( |
| 100 | __in BURN_PACKAGE* pPackage | 241 | __in BURN_PACKAGE* pPackage, |
| 242 | __in BURN_REGISTRATION* /*pRegistration*/, | ||
| 243 | __in BURN_USER_EXPERIENCE* pUserExperience | ||
| 101 | ) | 244 | ) |
| 102 | { | 245 | { |
| 103 | HRESULT hr = S_OK; | 246 | HRESULT hr = S_OK; |
| 104 | HKEY hkRegistration = NULL; | 247 | BUNDLE_QUERY_CONTEXT queryContext = { }; |
| 105 | DWORD dwInstalled = 0; | 248 | |
| 106 | BOOL fDetected = FALSE; | 249 | queryContext.pPackage = pPackage; |
| 107 | HKEY hkRoot = pPackage->fPerMachine ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER; | 250 | queryContext.pUserExperience = pUserExperience; |
| 108 | REG_KEY_BITNESS bitness = pPackage->Bundle.fWin64 ? REG_KEY_64BIT : REG_KEY_32BIT; | 251 | |
| 109 | 252 | hr = BundleQueryRelatedBundles( | |
| 110 | // TODO: detect all related bundles, so that the Obsolete state can be detected. | 253 | BUNDLE_INSTALL_CONTEXT_MACHINE, |
| 111 | hr = RegOpenEx(hkRoot, pPackage->Bundle.sczRegistrationKey, KEY_QUERY_VALUE, bitness, &hkRegistration); | 254 | const_cast<LPCWSTR*>(pPackage->Bundle.rgsczDetectCodes), |
| 112 | if (SUCCEEDED(hr)) | 255 | pPackage->Bundle.cDetectCodes, |
| 256 | const_cast<LPCWSTR*>(pPackage->Bundle.rgsczUpgradeCodes), | ||
| 257 | pPackage->Bundle.cUpgradeCodes, | ||
| 258 | const_cast<LPCWSTR*>(pPackage->Bundle.rgsczAddonCodes), | ||
| 259 | pPackage->Bundle.cAddonCodes, | ||
| 260 | const_cast<LPCWSTR*>(pPackage->Bundle.rgsczPatchCodes), | ||
| 261 | pPackage->Bundle.cPatchCodes, | ||
| 262 | QueryRelatedBundlesCallback, | ||
| 263 | &queryContext); | ||
| 264 | ExitOnFailure(hr, "Failed to query per-machine related bundle packages."); | ||
| 265 | |||
| 266 | hr = BundleQueryRelatedBundles( | ||
| 267 | BUNDLE_INSTALL_CONTEXT_USER, | ||
| 268 | const_cast<LPCWSTR*>(pPackage->Bundle.rgsczDetectCodes), | ||
| 269 | pPackage->Bundle.cDetectCodes, | ||
| 270 | const_cast<LPCWSTR*>(pPackage->Bundle.rgsczUpgradeCodes), | ||
| 271 | pPackage->Bundle.cUpgradeCodes, | ||
| 272 | const_cast<LPCWSTR*>(pPackage->Bundle.rgsczAddonCodes), | ||
| 273 | pPackage->Bundle.cAddonCodes, | ||
| 274 | const_cast<LPCWSTR*>(pPackage->Bundle.rgsczPatchCodes), | ||
| 275 | pPackage->Bundle.cPatchCodes, | ||
| 276 | QueryRelatedBundlesCallback, | ||
| 277 | &queryContext); | ||
| 278 | ExitOnFailure(hr, "Failed to query per-user related bundle packages."); | ||
| 279 | |||
| 280 | if (queryContext.fNewerFound) | ||
| 113 | { | 281 | { |
| 114 | hr = RegReadNumber(hkRegistration, REGISTRY_BUNDLE_INSTALLED, &dwInstalled); | 282 | pPackage->currentState = queryContext.fSelfFound ? BOOTSTRAPPER_PACKAGE_STATE_SUPERSEDED : BOOTSTRAPPER_PACKAGE_STATE_OBSOLETE; |
| 115 | } | 283 | } |
| 116 | 284 | else | |
| 117 | // Not finding the key or value is okay. | ||
| 118 | if (E_FILENOTFOUND == hr || E_PATHNOTFOUND == hr) | ||
| 119 | { | 285 | { |
| 120 | hr = S_OK; | 286 | pPackage->currentState = queryContext.fSelfFound ? BOOTSTRAPPER_PACKAGE_STATE_PRESENT : BOOTSTRAPPER_PACKAGE_STATE_ABSENT; |
| 121 | } | 287 | } |
| 122 | 288 | ||
| 123 | fDetected = (1 == dwInstalled); | ||
| 124 | |||
| 125 | // update detect state | ||
| 126 | pPackage->currentState = fDetected ? BOOTSTRAPPER_PACKAGE_STATE_PRESENT : BOOTSTRAPPER_PACKAGE_STATE_ABSENT; | ||
| 127 | |||
| 128 | if (pPackage->fCanAffectRegistration) | 289 | if (pPackage->fCanAffectRegistration) |
| 129 | { | 290 | { |
| 130 | pPackage->installRegistrationState = BOOTSTRAPPER_PACKAGE_STATE_ABSENT < pPackage->currentState ? BURN_PACKAGE_REGISTRATION_STATE_PRESENT : BURN_PACKAGE_REGISTRATION_STATE_ABSENT; | 291 | pPackage->installRegistrationState = BOOTSTRAPPER_PACKAGE_STATE_ABSENT < pPackage->currentState ? BURN_PACKAGE_REGISTRATION_STATE_PRESENT : BURN_PACKAGE_REGISTRATION_STATE_ABSENT; |
| 131 | } | 292 | } |
| 132 | 293 | ||
| 133 | ReleaseRegKey(hkRegistration); | 294 | // TODO: The bundle is registering itself as a dependent when installed as a chain package, which prevents us from uninstalling it. |
| 295 | //hr = DependencyDetectChainPackage(pPackage, pRegistration); | ||
| 296 | //ExitOnFailure(hr, "Failed to detect dependencies for BUNDLE package."); | ||
| 297 | |||
| 298 | // TODO: uninstalling compatible Bundles like MsiEngine supports? | ||
| 299 | |||
| 300 | LExit: | ||
| 134 | return hr; | 301 | return hr; |
| 135 | } | 302 | } |
| 136 | 303 | ||
| @@ -148,7 +315,8 @@ extern "C" HRESULT BundlePackageEnginePlanCalculatePackage( | |||
| 148 | // execute action | 315 | // execute action |
| 149 | switch (pPackage->currentState) | 316 | switch (pPackage->currentState) |
| 150 | { | 317 | { |
| 151 | case BOOTSTRAPPER_PACKAGE_STATE_PRESENT: | 318 | case BOOTSTRAPPER_PACKAGE_STATE_PRESENT: __fallthrough; |
| 319 | case BOOTSTRAPPER_PACKAGE_STATE_SUPERSEDED: | ||
| 152 | switch (pPackage->requested) | 320 | switch (pPackage->requested) |
| 153 | { | 321 | { |
| 154 | case BOOTSTRAPPER_REQUEST_STATE_PRESENT: | 322 | case BOOTSTRAPPER_REQUEST_STATE_PRESENT: |
| @@ -173,6 +341,7 @@ extern "C" HRESULT BundlePackageEnginePlanCalculatePackage( | |||
| 173 | } | 341 | } |
| 174 | break; | 342 | break; |
| 175 | 343 | ||
| 344 | case BOOTSTRAPPER_PACKAGE_STATE_OBSOLETE: __fallthrough; | ||
| 176 | case BOOTSTRAPPER_PACKAGE_STATE_ABSENT: | 345 | case BOOTSTRAPPER_PACKAGE_STATE_ABSENT: |
| 177 | switch (pPackage->requested) | 346 | switch (pPackage->requested) |
| 178 | { | 347 | { |
| @@ -200,7 +369,8 @@ extern "C" HRESULT BundlePackageEnginePlanCalculatePackage( | |||
| 200 | { | 369 | { |
| 201 | switch (pPackage->currentState) | 370 | switch (pPackage->currentState) |
| 202 | { | 371 | { |
| 203 | case BOOTSTRAPPER_PACKAGE_STATE_PRESENT: | 372 | case BOOTSTRAPPER_PACKAGE_STATE_PRESENT: __fallthrough; |
| 373 | case BOOTSTRAPPER_PACKAGE_STATE_SUPERSEDED: | ||
| 204 | switch (pPackage->requested) | 374 | switch (pPackage->requested) |
| 205 | { | 375 | { |
| 206 | case BOOTSTRAPPER_REQUEST_STATE_PRESENT: __fallthrough; | 376 | case BOOTSTRAPPER_REQUEST_STATE_PRESENT: __fallthrough; |
| @@ -218,6 +388,7 @@ extern "C" HRESULT BundlePackageEnginePlanCalculatePackage( | |||
| 218 | } | 388 | } |
| 219 | break; | 389 | break; |
| 220 | 390 | ||
| 391 | case BOOTSTRAPPER_PACKAGE_STATE_OBSOLETE: __fallthrough; | ||
| 221 | case BOOTSTRAPPER_PACKAGE_STATE_ABSENT: | 392 | case BOOTSTRAPPER_PACKAGE_STATE_ABSENT: |
| 222 | switch (pPackage->requested) | 393 | switch (pPackage->requested) |
| 223 | { | 394 | { |
| @@ -481,6 +652,66 @@ LExit: | |||
| 481 | return; | 652 | return; |
| 482 | } | 653 | } |
| 483 | 654 | ||
| 655 | static BUNDLE_QUERY_CALLBACK_RESULT CALLBACK QueryRelatedBundlesCallback( | ||
| 656 | __in const BUNDLE_QUERY_RELATED_BUNDLE_RESULT* pBundle, | ||
| 657 | __in_opt LPVOID pvContext | ||
| 658 | ) | ||
| 659 | { | ||
| 660 | HRESULT hr = S_OK; | ||
| 661 | BUNDLE_QUERY_CALLBACK_RESULT result = BUNDLE_QUERY_CALLBACK_RESULT_CONTINUE; | ||
| 662 | LPWSTR sczBundleVersion = NULL; | ||
| 663 | VERUTIL_VERSION* pVersion = NULL; | ||
| 664 | int nCompare = 0; | ||
| 665 | BUNDLE_QUERY_CONTEXT* pContext = reinterpret_cast<BUNDLE_QUERY_CONTEXT*>(pvContext); | ||
| 666 | BURN_PACKAGE* pPackage = pContext->pPackage; | ||
| 667 | BOOTSTRAPPER_RELATION_TYPE relationType = RelatedBundleConvertRelationType(pBundle->relationType); | ||
| 668 | BOOL fPerMachine = BUNDLE_INSTALL_CONTEXT_MACHINE == pBundle->installContext; | ||
| 669 | |||
| 670 | if (CSTR_EQUAL == ::CompareStringW(LOCALE_NEUTRAL, NORM_IGNORECASE, pBundle->wzBundleId, -1, pPackage->Bundle.sczBundleId, -1)) | ||
| 671 | { | ||
| 672 | Assert(BOOTSTRAPPER_RELATION_UPGRADE == relationType); | ||
| 673 | Assert(pPackage->Bundle.fWin64 == (REG_KEY_64BIT == pBundle->regBitness)); | ||
| 674 | |||
| 675 | pContext->fSelfFound = TRUE; | ||
| 676 | } | ||
| 677 | |||
| 678 | hr = RegReadString(pBundle->hkBundle, BURN_REGISTRATION_REGISTRY_BUNDLE_VERSION, &sczBundleVersion); | ||
| 679 | ExitOnFailure(hr, "Failed to read version from registry for related bundle package: %ls", pBundle->wzBundleId); | ||
| 680 | |||
| 681 | hr = VerParseVersion(sczBundleVersion, 0, FALSE, &pVersion); | ||
| 682 | ExitOnFailure(hr, "Failed to parse related bundle package version: %ls", sczBundleVersion); | ||
| 683 | |||
| 684 | if (pVersion->fInvalid) | ||
| 685 | { | ||
| 686 | LogId(REPORT_WARNING, MSG_RELATED_PACKAGE_INVALID_VERSION, pBundle->wzBundleId, sczBundleVersion); | ||
| 687 | } | ||
| 688 | |||
| 689 | if (BOOTSTRAPPER_RELATION_UPGRADE == relationType) | ||
| 690 | { | ||
| 691 | hr = VerCompareParsedVersions(pPackage->Bundle.pVersion, pVersion, &nCompare); | ||
| 692 | ExitOnFailure(hr, "Failed to compare related bundle package version: %ls", pVersion->sczVersion); | ||
| 693 | |||
| 694 | if (nCompare < 0) | ||
| 695 | { | ||
| 696 | pContext->fNewerFound = TRUE; | ||
| 697 | } | ||
| 698 | } | ||
| 699 | |||
| 700 | result = BUNDLE_QUERY_CALLBACK_RESULT_CANCEL; | ||
| 701 | |||
| 702 | // Pass to BA. | ||
| 703 | hr = UserExperienceOnDetectRelatedBundlePackage(pContext->pUserExperience, pPackage->sczId, pBundle->wzBundleId, relationType, fPerMachine, pVersion); | ||
| 704 | ExitOnRootFailure(hr, "BA aborted detect related BUNDLE package."); | ||
| 705 | |||
| 706 | result = BUNDLE_QUERY_CALLBACK_RESULT_CONTINUE; | ||
| 707 | |||
| 708 | LExit: | ||
| 709 | ReleaseVerutilVersion(pVersion); | ||
| 710 | ReleaseStr(sczBundleVersion); | ||
| 711 | |||
| 712 | return result; | ||
| 713 | } | ||
| 714 | |||
| 484 | static HRESULT ExecuteBundle( | 715 | static HRESULT ExecuteBundle( |
| 485 | __in BURN_CACHE* pCache, | 716 | __in BURN_CACHE* pCache, |
| 486 | __in BURN_VARIABLES* pVariables, | 717 | __in BURN_VARIABLES* pVariables, |
diff --git a/src/burn/engine/bundlepackageengine.h b/src/burn/engine/bundlepackageengine.h index 9271ac6a..e245f6ce 100644 --- a/src/burn/engine/bundlepackageengine.h +++ b/src/burn/engine/bundlepackageengine.h | |||
| @@ -13,11 +13,24 @@ HRESULT BundlePackageEngineParsePackageFromXml( | |||
| 13 | __in IXMLDOMNode* pixnBundlePackage, | 13 | __in IXMLDOMNode* pixnBundlePackage, |
| 14 | __in BURN_PACKAGE* pPackage | 14 | __in BURN_PACKAGE* pPackage |
| 15 | ); | 15 | ); |
| 16 | HRESULT BundlePackageEngineParseRelatedCodes( | ||
| 17 | __in IXMLDOMNode* pixnBundle, | ||
| 18 | __in LPWSTR** prgsczDetectCodes, | ||
| 19 | __in DWORD* pcDetectCodes, | ||
| 20 | __in LPWSTR** prgsczUpgradeCodes, | ||
| 21 | __in DWORD* pcUpgradeCodes, | ||
| 22 | __in LPWSTR** prgsczAddonCodes, | ||
| 23 | __in DWORD* pcAddonCodes, | ||
| 24 | __in LPWSTR** prgsczPatchCodes, | ||
| 25 | __in DWORD* pcPatchCodes | ||
| 26 | ); | ||
| 16 | void BundlePackageEnginePackageUninitialize( | 27 | void BundlePackageEnginePackageUninitialize( |
| 17 | __in BURN_PACKAGE* pPackage | 28 | __in BURN_PACKAGE* pPackage |
| 18 | ); | 29 | ); |
| 19 | HRESULT BundlePackageEngineDetectPackage( | 30 | HRESULT BundlePackageEngineDetectPackage( |
| 20 | __in BURN_PACKAGE* pPackage | 31 | __in BURN_PACKAGE* pPackage, |
| 32 | __in BURN_REGISTRATION* pRegistration, | ||
| 33 | __in BURN_USER_EXPERIENCE* pUserExperience | ||
| 21 | ); | 34 | ); |
| 22 | HRESULT BundlePackageEnginePlanCalculatePackage( | 35 | HRESULT BundlePackageEnginePlanCalculatePackage( |
| 23 | __in BURN_PACKAGE* pPackage | 36 | __in BURN_PACKAGE* pPackage |
diff --git a/src/burn/engine/core.cpp b/src/burn/engine/core.cpp index 3370ad05..0bbf7039 100644 --- a/src/burn/engine/core.cpp +++ b/src/burn/engine/core.cpp | |||
| @@ -2121,7 +2121,7 @@ static HRESULT DetectPackage( | |||
| 2121 | switch (pPackage->type) | 2121 | switch (pPackage->type) |
| 2122 | { | 2122 | { |
| 2123 | case BURN_PACKAGE_TYPE_BUNDLE: | 2123 | case BURN_PACKAGE_TYPE_BUNDLE: |
| 2124 | hr = BundlePackageEngineDetectPackage(pPackage); | 2124 | hr = BundlePackageEngineDetectPackage(pPackage, &pEngineState->registration, &pEngineState->userExperience); |
| 2125 | break; | 2125 | break; |
| 2126 | 2126 | ||
| 2127 | case BURN_PACKAGE_TYPE_EXE: | 2127 | case BURN_PACKAGE_TYPE_EXE: |
diff --git a/src/burn/engine/package.h b/src/burn/engine/package.h index c13c651b..4021031f 100644 --- a/src/burn/engine/package.h +++ b/src/burn/engine/package.h | |||
| @@ -303,14 +303,23 @@ typedef struct _BURN_PACKAGE | |||
| 303 | struct | 303 | struct |
| 304 | { | 304 | { |
| 305 | LPWSTR sczBundleId; | 305 | LPWSTR sczBundleId; |
| 306 | VERUTIL_VERSION* pVersion; | ||
| 306 | LPWSTR sczRegistrationKey; | 307 | LPWSTR sczRegistrationKey; |
| 307 | LPWSTR sczInstallArguments; | 308 | LPWSTR sczInstallArguments; |
| 308 | LPWSTR sczRepairArguments; | 309 | LPWSTR sczRepairArguments; |
| 309 | LPWSTR sczUninstallArguments; | 310 | LPWSTR sczUninstallArguments; |
| 310 | 311 | ||
| 311 | LPWSTR sczIgnoreDependencies; | 312 | LPWSTR* rgsczDetectCodes; |
| 312 | LPCWSTR wzAncestors; // points directly into engine state. | 313 | DWORD cDetectCodes; |
| 313 | LPCWSTR wzEngineWorkingDirectory; // points directly into engine state. | 314 | |
| 315 | LPWSTR* rgsczUpgradeCodes; | ||
| 316 | DWORD cUpgradeCodes; | ||
| 317 | |||
| 318 | LPWSTR* rgsczAddonCodes; | ||
| 319 | DWORD cAddonCodes; | ||
| 320 | |||
| 321 | LPWSTR* rgsczPatchCodes; | ||
| 322 | DWORD cPatchCodes; | ||
| 314 | 323 | ||
| 315 | BOOL fWin64; | 324 | BOOL fWin64; |
| 316 | BOOL fSupportsBurnProtocol; | 325 | BOOL fSupportsBurnProtocol; |
| @@ -320,6 +329,10 @@ typedef struct _BURN_PACKAGE | |||
| 320 | 329 | ||
| 321 | BURN_EXE_COMMAND_LINE_ARGUMENT* rgCommandLineArguments; | 330 | BURN_EXE_COMMAND_LINE_ARGUMENT* rgCommandLineArguments; |
| 322 | DWORD cCommandLineArguments; | 331 | DWORD cCommandLineArguments; |
| 332 | |||
| 333 | LPWSTR sczIgnoreDependencies; | ||
| 334 | LPCWSTR wzAncestors; // points directly into engine state. | ||
| 335 | LPCWSTR wzEngineWorkingDirectory; // points directly into engine state. | ||
| 323 | } Bundle; | 336 | } Bundle; |
| 324 | struct | 337 | struct |
| 325 | { | 338 | { |
diff --git a/src/burn/engine/registration.cpp b/src/burn/engine/registration.cpp index 78f8eeb1..f2b8383d 100644 --- a/src/burn/engine/registration.cpp +++ b/src/burn/engine/registration.cpp | |||
| @@ -71,10 +71,6 @@ static HRESULT UpdateResumeMode( | |||
| 71 | __in BOOTSTRAPPER_REGISTRATION_TYPE registrationType, | 71 | __in BOOTSTRAPPER_REGISTRATION_TYPE registrationType, |
| 72 | __in BOOL fRestartInitiated | 72 | __in BOOL fRestartInitiated |
| 73 | ); | 73 | ); |
| 74 | static HRESULT ParseRelatedCodes( | ||
| 75 | __in BURN_REGISTRATION* pRegistration, | ||
| 76 | __in IXMLDOMNode* pixnBundle | ||
| 77 | ); | ||
| 78 | static HRESULT FormatUpdateRegistrationKey( | 74 | static HRESULT FormatUpdateRegistrationKey( |
| 79 | __in BURN_REGISTRATION* pRegistration, | 75 | __in BURN_REGISTRATION* pRegistration, |
| 80 | __out_z LPWSTR* psczKey | 76 | __out_z LPWSTR* psczKey |
| @@ -143,7 +139,7 @@ extern "C" HRESULT RegistrationParseFromXml( | |||
| 143 | hr = XmlGetAttributeEx(pixnRegistrationNode, L"Tag", &pRegistration->sczTag); | 139 | hr = XmlGetAttributeEx(pixnRegistrationNode, L"Tag", &pRegistration->sczTag); |
| 144 | ExitOnFailure(hr, "Failed to get @Tag."); | 140 | ExitOnFailure(hr, "Failed to get @Tag."); |
| 145 | 141 | ||
| 146 | hr = ParseRelatedCodes(pRegistration, pixnBundle); | 142 | hr = BundlePackageEngineParseRelatedCodes(pixnBundle, &pRegistration->rgsczDetectCodes, &pRegistration->cDetectCodes, &pRegistration->rgsczUpgradeCodes, &pRegistration->cUpgradeCodes, &pRegistration->rgsczAddonCodes, &pRegistration->cAddonCodes, &pRegistration->rgsczPatchCodes, &pRegistration->cPatchCodes); |
| 147 | ExitOnFailure(hr, "Failed to parse related bundles"); | 143 | ExitOnFailure(hr, "Failed to parse related bundles"); |
| 148 | 144 | ||
| 149 | // @Version | 145 | // @Version |
| @@ -1395,87 +1391,6 @@ LExit: | |||
| 1395 | return hr; | 1391 | return hr; |
| 1396 | } | 1392 | } |
| 1397 | 1393 | ||
| 1398 | static HRESULT ParseRelatedCodes( | ||
| 1399 | __in BURN_REGISTRATION* pRegistration, | ||
| 1400 | __in IXMLDOMNode* pixnBundle | ||
| 1401 | ) | ||
| 1402 | { | ||
| 1403 | HRESULT hr = S_OK; | ||
| 1404 | IXMLDOMNodeList* pixnNodes = NULL; | ||
| 1405 | IXMLDOMNode* pixnElement = NULL; | ||
| 1406 | LPWSTR sczAction = NULL; | ||
| 1407 | LPWSTR sczId = NULL; | ||
| 1408 | DWORD cElements = 0; | ||
| 1409 | |||
| 1410 | hr = XmlSelectNodes(pixnBundle, L"RelatedBundle", &pixnNodes); | ||
| 1411 | ExitOnFailure(hr, "Failed to get RelatedBundle nodes"); | ||
| 1412 | |||
| 1413 | hr = pixnNodes->get_length((long*)&cElements); | ||
| 1414 | ExitOnFailure(hr, "Failed to get RelatedBundle element count."); | ||
| 1415 | |||
| 1416 | for (DWORD i = 0; i < cElements; ++i) | ||
| 1417 | { | ||
| 1418 | hr = XmlNextElement(pixnNodes, &pixnElement, NULL); | ||
| 1419 | ExitOnFailure(hr, "Failed to get next RelatedBundle element."); | ||
| 1420 | |||
| 1421 | hr = XmlGetAttributeEx(pixnElement, L"Action", &sczAction); | ||
| 1422 | ExitOnFailure(hr, "Failed to get @Action."); | ||
| 1423 | |||
| 1424 | hr = XmlGetAttributeEx(pixnElement, L"Id", &sczId); | ||
| 1425 | ExitOnFailure(hr, "Failed to get @Id."); | ||
| 1426 | |||
| 1427 | if (CSTR_EQUAL == ::CompareStringW(LOCALE_INVARIANT, 0, sczAction, -1, L"Detect", -1)) | ||
| 1428 | { | ||
| 1429 | hr = MemEnsureArraySize(reinterpret_cast<LPVOID*>(&pRegistration->rgsczDetectCodes), pRegistration->cDetectCodes + 1, sizeof(LPWSTR), 5); | ||
| 1430 | ExitOnFailure(hr, "Failed to resize Detect code array in registration"); | ||
| 1431 | |||
| 1432 | pRegistration->rgsczDetectCodes[pRegistration->cDetectCodes] = sczId; | ||
| 1433 | sczId = NULL; | ||
| 1434 | ++pRegistration->cDetectCodes; | ||
| 1435 | } | ||
| 1436 | else if (CSTR_EQUAL == ::CompareStringW(LOCALE_INVARIANT, 0, sczAction, -1, L"Upgrade", -1)) | ||
| 1437 | { | ||
| 1438 | hr = MemEnsureArraySize(reinterpret_cast<LPVOID*>(&pRegistration->rgsczUpgradeCodes), pRegistration->cUpgradeCodes + 1, sizeof(LPWSTR), 5); | ||
| 1439 | ExitOnFailure(hr, "Failed to resize Upgrade code array in registration"); | ||
| 1440 | |||
| 1441 | pRegistration->rgsczUpgradeCodes[pRegistration->cUpgradeCodes] = sczId; | ||
| 1442 | sczId = NULL; | ||
| 1443 | ++pRegistration->cUpgradeCodes; | ||
| 1444 | } | ||
| 1445 | else if (CSTR_EQUAL == ::CompareStringW(LOCALE_INVARIANT, 0, sczAction, -1, L"Addon", -1)) | ||
| 1446 | { | ||
| 1447 | hr = MemEnsureArraySize(reinterpret_cast<LPVOID*>(&pRegistration->rgsczAddonCodes), pRegistration->cAddonCodes + 1, sizeof(LPWSTR), 5); | ||
| 1448 | ExitOnFailure(hr, "Failed to resize Addon code array in registration"); | ||
| 1449 | |||
| 1450 | pRegistration->rgsczAddonCodes[pRegistration->cAddonCodes] = sczId; | ||
| 1451 | sczId = NULL; | ||
| 1452 | ++pRegistration->cAddonCodes; | ||
| 1453 | } | ||
| 1454 | else if (CSTR_EQUAL == ::CompareStringW(LOCALE_INVARIANT, 0, sczAction, -1, L"Patch", -1)) | ||
| 1455 | { | ||
| 1456 | hr = MemEnsureArraySize(reinterpret_cast<LPVOID*>(&pRegistration->rgsczPatchCodes), pRegistration->cPatchCodes + 1, sizeof(LPWSTR), 5); | ||
| 1457 | ExitOnFailure(hr, "Failed to resize Patch code array in registration"); | ||
| 1458 | |||
| 1459 | pRegistration->rgsczPatchCodes[pRegistration->cPatchCodes] = sczId; | ||
| 1460 | sczId = NULL; | ||
| 1461 | ++pRegistration->cPatchCodes; | ||
| 1462 | } | ||
| 1463 | else | ||
| 1464 | { | ||
| 1465 | hr = E_INVALIDARG; | ||
| 1466 | ExitOnFailure(hr, "Invalid value for @Action: %ls", sczAction); | ||
| 1467 | } | ||
| 1468 | } | ||
| 1469 | |||
| 1470 | LExit: | ||
| 1471 | ReleaseObject(pixnNodes); | ||
| 1472 | ReleaseObject(pixnElement); | ||
| 1473 | ReleaseStr(sczAction); | ||
| 1474 | ReleaseStr(sczId); | ||
| 1475 | |||
| 1476 | return hr; | ||
| 1477 | } | ||
| 1478 | |||
| 1479 | static HRESULT FormatUpdateRegistrationKey( | 1394 | static HRESULT FormatUpdateRegistrationKey( |
| 1480 | __in BURN_REGISTRATION* pRegistration, | 1395 | __in BURN_REGISTRATION* pRegistration, |
| 1481 | __out_z LPWSTR* psczKey | 1396 | __out_z LPWSTR* psczKey |
diff --git a/src/burn/engine/relatedbundle.cpp b/src/burn/engine/relatedbundle.cpp index 58911711..586446b1 100644 --- a/src/burn/engine/relatedbundle.cpp +++ b/src/burn/engine/relatedbundle.cpp | |||
| @@ -143,6 +143,30 @@ extern "C" void RelatedBundlesSortPlan( | |||
| 143 | qsort_s(pRelatedBundles->rgpPlanSortedRelatedBundles, pRelatedBundles->cRelatedBundles, sizeof(BURN_RELATED_BUNDLE*), CompareRelatedBundlesPlan, NULL); | 143 | qsort_s(pRelatedBundles->rgpPlanSortedRelatedBundles, pRelatedBundles->cRelatedBundles, sizeof(BURN_RELATED_BUNDLE*), CompareRelatedBundlesPlan, NULL); |
| 144 | } | 144 | } |
| 145 | 145 | ||
| 146 | extern "C" BOOTSTRAPPER_RELATION_TYPE RelatedBundleConvertRelationType( | ||
| 147 | __in BUNDLE_RELATION_TYPE relationType | ||
| 148 | ) | ||
| 149 | { | ||
| 150 | switch (relationType) | ||
| 151 | { | ||
| 152 | case BUNDLE_RELATION_DETECT: | ||
| 153 | return BOOTSTRAPPER_RELATION_DETECT; | ||
| 154 | case BUNDLE_RELATION_UPGRADE: | ||
| 155 | return BOOTSTRAPPER_RELATION_UPGRADE; | ||
| 156 | case BUNDLE_RELATION_ADDON: | ||
| 157 | return BOOTSTRAPPER_RELATION_ADDON; | ||
| 158 | case BUNDLE_RELATION_PATCH: | ||
| 159 | return BOOTSTRAPPER_RELATION_PATCH; | ||
| 160 | case BUNDLE_RELATION_DEPENDENT_ADDON: | ||
| 161 | return BOOTSTRAPPER_RELATION_DEPENDENT_ADDON; | ||
| 162 | case BUNDLE_RELATION_DEPENDENT_PATCH: | ||
| 163 | return BOOTSTRAPPER_RELATION_DEPENDENT_PATCH; | ||
| 164 | default: | ||
| 165 | AssertSz(BUNDLE_RELATION_NONE == relationType, "Unknown BUNDLE_RELATION_TYPE"); | ||
| 166 | return BOOTSTRAPPER_RELATION_NONE; | ||
| 167 | } | ||
| 168 | } | ||
| 169 | |||
| 146 | 170 | ||
| 147 | // internal helper functions | 171 | // internal helper functions |
| 148 | 172 | ||
| @@ -248,30 +272,6 @@ LExit: | |||
| 248 | return result; | 272 | return result; |
| 249 | } | 273 | } |
| 250 | 274 | ||
| 251 | static BOOTSTRAPPER_RELATION_TYPE ConvertRelationType( | ||
| 252 | __in BUNDLE_RELATION_TYPE relationType | ||
| 253 | ) | ||
| 254 | { | ||
| 255 | switch (relationType) | ||
| 256 | { | ||
| 257 | case BUNDLE_RELATION_DETECT: | ||
| 258 | return BOOTSTRAPPER_RELATION_DETECT; | ||
| 259 | case BUNDLE_RELATION_UPGRADE: | ||
| 260 | return BOOTSTRAPPER_RELATION_UPGRADE; | ||
| 261 | case BUNDLE_RELATION_ADDON: | ||
| 262 | return BOOTSTRAPPER_RELATION_ADDON; | ||
| 263 | case BUNDLE_RELATION_PATCH: | ||
| 264 | return BOOTSTRAPPER_RELATION_PATCH; | ||
| 265 | case BUNDLE_RELATION_DEPENDENT_ADDON: | ||
| 266 | return BOOTSTRAPPER_RELATION_DEPENDENT_ADDON; | ||
| 267 | case BUNDLE_RELATION_DEPENDENT_PATCH: | ||
| 268 | return BOOTSTRAPPER_RELATION_DEPENDENT_PATCH; | ||
| 269 | default: | ||
| 270 | AssertSz(BUNDLE_RELATION_NONE == relationType, "Unknown BUNDLE_RELATION_TYPE"); | ||
| 271 | return BOOTSTRAPPER_RELATION_NONE; | ||
| 272 | } | ||
| 273 | } | ||
| 274 | |||
| 275 | static HRESULT LoadIfRelatedBundle( | 275 | static HRESULT LoadIfRelatedBundle( |
| 276 | __in const BUNDLE_QUERY_RELATED_BUNDLE_RESULT* pBundle, | 276 | __in const BUNDLE_QUERY_RELATED_BUNDLE_RESULT* pBundle, |
| 277 | __in BURN_REGISTRATION* pRegistration, | 277 | __in BURN_REGISTRATION* pRegistration, |
| @@ -280,7 +280,7 @@ static HRESULT LoadIfRelatedBundle( | |||
| 280 | { | 280 | { |
| 281 | HRESULT hr = S_OK; | 281 | HRESULT hr = S_OK; |
| 282 | BOOL fPerMachine = BUNDLE_INSTALL_CONTEXT_MACHINE == pBundle->installContext; | 282 | BOOL fPerMachine = BUNDLE_INSTALL_CONTEXT_MACHINE == pBundle->installContext; |
| 283 | BOOTSTRAPPER_RELATION_TYPE relationType = ConvertRelationType(pBundle->relationType); | 283 | BOOTSTRAPPER_RELATION_TYPE relationType = RelatedBundleConvertRelationType(pBundle->relationType); |
| 284 | BURN_RELATED_BUNDLE* pRelatedBundle = NULL; | 284 | BURN_RELATED_BUNDLE* pRelatedBundle = NULL; |
| 285 | 285 | ||
| 286 | // If we found our bundle id, it's not a related bundle. | 286 | // If we found our bundle id, it's not a related bundle. |
diff --git a/src/burn/engine/relatedbundle.h b/src/burn/engine/relatedbundle.h index 24469f3d..e98d0ede 100644 --- a/src/burn/engine/relatedbundle.h +++ b/src/burn/engine/relatedbundle.h | |||
| @@ -25,6 +25,9 @@ void RelatedBundlesSortDetect( | |||
| 25 | void RelatedBundlesSortPlan( | 25 | void RelatedBundlesSortPlan( |
| 26 | __in BURN_RELATED_BUNDLES* pRelatedBundles | 26 | __in BURN_RELATED_BUNDLES* pRelatedBundles |
| 27 | ); | 27 | ); |
| 28 | BOOTSTRAPPER_RELATION_TYPE RelatedBundleConvertRelationType( | ||
| 29 | __in BUNDLE_RELATION_TYPE relationType | ||
| 30 | ); | ||
| 28 | 31 | ||
| 29 | #if defined(__cplusplus) | 32 | #if defined(__cplusplus) |
| 30 | } | 33 | } |
diff --git a/src/burn/engine/userexperience.cpp b/src/burn/engine/userexperience.cpp index 06f87363..a1a010d2 100644 --- a/src/burn/engine/userexperience.cpp +++ b/src/burn/engine/userexperience.cpp | |||
| @@ -104,7 +104,7 @@ extern "C" HRESULT UserExperienceLoad( | |||
| 104 | args.pCommand = pCommand; | 104 | args.pCommand = pCommand; |
| 105 | args.pfnBootstrapperEngineProc = EngineForApplicationProc; | 105 | args.pfnBootstrapperEngineProc = EngineForApplicationProc; |
| 106 | args.pvBootstrapperEngineProcContext = pEngineContext; | 106 | args.pvBootstrapperEngineProcContext = pEngineContext; |
| 107 | args.qwEngineAPIVersion = MAKEQWORDVERSION(2022, 3, 17, 0); | 107 | args.qwEngineAPIVersion = MAKEQWORDVERSION(2022, 3, 31, 0); |
| 108 | 108 | ||
| 109 | results.cbSize = sizeof(BOOTSTRAPPER_CREATE_RESULTS); | 109 | results.cbSize = sizeof(BOOTSTRAPPER_CREATE_RESULTS); |
| 110 | 110 | ||
| @@ -1247,6 +1247,40 @@ LExit: | |||
| 1247 | return hr; | 1247 | return hr; |
| 1248 | } | 1248 | } |
| 1249 | 1249 | ||
| 1250 | EXTERN_C BAAPI UserExperienceOnDetectRelatedBundlePackage( | ||
| 1251 | __in BURN_USER_EXPERIENCE* pUserExperience, | ||
| 1252 | __in_z LPCWSTR wzPackageId, | ||
| 1253 | __in_z LPCWSTR wzBundleId, | ||
| 1254 | __in BOOTSTRAPPER_RELATION_TYPE relationType, | ||
| 1255 | __in BOOL fPerMachine, | ||
| 1256 | __in VERUTIL_VERSION* pVersion | ||
| 1257 | ) | ||
| 1258 | { | ||
| 1259 | HRESULT hr = S_OK; | ||
| 1260 | BA_ONDETECTRELATEDBUNDLEPACKAGE_ARGS args = { }; | ||
| 1261 | BA_ONDETECTRELATEDBUNDLEPACKAGE_RESULTS results = { }; | ||
| 1262 | |||
| 1263 | args.cbSize = sizeof(args); | ||
| 1264 | args.wzPackageId = wzPackageId; | ||
| 1265 | args.wzBundleId = wzBundleId; | ||
| 1266 | args.relationType = relationType; | ||
| 1267 | args.fPerMachine = fPerMachine; | ||
| 1268 | args.wzVersion = pVersion->sczVersion; | ||
| 1269 | |||
| 1270 | results.cbSize = sizeof(results); | ||
| 1271 | |||
| 1272 | hr = SendBAMessage(pUserExperience, BOOTSTRAPPER_APPLICATION_MESSAGE_ONDETECTRELATEDBUNDLEPACKAGE, &args, &results); | ||
| 1273 | ExitOnFailure(hr, "BA OnDetectRelatedBundlePackage failed."); | ||
| 1274 | |||
| 1275 | if (results.fCancel) | ||
| 1276 | { | ||
| 1277 | hr = HRESULT_FROM_WIN32(ERROR_INSTALL_USEREXIT); | ||
| 1278 | } | ||
| 1279 | |||
| 1280 | LExit: | ||
| 1281 | return hr; | ||
| 1282 | } | ||
| 1283 | |||
| 1250 | EXTERN_C BAAPI UserExperienceOnDetectRelatedMsiPackage( | 1284 | EXTERN_C BAAPI UserExperienceOnDetectRelatedMsiPackage( |
| 1251 | __in BURN_USER_EXPERIENCE* pUserExperience, | 1285 | __in BURN_USER_EXPERIENCE* pUserExperience, |
| 1252 | __in_z LPCWSTR wzPackageId, | 1286 | __in_z LPCWSTR wzPackageId, |
diff --git a/src/burn/engine/userexperience.h b/src/burn/engine/userexperience.h index de558ad5..90a047ed 100644 --- a/src/burn/engine/userexperience.h +++ b/src/burn/engine/userexperience.h | |||
| @@ -298,6 +298,14 @@ BAAPI UserExperienceOnDetectRelatedBundle( | |||
| 298 | __in VERUTIL_VERSION* pVersion, | 298 | __in VERUTIL_VERSION* pVersion, |
| 299 | __in BOOL fMissingFromCache | 299 | __in BOOL fMissingFromCache |
| 300 | ); | 300 | ); |
| 301 | BAAPI UserExperienceOnDetectRelatedBundlePackage( | ||
| 302 | __in BURN_USER_EXPERIENCE* pUserExperience, | ||
| 303 | __in_z LPCWSTR wzPackageId, | ||
| 304 | __in_z LPCWSTR wzBundleId, | ||
| 305 | __in BOOTSTRAPPER_RELATION_TYPE relationType, | ||
| 306 | __in BOOL fPerMachine, | ||
| 307 | __in VERUTIL_VERSION* pVersion | ||
| 308 | ); | ||
| 301 | BAAPI UserExperienceOnDetectRelatedMsiPackage( | 309 | BAAPI UserExperienceOnDetectRelatedMsiPackage( |
| 302 | __in BURN_USER_EXPERIENCE* pUserExperience, | 310 | __in BURN_USER_EXPERIENCE* pUserExperience, |
| 303 | __in_z LPCWSTR wzPackageId, | 311 | __in_z LPCWSTR wzPackageId, |
diff --git a/src/burn/test/BurnUnitTest/TestData/PlanTest/BundlePackage_Multiple_manifest.xml b/src/burn/test/BurnUnitTest/TestData/PlanTest/BundlePackage_Multiple_manifest.xml index 6c60085f..0b521faa 100644 --- a/src/burn/test/BurnUnitTest/TestData/PlanTest/BundlePackage_Multiple_manifest.xml +++ b/src/burn/test/BurnUnitTest/TestData/PlanTest/BundlePackage_Multiple_manifest.xml | |||
| @@ -1 +1 @@ | |||
| <?xml version="1.0" encoding="utf-8"?><BurnManifest xmlns="http://wixtoolset.org/schemas/v4/2008/Burn"><Log PathVariable="WixBundleLog" Prefix="~BundlePackageTests_MultipleBundlePackagesBundle" Extension=".log" /><RelatedBundle Id="{86D214FB-8D74-456C-99B3-6557ECA6159C}" Action="Upgrade" /><Variable Id="TestGroupName" Value="BundlePackageTests" Type="string" Hidden="no" Persisted="no" /><Variable Id="WixBundleInProgressName" Hidden="no" Persisted="yes" /><Variable Id="WixBundleName" Hidden="no" Persisted="yes" /><Variable Id="WixBundleOriginalSource" Hidden="no" Persisted="yes" /><Variable Id="WixBundleOriginalSourceFolder" Hidden="no" Persisted="yes" /><Variable Id="WixBundleLastUsedSource" Hidden="no" Persisted="yes" /><RegistrySearch Id="NETFRAMEWORK45" Variable="NETFRAMEWORK45" Root="HKLM" Key="SOFTWARE\Microsoft\NET Framework Setup\NDP\v4\Full" Value="Release" Type="value" VariableType="string" /><UX><Payload Id="WixManagedBootstrapperApplicationHost" FilePath="mbahost.dll" SourcePath="u30" /><Payload Id="payO60IVK4ATGzPpMz3rwVbUWl6DyU" FilePath="WixToolset.Mba.Host.config" SourcePath="u0" /><Payload Id="payxj4zDAKL2NVlz4ohp0GvwFHepyI" FilePath="TestBA.dll" SourcePath="u1" /><Payload Id="pay1hOSAUC8_D633cD2TXpIXCL30OU" FilePath="mbanative.dll" SourcePath="u2" /><Payload Id="payujy6Izl_BlUNfHt2eI.ADfjYAv4" FilePath="WixToolset.Mba.Core.dll" SourcePath="u3" /><Payload Id="payR4EbR4OTDZpPEycWaSSM_gZRBWM" FilePath="mbapreq.thm" SourcePath="u4" /><Payload Id="paylVCy2Ecl8pHPdJTCQZryUG4T9us" FilePath="mbapreq.png" SourcePath="u5" /><Payload Id="payTaG4B_lob1aLcKFaOqSSG3MPMpU" FilePath="mbapreq.wxl" SourcePath="u6" /><Payload Id="payZwIGuiezVTitZOoZKxyh2DdRSGs" FilePath="1028\mbapreq.wxl" SourcePath="u7" /><Payload Id="pay.herBWX.LlOh8jLsx24aWdunV_0" FilePath="1029\mbapreq.wxl" SourcePath="u8" /><Payload Id="pay8DkMszYsoxxdgX14huLDMYXylQg" FilePath="1030\mbapreq.wxl" SourcePath="u9" /><Payload Id="payPaHpoTeOdkW.TK99IDwktNLhTAg" FilePath="1031\mbapreq.wxl" SourcePath="u10" /><Payload Id="pay45AtAzterLTMzZgdxxtuYvaiXwU" FilePath="1032\mbapreq.wxl" SourcePath="u11" /><Payload Id="payA2VEKIqhePyNIEmr14eyH3JoVLc" FilePath="1035\mbapreq.wxl" SourcePath="u12" /><Payload Id="payvre23ObscjzhcaFIifUAkXMdPa8" FilePath="1036\mbapreq.wxl" SourcePath="u13" /><Payload Id="paytxUV3vuBbG2c.a9c.d_sZX2x6wA" FilePath="1038\mbapreq.wxl" SourcePath="u14" /><Payload Id="payYvMWRK9xelo5.sQn7jRkJIaBp9A" FilePath="1040\mbapreq.wxl" SourcePath="u15" /><Payload Id="pay68KKSApyQimbA25t6kSbqhdeH10" FilePath="1041\mbapreq.wxl" SourcePath="u16" /><Payload Id="paypiqxaHpYZqx.9eDVjQrj1igLbRY" FilePath="1042\mbapreq.wxl" SourcePath="u17" /><Payload Id="payTO0YwZzxKpbqdrBVUcVRTu3BFe8" FilePath="1043\mbapreq.wxl" SourcePath="u18" /><Payload Id="payIXg2ldBJukRzhqWolJVOEbTmF34" FilePath="1044\mbapreq.wxl" SourcePath="u19" /><Payload Id="payOHIZbSkIvrpwKkkXI173tv3u3B4" FilePath="1045\mbapreq.wxl" SourcePath="u20" /><Payload Id="payQRQ_UZl_R2UtV0xDXB2yeH2bg3E" FilePath="1046\mbapreq.wxl" SourcePath="u21" /><Payload Id="payhrejLLBfc1i27iN._QPhQ4K337I" FilePath="1049\mbapreq.wxl" SourcePath="u22" /><Payload Id="payqEzaDNzxB68vGp29jgDcCos6dvg" FilePath="1051\mbapreq.wxl" SourcePath="u23" /><Payload Id="paydz8Vk8xSTyYohgGXTSIxWGXL5.Q" FilePath="1053\mbapreq.wxl" SourcePath="u24" /><Payload Id="pay0HRUZTlbC3taSOffJBsEj92Br8Y" FilePath="1055\mbapreq.wxl" SourcePath="u25" /><Payload Id="payIvUOkc_EMH7laMFehefNolV8hZo" FilePath="1060\mbapreq.wxl" SourcePath="u26" /><Payload Id="payLFhOb.rHuk4sW5CYAPMShG0NjGI" FilePath="2052\mbapreq.wxl" SourcePath="u27" /><Payload Id="payqIKCmERK7Nhxx_nNXvRxdKqKDbI" FilePath="2070\mbapreq.wxl" SourcePath="u28" /><Payload Id="payqeWUzIVaEqjuRXN0z8ECC3Y4tCc" FilePath="3082\mbapreq.wxl" SourcePath="u29" /><Payload Id="paylfeHEjJSSTnNzY9QMZM2Ye3Ipy4" FilePath="mbapreq.dll" SourcePath="u31" /><Payload Id="payDPxs6uy8nbky.R7zhir2RRAfc.c" FilePath="WixToolset.Mba.Host.dll" SourcePath="u32" /><Payload Id="uxTxMXPVMXwQrPTMIGa5WGt93w0Ns" FilePath="BootstrapperApplicationData.xml" SourcePath="u33" /><Payload Id="uxYRbgitOs0K878jn5L_z7LdJ21KI" FilePath="BundleExtensionData.xml" SourcePath="u34" /></UX><Container Id="WixAttachedContainer" FileSize="15696370" Hash="0F9966B421400E481D394DB4C4D7F0F92548E5BEB79B98880C926E817E8C1F381EC8A17053E2E66AE7132A3C9ECE441629E6A1FB3452C5C9282280C40252F141" FilePath="MultipleBundlePackagesBundle.exe" AttachedIndex="1" Attached="yes" Primary="yes" /><Payload Id="NetFx48Web" FilePath="redist\ndp48-web.exe" FileSize="1439328" CertificateRootPublicKeyIdentifier="F49F9B33E25E33CCA0BFB15A62B7C29FFAB3880B" CertificateRootThumbprint="ABDCA79AF9DD48A0EA702AD45260B3C03093FB4B" DownloadUrl="https://go.microsoft.com/fwlink/?LinkId=2085155" Packaging="external" SourcePath="redist\ndp48-web.exe" /><Payload Id="PackageA" FilePath="BundleA.exe" FileSize="5241635" Hash="20E1AFF76DE4693CB2876DC6BCCA0152DB16BE49AEDE2CD581C03FC39AB89DEA12BC25CB435F06E4D7D2B4443CE8A8935D5E92E2E49A4981B60A273980E4B29B" Packaging="embedded" SourcePath="a0" Container="WixAttachedContainer" /><Payload Id="PackageB" FilePath="BundleB_x64.exe" FileSize="10450821" Hash="43A58873D61D6E0FA83F6C5266F2F05FEA9BC85D11C195493B7FD9F0B4AA799C1EFCB78D76DCED32124D2EC62A4E7114B62CDE6F0B87E42A7E28CDBB0DA0FF8E" Packaging="embedded" SourcePath="a1" Container="WixAttachedContainer" /><RollbackBoundary Id="WixDefaultBoundary" Vital="yes" Transaction="no" /><Registration Id="{35192ED0-C70A-49B2-9D12-3B1FA39B5E6F}" ExecutableName="MultipleBundlePackagesBundle.exe" PerMachine="yes" Tag="" Version="1.0.0.0" ProviderKey="{35192ED0-C70A-49B2-9D12-3B1FA39B5E6F}"><Arp Register="yes" DisplayName="~BundlePackageTests - MultipleBundlePackagesBundle" DisplayVersion="1.0.0.0" /></Registration><Chain><ExePackage Id="NetFx48Web" Cache="remove" CacheId="642721C60D52051C7F3434D8710FE3406A7CFE10B2B39E90EA847719ED1697D7C614F2DF44AD50412B1DF8C98DD78FDC57CA1D047D28C81AC158092E5FB18040" InstallSize="1439328" Size="1439328" PerMachine="yes" Permanent="yes" Vital="yes" RollbackBoundaryForward="WixDefaultBoundary" LogPathVariable="NetFx48WebLog" RollbackLogPathVariable="WixBundleRollbackLog_NetFx48Web" DetectCondition="NETFRAMEWORK45 >= 528040" InstallArguments="/q /norestart /log "[NetFx48WebLog].html"" UninstallArguments="" Uninstallable="no" RepairArguments="" Repairable="no" Protocol="netfx4"><PayloadRef Id="NetFx48Web" /></ExePackage><BundlePackage Id="PackageA" Cache="keep" CacheId="{B39CEE4D-CCD7-4797-BE3A-6613BD1DC4BE}v1.0.0.0" InstallSize="2169" Size="5241635" PerMachine="yes" Permanent="no" Vital="yes" LogPathVariable="WixBundleLog_PackageA" RollbackLogPathVariable="WixBundleRollbackLog_PackageA" BundleId="{B39CEE4D-CCD7-4797-BE3A-6613BD1DC4BE}" InstallArguments="" UninstallArguments="" RepairArguments="" SupportsBurnProtocol="yes" Win64="no"><Provides Key="{B39CEE4D-CCD7-4797-BE3A-6613BD1DC4BE}" Version="1.0.0.0" DisplayName="~BasicFunctionalityTests - BundleA" Imported="yes" /><PayloadRef Id="PackageA" /></BundlePackage><BundlePackage Id="PackageB" Cache="keep" CacheId="{7506235A-7C59-4750-82C7-EB460A87ED3A}v1.0.0.0" InstallSize="1441497" Size="10450821" PerMachine="yes" Permanent="no" Vital="yes" RollbackBoundaryBackward="WixDefaultBoundary" LogPathVariable="WixBundleLog_PackageB" RollbackLogPathVariable="WixBundleRollbackLog_PackageB" BundleId="{7506235A-7C59-4750-82C7-EB460A87ED3A}" InstallArguments="" UninstallArguments="" RepairArguments="" SupportsBurnProtocol="yes" Win64="yes"><Provides Key="{7506235A-7C59-4750-82C7-EB460A87ED3A}" Version="1.0.0.0" DisplayName="~BasicFunctionalityTests - BundleB_x64" Imported="yes" /><PayloadRef Id="PackageB" /></BundlePackage></Chain><CommandLine Variables="upperCase" /></BurnManifest> \ No newline at end of file | <?xml version="1.0" encoding="utf-8"?><BurnManifest xmlns="http://wixtoolset.org/schemas/v4/2008/Burn"><Log PathVariable="WixBundleLog" Prefix="~BundlePackageTests_MultipleBundlePackagesBundle" Extension=".log" /><RelatedBundle Id="{86D214FB-8D74-456C-99B3-6557ECA6159C}" Action="Upgrade" /><Variable Id="TestGroupName" Value="BundlePackageTests" Type="string" Hidden="no" Persisted="no" /><Variable Id="WixBundleInProgressName" Hidden="no" Persisted="yes" /><Variable Id="WixBundleName" Hidden="no" Persisted="yes" /><Variable Id="WixBundleOriginalSource" Hidden="no" Persisted="yes" /><Variable Id="WixBundleOriginalSourceFolder" Hidden="no" Persisted="yes" /><Variable Id="WixBundleLastUsedSource" Hidden="no" Persisted="yes" /><RegistrySearch Id="NETFRAMEWORK45" Variable="NETFRAMEWORK45" Root="HKLM" Key="SOFTWARE\Microsoft\NET Framework Setup\NDP\v4\Full" Value="Release" Type="value" VariableType="string" /><UX><Payload Id="WixManagedBootstrapperApplicationHost" FilePath="mbahost.dll" SourcePath="u30" /><Payload Id="payO60IVK4ATGzPpMz3rwVbUWl6DyU" FilePath="WixToolset.Mba.Host.config" SourcePath="u0" /><Payload Id="payxj4zDAKL2NVlz4ohp0GvwFHepyI" FilePath="TestBA.dll" SourcePath="u1" /><Payload Id="pay1hOSAUC8_D633cD2TXpIXCL30OU" FilePath="mbanative.dll" SourcePath="u2" /><Payload Id="payujy6Izl_BlUNfHt2eI.ADfjYAv4" FilePath="WixToolset.Mba.Core.dll" SourcePath="u3" /><Payload Id="payR4EbR4OTDZpPEycWaSSM_gZRBWM" FilePath="mbapreq.thm" SourcePath="u4" /><Payload Id="paylVCy2Ecl8pHPdJTCQZryUG4T9us" FilePath="mbapreq.png" SourcePath="u5" /><Payload Id="payTaG4B_lob1aLcKFaOqSSG3MPMpU" FilePath="mbapreq.wxl" SourcePath="u6" /><Payload Id="payZwIGuiezVTitZOoZKxyh2DdRSGs" FilePath="1028\mbapreq.wxl" SourcePath="u7" /><Payload Id="pay.herBWX.LlOh8jLsx24aWdunV_0" FilePath="1029\mbapreq.wxl" SourcePath="u8" /><Payload Id="pay8DkMszYsoxxdgX14huLDMYXylQg" FilePath="1030\mbapreq.wxl" SourcePath="u9" /><Payload Id="payPaHpoTeOdkW.TK99IDwktNLhTAg" FilePath="1031\mbapreq.wxl" SourcePath="u10" /><Payload Id="pay45AtAzterLTMzZgdxxtuYvaiXwU" FilePath="1032\mbapreq.wxl" SourcePath="u11" /><Payload Id="payA2VEKIqhePyNIEmr14eyH3JoVLc" FilePath="1035\mbapreq.wxl" SourcePath="u12" /><Payload Id="payvre23ObscjzhcaFIifUAkXMdPa8" FilePath="1036\mbapreq.wxl" SourcePath="u13" /><Payload Id="paytxUV3vuBbG2c.a9c.d_sZX2x6wA" FilePath="1038\mbapreq.wxl" SourcePath="u14" /><Payload Id="payYvMWRK9xelo5.sQn7jRkJIaBp9A" FilePath="1040\mbapreq.wxl" SourcePath="u15" /><Payload Id="pay68KKSApyQimbA25t6kSbqhdeH10" FilePath="1041\mbapreq.wxl" SourcePath="u16" /><Payload Id="paypiqxaHpYZqx.9eDVjQrj1igLbRY" FilePath="1042\mbapreq.wxl" SourcePath="u17" /><Payload Id="payTO0YwZzxKpbqdrBVUcVRTu3BFe8" FilePath="1043\mbapreq.wxl" SourcePath="u18" /><Payload Id="payIXg2ldBJukRzhqWolJVOEbTmF34" FilePath="1044\mbapreq.wxl" SourcePath="u19" /><Payload Id="payOHIZbSkIvrpwKkkXI173tv3u3B4" FilePath="1045\mbapreq.wxl" SourcePath="u20" /><Payload Id="payQRQ_UZl_R2UtV0xDXB2yeH2bg3E" FilePath="1046\mbapreq.wxl" SourcePath="u21" /><Payload Id="payhrejLLBfc1i27iN._QPhQ4K337I" FilePath="1049\mbapreq.wxl" SourcePath="u22" /><Payload Id="payqEzaDNzxB68vGp29jgDcCos6dvg" FilePath="1051\mbapreq.wxl" SourcePath="u23" /><Payload Id="paydz8Vk8xSTyYohgGXTSIxWGXL5.Q" FilePath="1053\mbapreq.wxl" SourcePath="u24" /><Payload Id="pay0HRUZTlbC3taSOffJBsEj92Br8Y" FilePath="1055\mbapreq.wxl" SourcePath="u25" /><Payload Id="payIvUOkc_EMH7laMFehefNolV8hZo" FilePath="1060\mbapreq.wxl" SourcePath="u26" /><Payload Id="payLFhOb.rHuk4sW5CYAPMShG0NjGI" FilePath="2052\mbapreq.wxl" SourcePath="u27" /><Payload Id="payqIKCmERK7Nhxx_nNXvRxdKqKDbI" FilePath="2070\mbapreq.wxl" SourcePath="u28" /><Payload Id="payqeWUzIVaEqjuRXN0z8ECC3Y4tCc" FilePath="3082\mbapreq.wxl" SourcePath="u29" /><Payload Id="paylfeHEjJSSTnNzY9QMZM2Ye3Ipy4" FilePath="mbapreq.dll" SourcePath="u31" /><Payload Id="payDPxs6uy8nbky.R7zhir2RRAfc.c" FilePath="WixToolset.Mba.Host.dll" SourcePath="u32" /><Payload Id="uxTxMXPVMXwQrPTMIGa5WGt93w0Ns" FilePath="BootstrapperApplicationData.xml" SourcePath="u33" /><Payload Id="uxYRbgitOs0K878jn5L_z7LdJ21KI" FilePath="BundleExtensionData.xml" SourcePath="u34" /></UX><Container Id="WixAttachedContainer" FileSize="15696370" Hash="0F9966B421400E481D394DB4C4D7F0F92548E5BEB79B98880C926E817E8C1F381EC8A17053E2E66AE7132A3C9ECE441629E6A1FB3452C5C9282280C40252F141" FilePath="MultipleBundlePackagesBundle.exe" AttachedIndex="1" Attached="yes" Primary="yes" /><Payload Id="NetFx48Web" FilePath="redist\ndp48-web.exe" FileSize="1439328" CertificateRootPublicKeyIdentifier="F49F9B33E25E33CCA0BFB15A62B7C29FFAB3880B" CertificateRootThumbprint="ABDCA79AF9DD48A0EA702AD45260B3C03093FB4B" DownloadUrl="https://go.microsoft.com/fwlink/?LinkId=2085155" Packaging="external" SourcePath="redist\ndp48-web.exe" /><Payload Id="PackageA" FilePath="BundleA.exe" FileSize="5241635" Hash="20E1AFF76DE4693CB2876DC6BCCA0152DB16BE49AEDE2CD581C03FC39AB89DEA12BC25CB435F06E4D7D2B4443CE8A8935D5E92E2E49A4981B60A273980E4B29B" Packaging="embedded" SourcePath="a0" Container="WixAttachedContainer" /><Payload Id="PackageB" FilePath="BundleB_x64.exe" FileSize="10450821" Hash="43A58873D61D6E0FA83F6C5266F2F05FEA9BC85D11C195493B7FD9F0B4AA799C1EFCB78D76DCED32124D2EC62A4E7114B62CDE6F0B87E42A7E28CDBB0DA0FF8E" Packaging="embedded" SourcePath="a1" Container="WixAttachedContainer" /><RollbackBoundary Id="WixDefaultBoundary" Vital="yes" Transaction="no" /><Registration Id="{35192ED0-C70A-49B2-9D12-3B1FA39B5E6F}" ExecutableName="MultipleBundlePackagesBundle.exe" PerMachine="yes" Tag="" Version="1.0.0.0" ProviderKey="{35192ED0-C70A-49B2-9D12-3B1FA39B5E6F}"><Arp Register="yes" DisplayName="~BundlePackageTests - MultipleBundlePackagesBundle" DisplayVersion="1.0.0.0" /></Registration><Chain><ExePackage Id="NetFx48Web" Cache="remove" CacheId="642721C60D52051C7F3434D8710FE3406A7CFE10B2B39E90EA847719ED1697D7C614F2DF44AD50412B1DF8C98DD78FDC57CA1D047D28C81AC158092E5FB18040" InstallSize="1439328" Size="1439328" PerMachine="yes" Permanent="yes" Vital="yes" RollbackBoundaryForward="WixDefaultBoundary" LogPathVariable="NetFx48WebLog" RollbackLogPathVariable="WixBundleRollbackLog_NetFx48Web" DetectCondition="NETFRAMEWORK45 >= 528040" InstallArguments="/q /norestart /log "[NetFx48WebLog].html"" UninstallArguments="" Uninstallable="no" RepairArguments="" Repairable="no" Protocol="netfx4"><PayloadRef Id="NetFx48Web" /></ExePackage><BundlePackage Id="PackageA" Cache="keep" CacheId="{B39CEE4D-CCD7-4797-BE3A-6613BD1DC4BE}v1.0.0.0" InstallSize="2169" Size="5241635" Version="1.0.0.0" PerMachine="yes" Permanent="no" Vital="yes" LogPathVariable="WixBundleLog_PackageA" RollbackLogPathVariable="WixBundleRollbackLog_PackageA" BundleId="{B39CEE4D-CCD7-4797-BE3A-6613BD1DC4BE}" InstallArguments="" UninstallArguments="" RepairArguments="" SupportsBurnProtocol="yes" Win64="no"><Provides Key="{B39CEE4D-CCD7-4797-BE3A-6613BD1DC4BE}" Version="1.0.0.0" DisplayName="~BasicFunctionalityTests - BundleA" Imported="yes" /><RelatedBundle Id="{8C7E2C47-1EE7-4BBE-99A2-EAB7F3693F48}" Action="Upgrade" /><PayloadRef Id="PackageA" /></BundlePackage><BundlePackage Id="PackageB" Cache="keep" CacheId="{7506235A-7C59-4750-82C7-EB460A87ED3A}v1.0.0.0" InstallSize="1441497" Size="10450821" Version="1.0.0.0" PerMachine="yes" Permanent="no" Vital="yes" RollbackBoundaryBackward="WixDefaultBoundary" LogPathVariable="WixBundleLog_PackageB" RollbackLogPathVariable="WixBundleRollbackLog_PackageB" BundleId="{7506235A-7C59-4750-82C7-EB460A87ED3A}" InstallArguments="" UninstallArguments="" RepairArguments="" SupportsBurnProtocol="yes" Win64="yes"><Provides Key="{7506235A-7C59-4750-82C7-EB460A87ED3A}" Version="1.0.0.0" DisplayName="~BasicFunctionalityTests - BundleB_x64" Imported="yes" /><RelatedBundle Id="{79F45B7A-D990-46E4-819B-078D87C3321A}" Action="Upgrade" /><PayloadRef Id="PackageB" /></BundlePackage></Chain><CommandLine Variables="upperCase" /></BurnManifest> | ||
diff --git a/src/ext/Bal/wixstdba/WixStandardBootstrapperApplication.cpp b/src/ext/Bal/wixstdba/WixStandardBootstrapperApplication.cpp index 444a917f..969aca9d 100644 --- a/src/ext/Bal/wixstdba/WixStandardBootstrapperApplication.cpp +++ b/src/ext/Bal/wixstdba/WixStandardBootstrapperApplication.cpp | |||
| @@ -1489,6 +1489,9 @@ public: // IBootstrapperApplication | |||
| 1489 | case BOOTSTRAPPER_APPLICATION_MESSAGE_ONAPPLYDOWNGRADE: | 1489 | case BOOTSTRAPPER_APPLICATION_MESSAGE_ONAPPLYDOWNGRADE: |
| 1490 | OnApplyDowngradeFallback(reinterpret_cast<BA_ONAPPLYDOWNGRADE_ARGS*>(pvArgs), reinterpret_cast<BA_ONAPPLYDOWNGRADE_RESULTS*>(pvResults)); | 1490 | OnApplyDowngradeFallback(reinterpret_cast<BA_ONAPPLYDOWNGRADE_ARGS*>(pvArgs), reinterpret_cast<BA_ONAPPLYDOWNGRADE_RESULTS*>(pvResults)); |
| 1491 | break; | 1491 | break; |
| 1492 | case BOOTSTRAPPER_APPLICATION_MESSAGE_ONDETECTRELATEDBUNDLEPACKAGE: | ||
| 1493 | OnDetectRelatedBundlePackageFallback(reinterpret_cast<BA_ONDETECTRELATEDBUNDLEPACKAGE_ARGS*>(pvArgs), reinterpret_cast<BA_ONDETECTRELATEDBUNDLEPACKAGE_RESULTS*>(pvResults)); | ||
| 1494 | break; | ||
| 1492 | default: | 1495 | default: |
| 1493 | #ifdef DEBUG | 1496 | #ifdef DEBUG |
| 1494 | BalLog(BOOTSTRAPPER_LOG_LEVEL_STANDARD, "WIXSTDBA: Forwarding unknown BA message: %d", message); | 1497 | BalLog(BOOTSTRAPPER_LOG_LEVEL_STANDARD, "WIXSTDBA: Forwarding unknown BA message: %d", message); |
| @@ -2178,6 +2181,14 @@ private: // privates | |||
| 2178 | m_pfnBAFunctionsProc(BA_FUNCTIONS_MESSAGE_ONEXECUTEPROCESSCANCEL, pArgs, pResults, m_pvBAFunctionsProcContext); | 2181 | m_pfnBAFunctionsProc(BA_FUNCTIONS_MESSAGE_ONEXECUTEPROCESSCANCEL, pArgs, pResults, m_pvBAFunctionsProcContext); |
| 2179 | } | 2182 | } |
| 2180 | 2183 | ||
| 2184 | void OnDetectRelatedBundlePackageFallback( | ||
| 2185 | __in BA_ONDETECTRELATEDBUNDLEPACKAGE_ARGS* pArgs, | ||
| 2186 | __inout BA_ONDETECTRELATEDBUNDLEPACKAGE_RESULTS* pResults | ||
| 2187 | ) | ||
| 2188 | { | ||
| 2189 | m_pfnBAFunctionsProc(BA_FUNCTIONS_MESSAGE_ONDETECTRELATEDBUNDLEPACKAGE, pArgs, pResults, m_pvBAFunctionsProcContext); | ||
| 2190 | } | ||
| 2191 | |||
| 2181 | 2192 | ||
| 2182 | public: //CBalBaseBootstrapperApplication | 2193 | public: //CBalBaseBootstrapperApplication |
| 2183 | virtual STDMETHODIMP Initialize( | 2194 | virtual STDMETHODIMP Initialize( |
diff --git a/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundle.props b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundle.props new file mode 100644 index 00000000..1c761dfc --- /dev/null +++ b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundle.props | |||
| @@ -0,0 +1,14 @@ | |||
| 1 | <!-- Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information. --> | ||
| 2 | <Project> | ||
| 3 | <PropertyGroup> | ||
| 4 | <OutputType>Bundle</OutputType> | ||
| 5 | <BA>hyperlinkLicense</BA> | ||
| 6 | <UpgradeCode>{B9636854-B76B-4171-B63D-7C659F61DE3D}</UpgradeCode> | ||
| 7 | </PropertyGroup> | ||
| 8 | <ItemGroup> | ||
| 9 | <Compile Include="..\..\Templates\Bundle.wxs" Link="Bundle.wxs" /> | ||
| 10 | </ItemGroup> | ||
| 11 | <ItemGroup> | ||
| 12 | <PackageReference Include="WixToolset.Bal.wixext" /> | ||
| 13 | </ItemGroup> | ||
| 14 | </Project> | ||
diff --git a/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundlev1.wixproj b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundlev1.wixproj new file mode 100644 index 00000000..e1e0d601 --- /dev/null +++ b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundlev1.wixproj | |||
| @@ -0,0 +1,7 @@ | |||
| 1 | <!-- Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information. --> | ||
| 2 | <Project Sdk="WixToolset.Sdk"> | ||
| 3 | <Import Project="UpgradeBundlePackageBundle.props" /> | ||
| 4 | <ItemGroup> | ||
| 5 | <ProjectReference Include="..\..\UpgradeRelatedBundleTests\BundleAv1\BundleAv1.wixproj" /> | ||
| 6 | </ItemGroup> | ||
| 7 | </Project> \ No newline at end of file | ||
diff --git a/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundlev1.wxs b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundlev1.wxs new file mode 100644 index 00000000..ae70ebdb --- /dev/null +++ b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev1/UpgradeBundlePackageBundlev1.wxs | |||
| @@ -0,0 +1,10 @@ | |||
| 1 | <!-- Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information. --> | ||
| 2 | |||
| 3 | |||
| 4 | <Wix xmlns="http://wixtoolset.org/schemas/v4/wxs"> | ||
| 5 | <Fragment> | ||
| 6 | <PackageGroup Id="BundlePackages"> | ||
| 7 | <BundlePackage Id="PackageA" SourceFile="$(var.BundleAv1.TargetPath)" /> | ||
| 8 | </PackageGroup> | ||
| 9 | </Fragment> | ||
| 10 | </Wix> | ||
diff --git a/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev2/UpgradeBundlePackageBundlev2.wixproj b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev2/UpgradeBundlePackageBundlev2.wixproj new file mode 100644 index 00000000..61247a66 --- /dev/null +++ b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev2/UpgradeBundlePackageBundlev2.wixproj | |||
| @@ -0,0 +1,7 @@ | |||
| 1 | <!-- Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information. --> | ||
| 2 | <Project Sdk="WixToolset.Sdk"> | ||
| 3 | <Import Project="..\UpgradeBundlePackageBundlev1\UpgradeBundlePackageBundle.props" /> | ||
| 4 | <ItemGroup> | ||
| 5 | <ProjectReference Include="..\..\UpgradeRelatedBundleTests\BundleAv2\BundleAv2.wixproj" /> | ||
| 6 | </ItemGroup> | ||
| 7 | </Project> \ No newline at end of file | ||
diff --git a/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev2/UpgradeBundlePackageBundlev2.wxs b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev2/UpgradeBundlePackageBundlev2.wxs new file mode 100644 index 00000000..742881c7 --- /dev/null +++ b/src/test/burn/TestData/BundlePackageTests/UpgradeBundlePackageBundlev2/UpgradeBundlePackageBundlev2.wxs | |||
| @@ -0,0 +1,10 @@ | |||
| 1 | <!-- Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information. --> | ||
| 2 | |||
| 3 | |||
| 4 | <Wix xmlns="http://wixtoolset.org/schemas/v4/wxs"> | ||
| 5 | <Fragment> | ||
| 6 | <PackageGroup Id="BundlePackages"> | ||
| 7 | <BundlePackage Id="PackageA" SourceFile="$(var.BundleAv2.TargetPath)" /> | ||
| 8 | </PackageGroup> | ||
| 9 | </Fragment> | ||
| 10 | </Wix> | ||
diff --git a/src/test/burn/WixToolsetTest.BurnE2E/BundlePackageTests.cs b/src/test/burn/WixToolsetTest.BurnE2E/BundlePackageTests.cs index 2e95aedb..1e6cda9c 100644 --- a/src/test/burn/WixToolsetTest.BurnE2E/BundlePackageTests.cs +++ b/src/test/burn/WixToolsetTest.BurnE2E/BundlePackageTests.cs | |||
| @@ -45,7 +45,37 @@ namespace WixToolsetTest.BurnE2E | |||
| 45 | // Source file should *not* be installed | 45 | // Source file should *not* be installed |
| 46 | Assert.False(File.Exists(packageA32SourceCodeFilePath), $"PackageA payload should have been removed by uninstall from: {packageA32SourceCodeFilePath}"); | 46 | Assert.False(File.Exists(packageA32SourceCodeFilePath), $"PackageA payload should have been removed by uninstall from: {packageA32SourceCodeFilePath}"); |
| 47 | Assert.False(File.Exists(packageA64SourceCodeFilePath), $"PackageA_x64 payload should have been removed by uninstall from: {packageA64SourceCodeFilePath}"); | 47 | Assert.False(File.Exists(packageA64SourceCodeFilePath), $"PackageA_x64 payload should have been removed by uninstall from: {packageA64SourceCodeFilePath}"); |
| 48 | } | ||
| 49 | |||
| 50 | [Fact] | ||
| 51 | public void CanInstallUpgradeBundlePackage() | ||
| 52 | { | ||
| 53 | var bundleAv1 = this.CreateBundleInstaller(@"..\UpgradeRelatedBundleTests\BundleAv1"); | ||
| 54 | var bundleAv2 = this.CreateBundleInstaller(@"..\UpgradeRelatedBundleTests\BundleAv2"); | ||
| 55 | var upgradeBundlePackageBundlev2 = this.CreateBundleInstaller("UpgradeBundlePackageBundlev2"); | ||
| 56 | |||
| 57 | bundleAv1.Install(); | ||
| 58 | bundleAv1.VerifyRegisteredAndInPackageCache(); | ||
| 59 | |||
| 60 | upgradeBundlePackageBundlev2.Install(); | ||
| 61 | upgradeBundlePackageBundlev2.VerifyRegisteredAndInPackageCache(); | ||
| 62 | bundleAv2.VerifyRegisteredAndInPackageCache(); | ||
| 63 | bundleAv1.VerifyUnregisteredAndRemovedFromPackageCache(); | ||
| 64 | } | ||
| 65 | |||
| 66 | [Fact] | ||
| 67 | public void CanSkipObsoleteBundlePackage() | ||
| 68 | { | ||
| 69 | var bundleAv1 = this.CreateBundleInstaller(@"..\UpgradeRelatedBundleTests\BundleAv1"); | ||
| 70 | var bundleAv2 = this.CreateBundleInstaller(@"..\UpgradeRelatedBundleTests\BundleAv2"); | ||
| 71 | var upgradeBundlePackageBundlev1 = this.CreateBundleInstaller("UpgradeBundlePackageBundlev1"); | ||
| 72 | |||
| 73 | bundleAv2.Install(); | ||
| 74 | bundleAv2.VerifyRegisteredAndInPackageCache(); | ||
| 48 | 75 | ||
| 76 | upgradeBundlePackageBundlev1.Install(); | ||
| 77 | upgradeBundlePackageBundlev1.VerifyUnregisteredAndRemovedFromPackageCache(); | ||
| 78 | bundleAv1.VerifyUnregisteredAndRemovedFromPackageCache(); | ||
| 49 | } | 79 | } |
| 50 | } | 80 | } |
| 51 | } | 81 | } |
diff --git a/src/wix/WixToolset.Core.Burn/Bind/GenerateManifestDataFromIRCommand.cs b/src/wix/WixToolset.Core.Burn/Bind/GenerateManifestDataFromIRCommand.cs index 6b37580e..9d214211 100644 --- a/src/wix/WixToolset.Core.Burn/Bind/GenerateManifestDataFromIRCommand.cs +++ b/src/wix/WixToolset.Core.Burn/Bind/GenerateManifestDataFromIRCommand.cs | |||
| @@ -82,6 +82,7 @@ namespace WixToolset.Core.Burn.Bind | |||
| 82 | case SymbolDefinitionType.WixBundlePackageCommandLine: | 82 | case SymbolDefinitionType.WixBundlePackageCommandLine: |
| 83 | case SymbolDefinitionType.WixBundlePackageExitCode: | 83 | case SymbolDefinitionType.WixBundlePackageExitCode: |
| 84 | case SymbolDefinitionType.WixBundlePackageGroup: | 84 | case SymbolDefinitionType.WixBundlePackageGroup: |
| 85 | case SymbolDefinitionType.WixBundlePackageRelatedBundle: | ||
| 85 | case SymbolDefinitionType.WixBundlePatchTargetCode: | 86 | case SymbolDefinitionType.WixBundlePatchTargetCode: |
| 86 | case SymbolDefinitionType.WixBundlePayload: | 87 | case SymbolDefinitionType.WixBundlePayload: |
| 87 | case SymbolDefinitionType.WixBundlePayloadGroup: | 88 | case SymbolDefinitionType.WixBundlePayloadGroup: |
diff --git a/src/wix/WixToolset.Core.Burn/Bundles/CreateBurnManifestCommand.cs b/src/wix/WixToolset.Core.Burn/Bundles/CreateBurnManifestCommand.cs index 84923998..c0e54300 100644 --- a/src/wix/WixToolset.Core.Burn/Bundles/CreateBurnManifestCommand.cs +++ b/src/wix/WixToolset.Core.Burn/Bundles/CreateBurnManifestCommand.cs | |||
| @@ -319,6 +319,7 @@ namespace WixToolset.Core.Burn.Bundles | |||
| 319 | var targetCodesByPatch = this.Section.Symbols.OfType<WixBundlePatchTargetCodeSymbol>().ToLookup(r => r.PackageRef); | 319 | var targetCodesByPatch = this.Section.Symbols.OfType<WixBundlePatchTargetCodeSymbol>().ToLookup(r => r.PackageRef); |
| 320 | var msiFeaturesByPackage = this.Section.Symbols.OfType<WixBundleMsiFeatureSymbol>().ToLookup(r => r.PackageRef); | 320 | var msiFeaturesByPackage = this.Section.Symbols.OfType<WixBundleMsiFeatureSymbol>().ToLookup(r => r.PackageRef); |
| 321 | var msiPropertiesByPackage = this.Section.Symbols.OfType<WixBundleMsiPropertySymbol>().ToLookup(r => r.PackageRef); | 321 | var msiPropertiesByPackage = this.Section.Symbols.OfType<WixBundleMsiPropertySymbol>().ToLookup(r => r.PackageRef); |
| 322 | var relatedBundlesByPackage = this.Section.Symbols.OfType<WixBundlePackageRelatedBundleSymbol>().ToLookup(r => r.PackageRef); | ||
| 322 | var relatedPackagesByPackage = this.Section.Symbols.OfType<WixBundleRelatedPackageSymbol>().ToLookup(r => r.PackageRef); | 323 | var relatedPackagesByPackage = this.Section.Symbols.OfType<WixBundleRelatedPackageSymbol>().ToLookup(r => r.PackageRef); |
| 323 | var slipstreamMspsByPackage = this.Section.Symbols.OfType<WixBundleSlipstreamMspSymbol>().ToLookup(r => r.TargetPackageRef); | 324 | var slipstreamMspsByPackage = this.Section.Symbols.OfType<WixBundleSlipstreamMspSymbol>().ToLookup(r => r.TargetPackageRef); |
| 324 | var exitCodesByPackage = this.Section.Symbols.OfType<WixBundlePackageExitCodeSymbol>().ToLookup(r => r.ChainPackageId); | 325 | var exitCodesByPackage = this.Section.Symbols.OfType<WixBundlePackageExitCodeSymbol>().ToLookup(r => r.ChainPackageId); |
| @@ -384,6 +385,7 @@ namespace WixToolset.Core.Burn.Bundles | |||
| 384 | if (package.SpecificPackageSymbol is WixBundleBundlePackageSymbol bundlePackage) // BUNDLE | 385 | if (package.SpecificPackageSymbol is WixBundleBundlePackageSymbol bundlePackage) // BUNDLE |
| 385 | { | 386 | { |
| 386 | writer.WriteAttributeString("BundleId", bundlePackage.BundleId); | 387 | writer.WriteAttributeString("BundleId", bundlePackage.BundleId); |
| 388 | writer.WriteAttributeString("Version", bundlePackage.Version); | ||
| 387 | writer.WriteAttributeString("InstallArguments", bundlePackage.InstallCommand); | 389 | writer.WriteAttributeString("InstallArguments", bundlePackage.InstallCommand); |
| 388 | writer.WriteAttributeString("UninstallArguments", bundlePackage.UninstallCommand); | 390 | writer.WriteAttributeString("UninstallArguments", bundlePackage.UninstallCommand); |
| 389 | writer.WriteAttributeString("RepairArguments", bundlePackage.RepairCommand); | 391 | writer.WriteAttributeString("RepairArguments", bundlePackage.RepairCommand); |
| @@ -536,6 +538,16 @@ namespace WixToolset.Core.Burn.Bundles | |||
| 536 | writer.WriteEndElement(); | 538 | writer.WriteEndElement(); |
| 537 | } | 539 | } |
| 538 | 540 | ||
| 541 | var packageRelatedBundles = relatedBundlesByPackage[package.PackageId]; | ||
| 542 | |||
| 543 | foreach (var relatedBundle in packageRelatedBundles) | ||
| 544 | { | ||
| 545 | writer.WriteStartElement("RelatedBundle"); | ||
| 546 | writer.WriteAttributeString("Id", relatedBundle.BundleId); | ||
| 547 | writer.WriteAttributeString("Action", relatedBundle.Action.ToString()); | ||
| 548 | writer.WriteEndElement(); | ||
| 549 | } | ||
| 550 | |||
| 539 | var packageRelatedPackages = relatedPackagesByPackage[package.PackageId]; | 551 | var packageRelatedPackages = relatedPackagesByPackage[package.PackageId]; |
| 540 | 552 | ||
| 541 | foreach (var related in packageRelatedPackages) | 553 | foreach (var related in packageRelatedPackages) |
diff --git a/src/wix/WixToolset.Core.Burn/Bundles/ProcessBundlePackageCommand.cs b/src/wix/WixToolset.Core.Burn/Bundles/ProcessBundlePackageCommand.cs index fdb07a56..e8c68faa 100644 --- a/src/wix/WixToolset.Core.Burn/Bundles/ProcessBundlePackageCommand.cs +++ b/src/wix/WixToolset.Core.Burn/Bundles/ProcessBundlePackageCommand.cs | |||
| @@ -102,6 +102,8 @@ namespace WixToolset.Core.Burn.Bundles | |||
| 102 | 102 | ||
| 103 | var version = registrationElement.GetAttribute("Version"); | 103 | var version = registrationElement.GetAttribute("Version"); |
| 104 | packagePayload.Version = version; | 104 | packagePayload.Version = version; |
| 105 | bundlePackage.Version = version; | ||
| 106 | this.Facade.PackageSymbol.Version = version; | ||
| 105 | 107 | ||
| 106 | if (String.IsNullOrEmpty(this.Facade.PackageSymbol.CacheId)) | 108 | if (String.IsNullOrEmpty(this.Facade.PackageSymbol.CacheId)) |
| 107 | { | 109 | { |
| @@ -125,6 +127,8 @@ namespace WixToolset.Core.Burn.Bundles | |||
| 125 | 127 | ||
| 126 | this.ProcessPackages(document, namespaceManager); | 128 | this.ProcessPackages(document, namespaceManager); |
| 127 | 129 | ||
| 130 | this.ProcessRelatedBundles(document, namespaceManager); | ||
| 131 | |||
| 128 | // TODO: Add payloads? | 132 | // TODO: Add payloads? |
| 129 | } | 133 | } |
| 130 | } | 134 | } |
| @@ -148,5 +152,26 @@ namespace WixToolset.Core.Burn.Bundles | |||
| 148 | 152 | ||
| 149 | this.Facade.PackageSymbol.InstallSize = packageInstallSize; | 153 | this.Facade.PackageSymbol.InstallSize = packageInstallSize; |
| 150 | } | 154 | } |
| 155 | |||
| 156 | private void ProcessRelatedBundles(XmlDocument document, XmlNamespaceManager namespaceManager) | ||
| 157 | { | ||
| 158 | foreach (XmlElement relatedBundleElement in document.SelectNodes("/burn:BurnManifest/burn:RelatedBundle", namespaceManager)) | ||
| 159 | { | ||
| 160 | var id = relatedBundleElement.GetAttribute("Id"); | ||
| 161 | |||
| 162 | if (!Enum.TryParse(relatedBundleElement.GetAttribute("Action"), out RelatedBundleActionType action)) | ||
| 163 | { | ||
| 164 | // TODO: warning | ||
| 165 | continue; | ||
| 166 | } | ||
| 167 | |||
| 168 | this.Section.AddSymbol(new WixBundlePackageRelatedBundleSymbol | ||
| 169 | { | ||
| 170 | PackageRef = this.Facade.PackageId, | ||
| 171 | BundleId = id, | ||
| 172 | Action = action, | ||
| 173 | }); | ||
| 174 | } | ||
| 175 | } | ||
| 151 | } | 176 | } |
| 152 | } | 177 | } |
diff --git a/src/wix/test/WixToolsetTest.CoreIntegration/BundlePackageFixture.cs b/src/wix/test/WixToolsetTest.CoreIntegration/BundlePackageFixture.cs index 268920a6..6ccddbb7 100644 --- a/src/wix/test/WixToolsetTest.CoreIntegration/BundlePackageFixture.cs +++ b/src/wix/test/WixToolsetTest.CoreIntegration/BundlePackageFixture.cs | |||
| @@ -95,7 +95,11 @@ namespace WixToolsetTest.CoreIntegration | |||
| 95 | .ToArray(); | 95 | .ToArray(); |
| 96 | WixAssert.CompareLineByLine(new string[] | 96 | WixAssert.CompareLineByLine(new string[] |
| 97 | { | 97 | { |
| 98 | $"<BundlePackage Id='chain.exe' Cache='keep' CacheId='{chainBundleId}v1.0.0.0' InstallSize='34' Size='*' PerMachine='yes' Permanent='no' Vital='yes' RollbackBoundaryForward='WixDefaultBoundary' RollbackBoundaryBackward='WixDefaultBoundary' LogPathVariable='WixBundleLog_chain.exe' RollbackLogPathVariable='WixBundleRollbackLog_chain.exe' BundleId='{chainBundleId}' InstallArguments='' UninstallArguments='' RepairArguments='' SupportsBurnProtocol='yes' Win64='no'><Provides Key='MyProviderKey,v1.0' Version='1.0.0.0' DisplayName='BurnBundle' Imported='yes' /><PayloadRef Id='chain.exe' /></BundlePackage>", | 98 | $"<BundlePackage Id='chain.exe' Cache='keep' CacheId='{chainBundleId}v1.0.0.0' InstallSize='34' Size='*' PerMachine='yes' Permanent='no' Vital='yes' RollbackBoundaryForward='WixDefaultBoundary' RollbackBoundaryBackward='WixDefaultBoundary' LogPathVariable='WixBundleLog_chain.exe' RollbackLogPathVariable='WixBundleRollbackLog_chain.exe' BundleId='{chainBundleId}' Version='1.0.0.0' InstallArguments='' UninstallArguments='' RepairArguments='' SupportsBurnProtocol='yes' Win64='no'>" + |
| 99 | "<Provides Key='MyProviderKey,v1.0' Version='1.0.0.0' DisplayName='BurnBundle' Imported='yes' />" + | ||
| 100 | "<RelatedBundle Id='{B94478B1-E1F3-4700-9CE8-6AA090854AEC}' Action='Upgrade' />" + | ||
| 101 | "<PayloadRef Id='chain.exe' />" + | ||
| 102 | "</BundlePackage>", | ||
| 99 | }, bundlePackages); | 103 | }, bundlePackages); |
| 100 | 104 | ||
| 101 | var registrations = extractResult.SelectManifestNodes("/burn:BurnManifest/burn:Registration") | 105 | var registrations = extractResult.SelectManifestNodes("/burn:BurnManifest/burn:Registration") |
| @@ -108,6 +112,19 @@ namespace WixToolsetTest.CoreIntegration | |||
| 108 | "<Arp Register='yes' DisplayName='BundlePackageBundle' DisplayVersion='1.0.1.0' Publisher='Example Corporation' />" + | 112 | "<Arp Register='yes' DisplayName='BundlePackageBundle' DisplayVersion='1.0.1.0' Publisher='Example Corporation' />" + |
| 109 | "</Registration>" | 113 | "</Registration>" |
| 110 | }, registrations); | 114 | }, registrations); |
| 115 | |||
| 116 | ignoreAttributesByElementName = new Dictionary<string, List<string>> | ||
| 117 | { | ||
| 118 | { "WixPackageProperties", new List<string> { "DownloadSize", "PackageSize" } }, | ||
| 119 | }; | ||
| 120 | var packageElements = extractResult.SelectBADataNodes("/ba:BootstrapperApplicationData/ba:WixPackageProperties") | ||
| 121 | .Cast<XmlElement>() | ||
| 122 | .Select(e => e.GetTestXml(ignoreAttributesByElementName)) | ||
| 123 | .ToArray(); | ||
| 124 | WixAssert.CompareLineByLine(new string[] | ||
| 125 | { | ||
| 126 | "<WixPackageProperties Package='chain.exe' Vital='yes' DisplayName='BurnBundle' Description='BurnBundle' DownloadSize='*' PackageSize='*' InstalledSize='34' PackageType='Bundle' Permanent='no' LogPathVariable='WixBundleLog_chain.exe' RollbackLogPathVariable='WixBundleRollbackLog_chain.exe' Compressed='yes' Version='1.0.0.0' Cache='keep' />", | ||
| 127 | }, packageElements); | ||
| 111 | } | 128 | } |
| 112 | } | 129 | } |
| 113 | } | 130 | } |
