summaryrefslogtreecommitdiff
path: root/rbutil
diff options
context:
space:
mode:
authorDominik Riebeling <Dominik.Riebeling@gmail.com>2012-03-12 22:16:14 +0100
committerDominik Riebeling <Dominik.Riebeling@gmail.com>2012-03-12 22:16:14 +0100
commit674bbbaf832b83042b906c564b9240cd894c2583 (patch)
treeb8cc5935fbc5c78000aaf89256aa0a6d10d35505 /rbutil
parent93cf878a487d952c29dabd0076592d006ca221a4 (diff)
Fix false positive in version detection.
Some git revisions are falsely detected as release version. Extend test and fix the wrong regexp. Corrects font download trying to get the wrong file for some current builds. Change-Id: Ia779896c927385ec0d7ba68ee3d953f3c6e4494c
Diffstat (limited to 'rbutil')
-rw-r--r--rbutil/rbutilqt/base/rockboxinfo.cpp2
-rw-r--r--rbutil/rbutilqt/test/test-rockboxinfo.cpp1
2 files changed, 2 insertions, 1 deletions
diff --git a/rbutil/rbutilqt/base/rockboxinfo.cpp b/rbutil/rbutilqt/base/rockboxinfo.cpp
index 9d8aba65f9..f9c0ea5b23 100644
--- a/rbutil/rbutilqt/base/rockboxinfo.cpp
+++ b/rbutil/rbutilqt/base/rockboxinfo.cpp
@@ -35,7 +35,7 @@ RockboxInfo::RockboxInfo(QString mountpoint, QString fname)
// read file contents
QRegExp hash("^Version:\\s+(r?)([0-9a-fM]+)");
QRegExp version("^Version:\\s+(\\S.*)");
- QRegExp release("^Version:\\s+([0-9\\.]+)");
+ QRegExp release("^Version:\\s+([0-9\\.]+)\\s*$");
QRegExp target("^Target:\\s+(\\S.*)");
QRegExp features("^Features:\\s+(\\S.*)");
QRegExp targetid("^Target id:\\s+(\\S.*)");
diff --git a/rbutil/rbutilqt/test/test-rockboxinfo.cpp b/rbutil/rbutilqt/test/test-rockboxinfo.cpp
index 310a0770b3..e7191860d2 100644
--- a/rbutil/rbutilqt/test/test-rockboxinfo.cpp
+++ b/rbutil/rbutilqt/test/test-rockboxinfo.cpp
@@ -54,6 +54,7 @@ void TestRockboxInfo::testVersion()
{ "Version: e5b1b0f-120218", "e5b1b0f", "e5b1b0f-120218", "" },
{ "Version: e5b1b0fM-120218", "e5b1b0fM", "e5b1b0fM-120218", "" },
{ "#Version: e5b1b0f-120218", "", "", "" },
+ { "Version: 3448f5b-120310", "3448f5b", "3448f5b-120310", "" },
};