From 64b70acac19670cc58a5ddaa59aa00a22d983e89 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20Mr=C3=A1zek?= <peterix@gmail.com>
Date: Tue, 9 Jun 2015 00:48:25 +0200
Subject: [PATCH] GH-1060 tweaks to new update mechanism - logging

---
 application/MultiMC.cpp | 27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/application/MultiMC.cpp b/application/MultiMC.cpp
index a21455f0..263064e1 100644
--- a/application/MultiMC.cpp
+++ b/application/MultiMC.cpp
@@ -647,6 +647,10 @@ static QFile::Permissions unixModeToPermissions(const int mode)
 
 void MultiMC::installUpdates(const QString updateFilesDir, GoUpdate::OperationList operations)
 {
+	qint64 pid = -1;
+	QStringList args;
+	bool started = false;
+
 	qDebug() << "Installing updates.";
 #ifdef WINDOWS
 	QString finishCmd = applicationFilePath();
@@ -687,13 +691,21 @@ void MultiMC::installUpdates(const QString updateFilesDir, GoUpdate::OperationLi
 				if(replaced.exists())
 				{
 					QString backupFilePath = PathCombine(backupPath, replaced.completeBaseName());
-					QFile::rename(replaced.absoluteFilePath(), backupFilePath);
+					if(!QFile::rename(replaced.absoluteFilePath(), backupFilePath))
+					{
+						qWarning() << "Couldn't rename:" << replaced.absoluteFilePath() << "to" << backupFilePath;
+						goto FAILED;
+					}
 					BackupEntry be;
 					be.orig = replaced.absoluteFilePath();
 					be.backup = backupFilePath;
 					backups.append(be);
 				}
-				QFile::copy(op.file, replaced.absoluteFilePath());
+				if(!QFile::copy(op.file, replaced.absoluteFilePath()))
+				{
+					qWarning() << "Couldn't copy:" << op.file << "to" << replaced.absoluteFilePath();
+					goto FAILED;
+				}
 				QFile::setPermissions(replaced.absoluteFilePath(), unixModeToPermissions(op.mode));
 			}
 			break;
@@ -715,13 +727,13 @@ void MultiMC::installUpdates(const QString updateFilesDir, GoUpdate::OperationLi
 	}
 
 	// try to start the new binary
-	qint64 pid = -1;
-	auto args = qApp->arguments();
+	args = qApp->arguments();
 	args.removeFirst();
-	QProcess::startDetached(finishCmd, args, QDir::currentPath(), &pid);
+	started = QProcess::startDetached(finishCmd, args, QDir::currentPath(), &pid);
 	// failed to start... ?
-	if(pid == -1)
+	if(!started || pid == -1)
 	{
+		qWarning() << "Couldn't start new process properly!";
 		goto FAILED;
 	}
 	// now clean up the backed up stuff.
@@ -737,14 +749,17 @@ void MultiMC::installUpdates(const QString updateFilesDir, GoUpdate::OperationLi
 	return;
 
 FAILED:
+	qWarning() << "Update failed!";
 	// if the above failed, roll back changes
 	for(auto backup:backups)
 	{
+		qWarning() << "restoring" << backup.orig << "from" << backup.backup;
 		QFile::remove(backup.orig);
 		QFile::rename(backup.backup, backup.orig);
 	}
 	for(auto backup:trashcan)
 	{
+		qWarning() << "restoring" << backup.orig << "from" << backup.backup;
 		QFile::rename(backup.backup, backup.orig);
 	}
 	// and do nothing