diff options
author | Dominik Riebeling <Dominik.Riebeling@gmail.com> | 2020-11-15 21:23:02 +0100 |
---|---|---|
committer | Dominik Riebeling <Dominik.Riebeling@gmail.com> | 2020-11-15 21:24:51 +0100 |
commit | 6c2780e70916dea5159aad6b7de815beef769c85 (patch) | |
tree | 7e84be9b8957fb1b623ec44515776aa7ae1fe725 | |
parent | 9bb60b7d854734475727f7566ae8d8dbb9d46fdb (diff) | |
download | rockbox-6c2780e70916dea5159aad6b7de815beef769c85.tar.gz rockbox-6c2780e70916dea5159aad6b7de815beef769c85.zip |
rbutil: Rework server info data handling.
Allow constructing release candidate URL from configuration the same way
we construct the URL for releases.
Note that the server path used does not exist, taken from old release
candidate builds not available anymore.
Change-Id: I7332f08df05ed39724b4900556aaa1f5e0ba463b
-rw-r--r-- | rbutil/rbutilqt/base/serverinfo.cpp | 13 | ||||
-rw-r--r-- | rbutil/rbutilqt/base/systeminfo.cpp | 1 | ||||
-rw-r--r-- | rbutil/rbutilqt/base/systeminfo.h | 1 | ||||
-rw-r--r-- | rbutil/rbutilqt/rbutil.ini | 1 | ||||
-rw-r--r-- | rbutil/rbutilqt/test/stubs/stubs-serverinfo.cpp | 3 | ||||
-rw-r--r-- | rbutil/rbutilqt/test/test-serverinfo.cpp | 4 |
6 files changed, 14 insertions, 9 deletions
diff --git a/rbutil/rbutilqt/base/serverinfo.cpp b/rbutil/rbutilqt/base/serverinfo.cpp index 94c369af72..84aa89039d 100644 --- a/rbutil/rbutilqt/base/serverinfo.cpp +++ b/rbutil/rbutilqt/base/serverinfo.cpp | |||
@@ -85,22 +85,21 @@ QVariant ServerInfo::platformValue(enum ServerInfos info, QString platform) | |||
85 | value = value.toStringList().at(0); | 85 | value = value.toStringList().at(0); |
86 | break; | 86 | break; |
87 | case CurReleaseUrl: | 87 | case CurReleaseUrl: |
88 | case RelCandidateUrl: | ||
88 | { | 89 | { |
89 | QString version = value.toStringList().at(0); | 90 | QString version = value.toStringList().at(0); |
90 | if(value.toStringList().size() > 1) | 91 | if(value.toStringList().size() > 1) |
91 | value = value.toStringList().at(1); | 92 | value = value.toStringList().at(1); |
92 | else if(!version.isEmpty()) // if value is empty, return empty url. | 93 | else if(!version.isEmpty() && info == CurReleaseUrl) |
93 | value = SystemInfo::value(SystemInfo::ReleaseUrl).toString() | 94 | value = SystemInfo::value(SystemInfo::ReleaseUrl).toString() |
94 | .replace("%MODEL%", platform) | 95 | .replace("%MODEL%", platform) |
95 | .replace("%RELVERSION%", version); | 96 | .replace("%RELVERSION%", version); |
97 | else if(!version.isEmpty() && info == RelCandidateUrl) | ||
98 | value = SystemInfo::value(SystemInfo::CandidateUrl).toString() | ||
99 | .replace("%MODEL%", platform) | ||
100 | .replace("%RELVERSION%", version); | ||
96 | } | 101 | } |
97 | break; | 102 | break; |
98 | case RelCandidateUrl: | ||
99 | if(value.toStringList().size() > 1) | ||
100 | value = value.toStringList().at(1); | ||
101 | else | ||
102 | value.clear(); | ||
103 | break; | ||
104 | case CurDevelUrl: | 103 | case CurDevelUrl: |
105 | value = SystemInfo::value(SystemInfo::BleedingUrl).toString() | 104 | value = SystemInfo::value(SystemInfo::BleedingUrl).toString() |
106 | .replace("%MODEL%", platform); | 105 | .replace("%MODEL%", platform); |
diff --git a/rbutil/rbutilqt/base/systeminfo.cpp b/rbutil/rbutilqt/base/systeminfo.cpp index d14c3f5436..dee69a2e1e 100644 --- a/rbutil/rbutilqt/base/systeminfo.cpp +++ b/rbutil/rbutilqt/base/systeminfo.cpp | |||
@@ -42,6 +42,7 @@ const static struct { | |||
42 | { SystemInfo::Wolf3DUrl, "wolf3d_url" }, | 42 | { SystemInfo::Wolf3DUrl, "wolf3d_url" }, |
43 | { SystemInfo::XWorldUrl, "xworld_url" }, | 43 | { SystemInfo::XWorldUrl, "xworld_url" }, |
44 | { SystemInfo::ReleaseUrl, "release_url" }, | 44 | { SystemInfo::ReleaseUrl, "release_url" }, |
45 | { SystemInfo::CandidateUrl, "rc_url" }, | ||
45 | { SystemInfo::DailyUrl, "daily_url" }, | 46 | { SystemInfo::DailyUrl, "daily_url" }, |
46 | { SystemInfo::BuildInfoUrl, "build_info_url" }, | 47 | { SystemInfo::BuildInfoUrl, "build_info_url" }, |
47 | { SystemInfo::GenlangUrl, "genlang_url" }, | 48 | { SystemInfo::GenlangUrl, "genlang_url" }, |
diff --git a/rbutil/rbutilqt/base/systeminfo.h b/rbutil/rbutilqt/base/systeminfo.h index 52b50ec4d7..c4afe4b27d 100644 --- a/rbutil/rbutilqt/base/systeminfo.h +++ b/rbutil/rbutilqt/base/systeminfo.h | |||
@@ -50,6 +50,7 @@ class SystemInfo : public QObject | |||
50 | Wolf3DUrl, | 50 | Wolf3DUrl, |
51 | XWorldUrl, | 51 | XWorldUrl, |
52 | ReleaseUrl, | 52 | ReleaseUrl, |
53 | CandidateUrl, | ||
53 | ReleaseVoiceUrl, | 54 | ReleaseVoiceUrl, |
54 | ReleaseFontUrl, | 55 | ReleaseFontUrl, |
55 | BuildInfoUrl, | 56 | BuildInfoUrl, |
diff --git a/rbutil/rbutilqt/rbutil.ini b/rbutil/rbutilqt/rbutil.ini index b92ddcd557..93af1bfb1b 100644 --- a/rbutil/rbutilqt/rbutil.ini +++ b/rbutil/rbutilqt/rbutil.ini | |||
@@ -2,6 +2,7 @@ | |||
2 | ; builds | 2 | ; builds |
3 | release_url=http://download.rockbox.org/release/%RELVERSION%/rockbox-%MODEL%-%RELVERSION%.zip | 3 | release_url=http://download.rockbox.org/release/%RELVERSION%/rockbox-%MODEL%-%RELVERSION%.zip |
4 | bleeding_url=http://build.rockbox.org/data/rockbox-%MODEL%.zip | 4 | bleeding_url=http://build.rockbox.org/data/rockbox-%MODEL%.zip |
5 | rc_url=http://download.rockbox.org/release-candidate/%RELVERSION%/rockbox-%MODEL%-%RELVERSION%.zip | ||
5 | 6 | ||
6 | ; voice files | 7 | ; voice files |
7 | release_voice_url=http://download.rockbox.org/release/%RELVERSION%/%MODEL%-%RELVERSION%-english.zip | 8 | release_voice_url=http://download.rockbox.org/release/%RELVERSION%/%MODEL%-%RELVERSION%-english.zip |
diff --git a/rbutil/rbutilqt/test/stubs/stubs-serverinfo.cpp b/rbutil/rbutilqt/test/stubs/stubs-serverinfo.cpp index 7d2016d6bd..38336a7a6f 100644 --- a/rbutil/rbutilqt/test/stubs/stubs-serverinfo.cpp +++ b/rbutil/rbutilqt/test/stubs/stubs-serverinfo.cpp | |||
@@ -53,6 +53,9 @@ QVariant SystemInfo::value(SystemInfo::SystemInfos info) | |||
53 | case SystemInfo::ReleaseUrl: | 53 | case SystemInfo::ReleaseUrl: |
54 | return QString("https://unittest/release/%RELVERSION%/rockbox-%MODEL%-%RELVERSION%.zip"); | 54 | return QString("https://unittest/release/%RELVERSION%/rockbox-%MODEL%-%RELVERSION%.zip"); |
55 | break; | 55 | break; |
56 | case SystemInfo::CandidateUrl: | ||
57 | return QString("https://unittest/rc/%RELVERSION%/rockbox-%MODEL%-%RELVERSION%.zip"); | ||
58 | break; | ||
56 | default: | 59 | default: |
57 | return QString(); | 60 | return QString(); |
58 | } | 61 | } |
diff --git a/rbutil/rbutilqt/test/test-serverinfo.cpp b/rbutil/rbutilqt/test/test-serverinfo.cpp index 3e71a9763a..9aa8306e50 100644 --- a/rbutil/rbutilqt/test/test-serverinfo.cpp +++ b/rbutil/rbutilqt/test/test-serverinfo.cpp | |||
@@ -83,9 +83,9 @@ const struct testvector testdata[] = | |||
83 | { "gigabeatfx", ServerInfo::RelCandidateVersion, "f9dce96" }, | 83 | { "gigabeatfx", ServerInfo::RelCandidateVersion, "f9dce96" }, |
84 | { "gigabeatfx", ServerInfo::RelCandidateUrl, "http://dl.rockbox.org/rc/f9dce96/rockbox-gigabeatfx.zip" }, | 84 | { "gigabeatfx", ServerInfo::RelCandidateUrl, "http://dl.rockbox.org/rc/f9dce96/rockbox-gigabeatfx.zip" }, |
85 | { "archosfmrecorder", ServerInfo::RelCandidateVersion, "f9dce96" }, | 85 | { "archosfmrecorder", ServerInfo::RelCandidateVersion, "f9dce96" }, |
86 | { "archosfmrecorder", ServerInfo::RelCandidateUrl, "" }, | 86 | { "archosfmrecorder", ServerInfo::RelCandidateUrl, "https://unittest/rc/f9dce96/rockbox-archosfmrecorder-f9dce96.zip" }, |
87 | { "archosrecorder", ServerInfo::RelCandidateVersion, "f9dce96" }, | 87 | { "archosrecorder", ServerInfo::RelCandidateVersion, "f9dce96" }, |
88 | { "archosrecorder", ServerInfo::RelCandidateUrl, "" }, | 88 | { "archosrecorder", ServerInfo::RelCandidateUrl, "https://unittest/rc/f9dce96/rockbox-archosrecorder-f9dce96.zip" }, |
89 | { "iaudiox5", ServerInfo::RelCandidateVersion, "f9dce96" }, | 89 | { "iaudiox5", ServerInfo::RelCandidateVersion, "f9dce96" }, |
90 | { "iaudiox5", ServerInfo::RelCandidateUrl, "http://dl.rockbox.org/rc/f9dce96/rockbox-iaudiox5.zip" }, | 90 | { "iaudiox5", ServerInfo::RelCandidateUrl, "http://dl.rockbox.org/rc/f9dce96/rockbox-iaudiox5.zip" }, |
91 | { "iaudiox5.v", ServerInfo::RelCandidateVersion, "f9dce96" }, | 91 | { "iaudiox5.v", ServerInfo::RelCandidateVersion, "f9dce96" }, |