diff options
author | Thomas Martitz <kugel@rockbox.org> | 2012-09-24 09:43:36 +0200 |
---|---|---|
committer | Thomas Martitz <kugel@rockbox.org> | 2012-09-24 09:47:03 +0200 |
commit | c555938ea0da97b2538530c1da9aeae4822bece7 (patch) | |
tree | 75388d3f00b9324104bf0d83d65e854fab463d0b | |
parent | e3cd7ec70bf80d55d1fb121f6b6c0c7fdcef5be0 (diff) | |
download | rockbox-c555938ea0da97b2538530c1da9aeae4822bece7.tar.gz rockbox-c555938ea0da97b2538530c1da9aeae4822bece7.zip |
checkwps: Do not error out on rwps for non-remote targets.
This breaks themes on the theme site (that ship an rwps for remote targets) for
non-remote targets. As rockbox just ignores the rwps checkwps now does the same.
Change-Id: Id8c7833f0bd88d239dce87711456b94c93480a51
-rw-r--r-- | tools/checkwps/checkwps.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/tools/checkwps/checkwps.c b/tools/checkwps/checkwps.c index 3ee2908088..06f88bbbbf 100644 --- a/tools/checkwps/checkwps.c +++ b/tools/checkwps/checkwps.c | |||
@@ -301,13 +301,19 @@ int main(int argc, char **argv) | |||
301 | return 2; | 301 | return 2; |
302 | } | 302 | } |
303 | ext++; | 303 | ext++; |
304 | #ifdef HAVE_REMOTE_LCD | ||
305 | if (!strcmp(ext, "rwps") || !strcmp(ext, "rsbs") || !strcmp(ext, "rfms")) | 304 | if (!strcmp(ext, "rwps") || !strcmp(ext, "rsbs") || !strcmp(ext, "rfms")) |
305 | { | ||
306 | #ifdef HAVE_REMOTE_LCD | ||
306 | screen = SCREEN_REMOTE; | 307 | screen = SCREEN_REMOTE; |
307 | else | 308 | #else |
309 | /* skip rwps etc. if not supported on this target (not an error) */ | ||
310 | continue; | ||
308 | #endif | 311 | #endif |
309 | if (!strcmp(ext, "wps") || !strcmp(ext, "sbs") || !strcmp(ext, "fms")) | 312 | } |
313 | else if (!strcmp(ext, "wps") || !strcmp(ext, "sbs") || !strcmp(ext, "fms")) | ||
314 | { | ||
310 | screen = SCREEN_MAIN; | 315 | screen = SCREEN_MAIN; |
316 | } | ||
311 | else | 317 | else |
312 | { | 318 | { |
313 | printf("Invalid extension\n"); | 319 | printf("Invalid extension\n"); |