From: Ben Kaduk Date: Wed, 27 May 2015 19:36:51 +0000 (-0400) Subject: Stop using the WiX Registry element X-Git-Tag: kfw-4.1-beta2~7 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e0fa3bb33b7e7b05c584e0529364d8f1d914048c;p=thirdparty%2Fkrb5.git Stop using the WiX Registry element It is deprecated in favor of more specific XML elements such as RegistryKey, RegistryValue, and RemoveRegistryKey, so as to stop overloading a single element for what are fundamentally different types and operations. RegistryValue elements can be children of RegistryKey elements, allowing the Key attribute to be inherited, or bare within the containing Component. We do not take advantage of the inheritance at this time, since that would be a more disruptive change. WiX would prefer for us to not use the createAndRemoveOnUninstall attribute of , in favor of ForceCreateOnInstall and/or ForceRemoveOnUninstall, but that can wait for a follow-up commit. Some instances of were commented-out and can simply be removed. Some of the elements used to create keys were also setting the KeyPath attribute, which is not permitted in the element. According to http://sourceforge.net/p/wix/bugs/3197/ , this should never have been allowed, and non-value registry keys should not be used as MSI KeyPaths. In all affected cases, there are child RegistryValue elements that are suitable for use as KeyPath elements instead. Some of the elements were present with a duplicate element with different Id that added no new attributes; those duplicate elements can safely be removed. is used to replace elements with Action=removeKeyOnInstall. (cherry picked from commit 280a4b1125743bae3bf05af266131e2a524c8adf) ticket: 7392 version_fixed: 1.13.3 status: resolved --- diff --git a/src/windows/installer/wix/files.wxi b/src/windows/installer/wix/files.wxi index aadb27c4d7..5b49fb5fcc 100644 --- a/src/windows/installer/wix/files.wxi +++ b/src/windows/installer/wix/files.wxi @@ -68,15 +68,15 @@ - + KRB5CONFIG - + KRB5CCNAME - + KRB5PRESERVEIDENTITY @@ -85,56 +85,56 @@ - - + + - - + + - - + + - - + + - - + + - - + + - - + + - - + + - - + + - - + + @@ -161,14 +161,14 @@ - - + + - - + + - + LEASHAFSSTATUS - + LEASHCREATEMISSINGCONFIG - + LEASHAUTORENEWTICKETS - - + LEASHLOCKFILELOCATIONS - + LEASHMSLSAIMPORT - + @@ -217,84 +212,79 @@ - + LEASHLIFETIME - + LEASHRENEWTILL - + LEASHRENEWABLE - + LEASHFORWARDABLE - + LEASHNOADDRESSES - + LEASHPROXIABLE - + LEASHPUBLICIP - + LEASHUSEKRB4 - + LEASHHIDEKINITOPTIONS - + LEASHLIFEMIN - + LEASHLIFEMAX - + LEASHRENEWMIN - + LEASHRENEWMAX - - + LEASHUPPERCASEREALM - + LEASHTIMEHOST - + LEASHPRESERVEKINITOPTIONS - - + + - - + + @@ -488,110 +478,103 @@ - - - + + - - - - + + + + - - + + - + - + - - - - + + + + - - + + - + - - - - - - - - - - + + + + + + + + + - - - - - - - - - - + + + + + + + + + - + - - - - - - - - - - + + + + + + + + + - - - - - - - - - - + + + + + + + + + - + - - - - - - - - - - + + + + + + + + + - - - - - - - - - - + + + + + + + + +