From 968448c9cdd2583225cff015b43aeeb9d1c304da Mon Sep 17 00:00:00 2001 From: Dominik Riebeling Date: Wed, 23 May 2012 21:05:28 +0200 Subject: ZipIntaller: use file timestamp if version string is missing. Instead of using some arbitrary and not really useful string use the timestamp of the downloaded file if version string is provided by the caller. This also makes it possible to check if the downloaded file is actually a different one. Use it for manual and game file downloads, as this gives more reasonable values than using the date transmitted for daily (archived) build and as support for archived builds will be removed shortly. Change-Id: I0c751fabe7bb516edca93a5f73f077a611d4ef87 --- rbutil/rbutilqt/base/zipinstaller.cpp | 6 +++++- rbutil/rbutilqt/base/zipinstaller.h | 6 ++++-- rbutil/rbutilqt/gui/manualwidget.cpp | 2 +- rbutil/rbutilqt/rbutilqt.cpp | 2 +- 4 files changed, 11 insertions(+), 5 deletions(-) diff --git a/rbutil/rbutilqt/base/zipinstaller.cpp b/rbutil/rbutilqt/base/zipinstaller.cpp index 3f3179f5f2..117fa2dae9 100644 --- a/rbutil/rbutilqt/base/zipinstaller.cpp +++ b/rbutil/rbutilqt/base/zipinstaller.cpp @@ -59,7 +59,7 @@ void ZipInstaller::installContinue() m_url = m_urllist.at(runner); m_logsection = m_loglist.at(runner); if(runner < m_verlist.size()) m_logver = m_verlist.at(runner); - else m_logver = "0"; + else m_logver = ""; installStart(); } else { @@ -176,6 +176,10 @@ void ZipInstaller::downloadDone(bool error) // add file to log zipContents.append(m_target); } + if(m_logver.isEmpty()) { + // if no version info is set use the timestamp of the server file. + m_logver = getter->timestamp().toString(Qt::ISODate); + } emit logItem(tr("Creating installation log"),LOGINFO); QSettings installlog(m_mountpoint + "/.rockbox/rbutil.log", QSettings::IniFormat, 0); diff --git a/rbutil/rbutilqt/base/zipinstaller.h b/rbutil/rbutilqt/base/zipinstaller.h index cb154c03a8..e15bbcf783 100644 --- a/rbutil/rbutilqt/base/zipinstaller.h +++ b/rbutil/rbutilqt/base/zipinstaller.h @@ -41,8 +41,10 @@ public: void setUrl(QStringList url) { m_urllist = url; } void setLogSection(QString name) {m_loglist = QStringList(name);} void setLogSection(QStringList name) { m_loglist = name; } - void setLogVersion(QString v) { m_verlist = QStringList(v); qDebug() << m_verlist;} - void setLogVersion(QStringList v) { m_verlist = v; qDebug() << m_verlist;} + void setLogVersion(QString v = "") + { m_verlist = QStringList(v); qDebug() << m_verlist;} + void setLogVersion(QStringList v) + { m_verlist = v; qDebug() << m_verlist;} void setUnzip(bool i) { m_unzip = i; } void setTarget(QString t) { m_target = t; } void setCache(QDir c) { m_cache = c; m_usecache = true; }; diff --git a/rbutil/rbutilqt/gui/manualwidget.cpp b/rbutil/rbutilqt/gui/manualwidget.cpp index 3bc795f098..ae38980479 100644 --- a/rbutil/rbutilqt/gui/manualwidget.cpp +++ b/rbutil/rbutilqt/gui/manualwidget.cpp @@ -127,7 +127,7 @@ void ManualWidget::downloadManual(void) } qDebug() << "[ManualWidget] Manual URL:" << manualurl; - installer->setLogVersion(ServerInfo::value(ServerInfo::DailyDate).toString()); + installer->setLogVersion(); installer->setUrl(manualurl); installer->setUnzip(false); diff --git a/rbutil/rbutilqt/rbutilqt.cpp b/rbutil/rbutilqt/rbutilqt.cpp index 5c496585ec..ae45068bde 100644 --- a/rbutil/rbutilqt/rbutilqt.cpp +++ b/rbutil/rbutilqt/rbutilqt.cpp @@ -994,7 +994,7 @@ void RbUtilQt::installDoom() installer->setUrl(SystemInfo::value(SystemInfo::DoomUrl).toString()); installer->setLogSection("Game Addons"); - installer->setLogVersion(ServerInfo::value(ServerInfo::DailyDate).toString()); + installer->setLogVersion(); installer->setMountPoint(RbSettings::value(RbSettings::Mountpoint).toString()); if(!RbSettings::value(RbSettings::CacheDisabled).toBool()) installer->setCache(true); -- cgit v1.2.3