[Libreoffice-commits] core.git: configmgr/source

Michael Meeks michael.meeks at collabora.com
Tue Feb 7 13:39:07 UTC 2017


 configmgr/source/components.cxx |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit e5b062758950b0d6c5cea0a1914998433ef19774
Author: Michael Meeks <michael.meeks at collabora.com>
Date:   Tue Feb 7 12:27:54 2017 +0100

    configmgr: add SAL_CONFIG_WINREG_RETAIN_TMP variable.
    
    Windows registry dumping behaves very strangely, and in hard to debug
    ways - this should help in-the-field debugging of this in future.
    
    Change-Id: Ia35d58a8be36d1b683db39ae7d2128a3ccfd10a5
    Reviewed-on: https://gerrit.libreoffice.org/33992
    Reviewed-by: Michael Meeks <michael.meeks at collabora.com>
    Tested-by: Michael Meeks <michael.meeks at collabora.com>

diff --git a/configmgr/source/components.cxx b/configmgr/source/components.cxx
index 7eb8796..7fdb43a 100644
--- a/configmgr/source/components.cxx
+++ b/configmgr/source/components.cxx
@@ -561,7 +561,8 @@ Components::Components(
             OUString aTempFileURL;
             if (dumpWindowsRegistry(&aTempFileURL, eType)) {
                 parseFileLeniently(&parseXcuFile, aTempFileURL, layer, nullptr, nullptr, nullptr);
-                osl::File::remove(aTempFileURL);
+                if (!getenv("SAL_CONFIG_WINREG_RETAIN_TMP"))
+                    osl::File::remove(aTempFileURL);
             }
             ++layer; //TODO: overflow
 #endif


More information about the Libreoffice-commits mailing list