summaryrefslogtreecommitdiffstats
path: root/logic/updater
diff options
context:
space:
mode:
Diffstat (limited to 'logic/updater')
-rw-r--r--logic/updater/DownloadUpdateTask.cpp64
-rw-r--r--logic/updater/DownloadUpdateTask.h2
2 files changed, 20 insertions, 46 deletions
diff --git a/logic/updater/DownloadUpdateTask.cpp b/logic/updater/DownloadUpdateTask.cpp
index 9282c4d8..6e0a92f0 100644
--- a/logic/updater/DownloadUpdateTask.cpp
+++ b/logic/updater/DownloadUpdateTask.cpp
@@ -213,12 +213,11 @@ bool DownloadUpdateTask::parseVersionInfo(const QByteArray &data, VersionFileLis
// basically, anything that isn't in the .app folder is ignored.
// everything else is changed so the code that processes the files actually finds
// them and puts the replacements in the right spots.
- if(!fixPathForOSX(file_path))
+ if (!fixPathForOSX(file_path))
continue;
#endif
- VersionFileEntry file{
- file_path , fileObj.value("Perms").toVariant().toInt(),
- FileSourceList(), fileObj.value("MD5").toString(), };
+ VersionFileEntry file{file_path, fileObj.value("Perms").toVariant().toInt(),
+ FileSourceList(), fileObj.value("MD5").toString(), };
QLOG_DEBUG() << "File" << file.path << "with perms" << file.mode;
QJsonArray sourceArray = fileObj.value("Sources").toArray();
@@ -234,9 +233,9 @@ bool DownloadUpdateTask::parseVersionInfo(const QByteArray &data, VersionFileLis
}
else if (type == "httpc")
{
- file.sources.append(FileSource("httpc",
- fixPathForTests(sourceObj.value("Url").toString()),
- sourceObj.value("CompressionType").toString()));
+ file.sources.append(
+ FileSource("httpc", fixPathForTests(sourceObj.value("Url").toString()),
+ sourceObj.value("CompressionType").toString()));
}
else
{
@@ -401,25 +400,23 @@ DownloadUpdateTask::processFileLists(NetJob *job,
QString dlPath =
PathCombine(m_updateFilesDir.path(), QString(entry.path).replace("/", "_"));
- if (job)
+ if (isUpdater)
+ {
+ auto cache_entry = MMC->metacache()->resolveEntry("root", entry.path);
+ QLOG_DEBUG() << "Updater will be in " << cache_entry->getFullPath();
+ if(cache_entry->stale)
+ {
+ auto download = CacheDownload::make(QUrl(source.url), cache_entry);
+ job->addNetAction(download);
+ }
+ }
+ else
{
// We need to download the file to the updatefiles folder and add a task
// to copy it to its install path.
auto download = MD5EtagDownload::make(source.url, dlPath);
download->m_expected_md5 = entry.md5;
job->addNetAction(download);
-
- if (isUpdater)
- {
- download->setProperty("finalPath", entry.path);
- download->setProperty("finalPerms", entry.mode);
- connect(download.get(), &MD5EtagDownload::succeeded, this, &DownloadUpdateTask::directDeployFile);
- }
- }
-
- if (!isUpdater)
- {
- // Now add a copy operation to our operations list to install the file.
ops.append(UpdateOperation::CopyOp(dlPath, entry.path, entry.mode));
}
}
@@ -503,7 +500,7 @@ bool DownloadUpdateTask::writeInstallScript(UpdateOperationList &opsList, QStrin
QString DownloadUpdateTask::fixPathForTests(const QString &path)
{
- if(path.startsWith("$PWD"))
+ if (path.startsWith("$PWD"))
{
QString foo = path;
foo.replace("$PWD", qApp->applicationDirPath());
@@ -514,10 +511,10 @@ QString DownloadUpdateTask::fixPathForTests(const QString &path)
bool DownloadUpdateTask::fixPathForOSX(QString &path)
{
- if(path.startsWith("MultiMC.app/"))
+ if (path.startsWith("MultiMC.app/"))
{
// remove the prefix and add a new, more appropriate one.
- path.remove(0,12);
+ path.remove(0, 12);
path = QString("../../") + path;
return true;
}
@@ -528,7 +525,6 @@ bool DownloadUpdateTask::fixPathForOSX(QString &path)
}
}
-
void DownloadUpdateTask::fileDownloadFinished()
{
emitSucceeded();
@@ -546,26 +542,6 @@ void DownloadUpdateTask::fileDownloadProgressChanged(qint64 current, qint64 tota
setProgress((int)(((float)current / (float)total) * 100));
}
-void DownloadUpdateTask::directDeployFile(const int index)
-{
- Md5EtagDownloadPtr download = std::dynamic_pointer_cast<MD5EtagDownload>(m_filesNetJob->operator[](index));
- const QString finalPath = download->property("finalPath").toString();
- bool ok = true;
- int finalMode = download->property("finalPerms").toInt(&ok);
- if(!ok)
- finalMode = 0755;
- QLOG_INFO() << "Replacing" << finalPath << "with" << download->m_output_file.fileName();
- if (QFile::remove(finalPath))
- {
- if (download->m_output_file.copy(finalPath))
- {
- QFile::setPermissions(finalPath, (QFileDevice::Permission) finalMode);
- return;
- }
- }
- emitFailed("Couldn't copy updater files");
-}
-
QString DownloadUpdateTask::updateFilesDir()
{
return m_updateFilesDir.path();
diff --git a/logic/updater/DownloadUpdateTask.h b/logic/updater/DownloadUpdateTask.h
index d82b044f..b1d14846 100644
--- a/logic/updater/DownloadUpdateTask.h
+++ b/logic/updater/DownloadUpdateTask.h
@@ -221,7 +221,5 @@ protected slots:
void fileDownloadFinished();
void fileDownloadFailed();
void fileDownloadProgressChanged(qint64 current, qint64 total);
-
- void directDeployFile(const int index);
};