Harmattan: Fix package version updates.

Obviously, we can't just globally replace all version numbers. Instead,
we now duplicate the latest entry and replace version number, date and
comments. Also implemented is a simple check guarding against
downgrading, which just checks whether the "new" version number already
appears in the changlog file.

Task-number: https://bugreports.qt.nokia.com/browse/QTCREATORBUG-5797
Change-Id: I17512169972415f60d1a9715ac8b98d185da8694
Reviewed-on: http://codereview.qt.nokia.com/4209
Reviewed-by: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Eike Ziller <eike.ziller@nokia.com>
This commit is contained in:
Christian Kandeler
2011-09-05 15:49:30 +02:00
committed by Eike Ziller
parent 38fce2ccb8
commit 51d0e38951
2 changed files with 58 additions and 5 deletions

View File

@@ -72,7 +72,6 @@ void MaemoPackageCreationWidget::initGui()
{ {
m_ui->shortDescriptionLineEdit->setMaxLength(60); m_ui->shortDescriptionLineEdit->setMaxLength(60);
updateVersionInfo(); updateVersionInfo();
versionInfoChanged();
const AbstractDebBasedQt4MaemoTarget * const debBasedMaemoTarget const AbstractDebBasedQt4MaemoTarget * const debBasedMaemoTarget
= m_step->debBasedMaemoTarget(); = m_step->debBasedMaemoTarget();
if (debBasedMaemoTarget) { if (debBasedMaemoTarget) {
@@ -142,9 +141,17 @@ void MaemoPackageCreationWidget::updateVersionInfo()
} }
const QStringList list = versionString.split(QLatin1Char('.'), const QStringList list = versionString.split(QLatin1Char('.'),
QString::SkipEmptyParts); QString::SkipEmptyParts);
const bool blocked = m_ui->major->signalsBlocked();
m_ui->major->blockSignals(true);
m_ui->minor->blockSignals(true);
m_ui->patch->blockSignals(true);
m_ui->major->setValue(list.value(0, QLatin1String("0")).toInt()); m_ui->major->setValue(list.value(0, QLatin1String("0")).toInt());
m_ui->minor->setValue(list.value(1, QLatin1String("0")).toInt()); m_ui->minor->setValue(list.value(1, QLatin1String("0")).toInt());
m_ui->patch->setValue(list.value(2, QLatin1String("0")).toInt()); m_ui->patch->setValue(list.value(2, QLatin1String("0")).toInt());
m_ui->major->blockSignals(blocked);
m_ui->minor->blockSignals(blocked);
m_ui->patch->blockSignals(blocked);
updateSummary();
} }
void MaemoPackageCreationWidget::handleControlFileUpdate() void MaemoPackageCreationWidget::handleControlFileUpdate()
@@ -249,8 +256,10 @@ void MaemoPackageCreationWidget::versionInfoChanged()
const bool success = m_step->setVersionString(m_ui->major->text() const bool success = m_step->setVersionString(m_ui->major->text()
+ QLatin1Char('.') + m_ui->minor->text() + QLatin1Char('.') + QLatin1Char('.') + m_ui->minor->text() + QLatin1Char('.')
+ m_ui->patch->text(), &error); + m_ui->patch->text(), &error);
if (!success) if (!success) {
QMessageBox::critical(this, tr("Could Not Set Version Number"), error); QMessageBox::critical(this, tr("Could Not Set Version Number"), error);
updateVersionInfo();
}
} }
void MaemoPackageCreationWidget::editDebianFile() void MaemoPackageCreationWidget::editDebianFile()

View File

@@ -55,6 +55,8 @@
#include <QtGui/QApplication> #include <QtGui/QApplication>
#include <QtGui/QMainWindow> #include <QtGui/QMainWindow>
#include <QtCore/QBuffer> #include <QtCore/QBuffer>
#include <QtCore/QDateTime>
#include <QtCore/QLocale>
#include <QtCore/QRegExp> #include <QtCore/QRegExp>
#include <QtCore/QDir> #include <QtCore/QDir>
#include <QtCore/QFile> #include <QtCore/QFile>
@@ -412,8 +414,50 @@ bool AbstractDebBasedQt4MaemoTarget::setProjectVersionInternal(const QString &ve
if (!reader.fetch(filePath, error)) if (!reader.fetch(filePath, error))
return false; return false;
QString content = QString::fromUtf8(reader.data()); QString content = QString::fromUtf8(reader.data());
content.replace(QRegExp(QLatin1String("\\([a-zA-Z0-9_\\.]+\\)")), if (content.contains(QLatin1Char('(') + version + QLatin1Char(')'))) {
QLatin1Char('(') + version + QLatin1Char(')')); if (error) {
*error = tr("Refusing to update changelog file: Already contains version '%1'.")
.arg(version);
}
return false;
}
int maintainerOffset = content.indexOf(QLatin1String("\n -- "));
const int eolOffset = content.indexOf(QLatin1Char('\n'), maintainerOffset+1);
if (maintainerOffset == -1 || eolOffset == -1) {
if (error) {
*error = tr("Cannot update changelog: Invalid format (no maintainer entry found).");
}
return false;
}
++maintainerOffset;
const QDateTime currentDateTime = QDateTime::currentDateTime();
QDateTime utcDateTime = QDateTime(currentDateTime);
utcDateTime.setTimeSpec(Qt::UTC);
int utcOffsetSeconds = currentDateTime.secsTo(utcDateTime);
QChar sign;
if (utcOffsetSeconds < 0) {
utcOffsetSeconds = -utcOffsetSeconds;
sign = QLatin1Char('-');
} else {
sign = QLatin1Char('+');
}
const int utcOffsetMinutes = (utcOffsetSeconds / 60) % 60;
const int utcOffsetHours = utcOffsetSeconds / 3600;
const QString dateString = QString::fromLatin1("%1 %2%3%4")
.arg(currentDateTime.toString(QLatin1String("ddd, dd MMM yyyy hh:mm:ss"))).arg(sign)
.arg(utcOffsetHours, 2, 10, QLatin1Char('0'))
.arg(utcOffsetMinutes, 2, 10, QLatin1Char('0'));
const QString maintainerLine = content.mid(maintainerOffset, eolOffset - maintainerOffset + 1)
.replace(QRegExp(QLatin1String("> [^\\n]*\n")),
QString::fromLocal8Bit("> %1").arg(dateString));
QString versionLine = content.left(content.indexOf(QLatin1Char('\n')))
.replace(QRegExp(QLatin1String("\\([a-zA-Z0-9_\\.]+\\)")),
QLatin1Char('(') + version + QLatin1Char(')'));
const QString newEntry = versionLine + QLatin1String("\n * <Add change description here>\n\n")
+ maintainerLine + QLatin1String("\n\n");
content.prepend(newEntry);
Core::FileChangeBlocker update(filePath); Core::FileChangeBlocker update(filePath);
Utils::FileSaver saver(filePath); Utils::FileSaver saver(filePath);
saver.write(content.toUtf8()); saver.write(content.toUtf8());
@@ -664,6 +708,7 @@ void AbstractDebBasedQt4MaemoTarget::handleTargetAddedSpecial()
if (QFileInfo(iconPath).exists()) if (QFileInfo(iconPath).exists())
setPackageManagerIcon(iconPath); setPackageManagerIcon(iconPath);
} }
m_filesWatcher->addDirectory(debianDirPath(), Utils::FileSystemWatcher::WatchAllChanges); m_filesWatcher->addDirectory(debianDirPath(), Utils::FileSystemWatcher::WatchAllChanges);
m_controlFile = new WatchableFile(controlFilePath(), this); m_controlFile = new WatchableFile(controlFilePath(), this);
connect(m_controlFile, SIGNAL(modified()), SIGNAL(controlChanged())); connect(m_controlFile, SIGNAL(modified()), SIGNAL(controlChanged()));
@@ -871,7 +916,6 @@ bool AbstractDebBasedQt4MaemoTarget::setPackageManagerIcon(const QString &iconFi
return success; return success;
} }
AbstractRpmBasedQt4MaemoTarget::AbstractRpmBasedQt4MaemoTarget(Qt4Project *parent, AbstractRpmBasedQt4MaemoTarget::AbstractRpmBasedQt4MaemoTarget(Qt4Project *parent,
const QString &id) : AbstractQt4MaemoTarget(parent, id) const QString &id) : AbstractQt4MaemoTarget(parent, id)
{ {