Changeset 535 for trunk/qmake/generators/makefile.cpp

Timestamp:

Feb 5, 2010, 1:44:09 PM (15 years ago)

Author:

Dmitry A. Kuminov

Message:

qmake: Don't prepend install commands with '-' in OS/2 mode to have install errors reported (copy command can silently overwrite the target and doesn't return an error). Add suppressing del command output in uninstall targets.

File:

• trunk/qmake/generators/makefile.cpp (modified) (7 diffs)

trunk/qmake/generators/makefile.cpp

@@ -1209 +1209 @@
 MakefileGenerator::writeInstalls(QTextStream &t, const QString &installs, bool noBuild)
 {
+    const QString del_suffix =
+        Option::target_mode == Option::TARG_OS2_MODE ?
+            QString(" >nul 2>&1"): // reduce noise
+            QString::null;
+
+    const QString inst_prefix =
+        Option::target_mode == Option::TARG_OS2_MODE ?
+            QString::null: // report errors (copy command overwrites quietly)
+            QString("-");
+
     QString rm_dir_contents("-$(DEL_FILE)");
     if (!isDosLikeShell()) //ick
@@ -1276 +1286 @@
                     QString cmd;
                     if (fi.isDir())
-                       cmd = "-$(INSTALL_DIR)";
+                       cmd = inst_prefix + "$(INSTALL_DIR)";
                     else if (fi.isExecutable())
-                       cmd = "-$(INSTALL_PROGRAM)";
+                       cmd = inst_prefix + "$(INSTALL_PROGRAM)";
                     else
-                       cmd = "-$(INSTALL_FILE)";
+                       cmd = inst_prefix + "$(INSTALL_FILE)";
                     cmd += " " + escapeFilePath(wild) + " " + dst_file + "\n";
                     target += cmd;
@@ -1289 +1299 @@
                     if(!uninst.isEmpty())
                         uninst.append("\n\t");
-                    uninst.append(rm_dir_contents + " " + filePrefixRoot(root, fileFixify(dst + filestr, FileFixifyAbsolute, false)));
+                    uninst.append(rm_dir_contents + " " + filePrefixRoot(root, fileFixify(dst + filestr, FileFixifyAbsolute, false)) + del_suffix);
                     continue;
                 }
@@ -1299 +1309 @@
                     QString dst_file = filePrefixRoot(root, dst);
                     QFileInfo fi(fileInfo(wild));
-                    QString cmd =  QString(fi.isExecutable() ? "-$(INSTALL_PROGRAM)" : "-$(INSTALL_FILE)") + " " +
+                    QString cmd = inst_prefix + QString(fi.isExecutable() ? "$(INSTALL_PROGRAM)" : "$(INSTALL_FILE)") + " " +
                                    wild + " " + dst_file + "\n";
                     target += cmd;
                     if(!uninst.isEmpty())
                         uninst.append("\n\t");
-                    uninst.append(rm_dir_contents + " " + filePrefixRoot(root, fileFixify(dst + filestr, FileFixifyAbsolute, false)));
+                    uninst.append(rm_dir_contents + " " + filePrefixRoot(root, fileFixify(dst + filestr, FileFixifyAbsolute, false)) + del_suffix);
                 }
                 for(int x = 0; x < files.count(); x++) {
@@ -1312 +1322 @@
                     if(!uninst.isEmpty())
                         uninst.append("\n\t");
-                    uninst.append(rm_dir_contents + " " + filePrefixRoot(root, fileFixify(dst + file, FileFixifyAbsolute, false)));
+                    uninst.append(rm_dir_contents + " " + filePrefixRoot(root, fileFixify(dst + file, FileFixifyAbsolute, false)) + del_suffix);
                     QFileInfo fi(fileInfo(dirstr + file));
                     if(!target.isEmpty())
@@ -1322 +1332 @@
                         dst_file += fi.fileName();
                     }
-                    QString cmd = QString(fi.isDir() ? "-$(INSTALL_DIR)" : "-$(INSTALL_FILE)") + " " +
+                    QString cmd = inst_prefix + QString(fi.isDir() ? "$(INSTALL_DIR)" : "$(INSTALL_FILE)") + " " +
                                   dirstr + file + " " + dst_file + "\n";
                     target += cmd;
@@ -1372 +1382 @@
                 t << "\n\t"
                   << uninst.join(" ") << "\n\t"
-                  << "-$(DEL_DIR) " << filePrefixRoot(root, dst) << " " << endl << endl;
+                  << "-$(DEL_DIR) " << filePrefixRoot(root, dst) << " " << del_suffix << endl << endl;
             }
             t << endl;