Revert "Allow signing maintenancetool in Windows"

This reverts commit 76fd6e8f2953347ea0bf7a57b643968784bc9acc.
The change caused that two installation files were needed -
one executable and one dat file. Many users did not find it acceptable.
We need to figure out something else.

Change-Id: Ief12cd47f9897cee8a234d8611a5c18296d42b1c
Reviewed-by: Iikka Eklund <iikka.eklund@qt.io>

2 files changed, 12 insertions, 25 deletions

diff --git a/src/libs/installer/packagemanagercore_p.cpp b/src/libs/installer/packagemanagercore_p.cpp
index 019bbfe78..b972545ce 100644
--- a/src/libs/installer/packagemanagercore_p.cpp
+++ b/src/libs/installer/packagemanagercore_p.cpp
@@ -995,14 +995,12 @@ void PackageManagerCorePrivate::writeMaintenanceToolBinary(QFile *const input, q
 
     QInstaller::appendData(&out, input, size);
     if (writeBinaryLayout) {
-#if defined Q_OS_OSX || defined Q_OS_WIN
-        QDir resourcePath(QFileInfo(maintenanceToolRenamedName).dir());
 #ifdef Q_OS_OSX
+        QDir resourcePath(QFileInfo(maintenanceToolRenamedName).dir());
         if (!resourcePath.path().endsWith(QLatin1String("Contents/MacOS")))
             throw Error(tr("Maintenance tool is not a bundle"));
         resourcePath.cdUp();
         resourcePath.cd(QLatin1String("Resources"));
-#endif
         // It's a bit odd to have only the magic in the data file, but this simplifies
         // other code a lot (since installers don't have any appended data either)
         QTemporaryFile dataOut;
@@ -1029,7 +1027,7 @@ void PackageManagerCorePrivate::writeMaintenanceToolBinary(QFile *const input, q
         dataOut.setAutoRemove(false);
         dataOut.setPermissions(dataOut.permissions() | QFile::WriteUser | QFile::ReadGroup
             | QFile::ReadOther);
-#elif defined(Q_OS_LINUX)
+#else
         QInstaller::appendInt64(&out, 0);   // operations start
         QInstaller::appendInt64(&out, 0);   // operations end
         QInstaller::appendInt64(&out, 0);   // resource count
@@ -1298,15 +1296,12 @@ void PackageManagerCorePrivate::writeMaintenanceTool(OperationList performedOper
             QInstaller::openForRead(&input);
             layout = BinaryContent::binaryLayout(&input, BinaryContent::MagicCookieDat);
         } catch (const Error &/*error*/) {
-#if defined Q_OS_OSX || defined Q_OS_WIN
-            // On Mac and Windows data is always in a separate file
-            // so that the binary can be signed
+#ifdef Q_OS_OSX
+            // On Mac, data is always in a separate file so that the binary can be signed
             QString binaryName = isInstaller() ? installerBinaryPath() : maintenanceToolName();
             QDir dataPath(QFileInfo(binaryName).dir());
-#ifdef Q_OS_OSX
             dataPath.cdUp();
             dataPath.cd(QLatin1String("Resources"));
-#endif
             input.setFileName(dataPath.filePath(QLatin1String("installer.dat")));
 
             QInstaller::openForRead(&input);
diff --git a/src/sdk/sdkapp.h b/src/sdk/sdkapp.h
index d06cd9eb3..c5cc4ca50 100644
--- a/src/sdk/sdkapp.h
+++ b/src/sdk/sdkapp.h
@@ -76,26 +76,21 @@ public:
         installer binary itself, which contains the binary layout and the binary content. In case
         of an maintenance tool, it will return a binary that has just a binary layout append.
 
-        Note on OS X and Windows: For compatibility reason this function will return the .dat file
-        as the binary layout cannot be appended to the actual installer / maintenance tool binary
-        itself because of signing. .dat file is located inside the resource folder in the application
-        bundle in OS X and next to maintenance tool in Windows.
+        Note on OS X: For compatibility reason this function will return the a .dat file located
+        inside the resource folder in the application bundle, as on OS X the binary layout cannot
+        be appended to the actual installer / maintenance tool binary itself because of signing.
     */
     QString binaryFile() const
     {
         QString binaryFile = QCoreApplication::applicationFilePath();
-#if defined Q_OS_OSX || defined Q_OS_WIN
-        // The installer binary on OSX and Windows does not contain the binary
-        // content, it's put into the resources folder as separate file.
-        // Adjust the actual binary path. No error checking here since we
-        // will fail later while reading the binary content.
-        QDir resourcePath(QFileInfo(binaryFile).dir());
-
 #ifdef Q_OS_OSX
+        // The installer binary on OSX does not contain the binary content, it's put into
+        // the resources folder as separate file. Adjust the actual binary path. No error
+        // checking here since we will fail later while reading the binary content.
+        QDir resourcePath(QFileInfo(binaryFile).dir());
         resourcePath.cdUp();
         resourcePath.cd(QLatin1String("Resources"));
-#endif
-        binaryFile = resourcePath.filePath(QLatin1String("installer.dat"));
+        return resourcePath.filePath(QLatin1String("installer.dat"));
 #endif
         return binaryFile;
     }
@@ -119,9 +114,6 @@ public:
             QString bundlePath;
             if (QInstaller::isInBundle(fi.absoluteFilePath(), &bundlePath))
                 fi.setFile(bundlePath);
-#ifdef Q_OS_WIN
-            return fi.absoluteDir().filePath(qApp->applicationName() + QLatin1String(".dat"));
-#endif
             return fi.absoluteDir().filePath(fi.baseName() + QLatin1String(".dat"));
         }
         return QString();