From f928c8f0f854a91323b19357021bcaef9a7cc27e Mon Sep 17 00:00:00 2001
From: Jens Arnold <amiconn@rockbox.org>
Date: Sat, 23 Feb 2008 22:54:33 +0000
Subject: [PATCH] Turned out that setting the viewer iconset to empty properly
 chooses the default iconset, and the problem with the default viewer iconset
 on greyscale targets was that there was a file too much in icons/. The
 defaults are copied from elsewhere, and differ between grey/mono and colour
 targets. The extraneous file overwrote the default when building the .zip.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@16399 a1c6a512-1295-4272-9138-f99709370657
---
 icons/viewers.bmp | Bin 2134 -> 0 bytes
 wps/WPSLIST       |  20 ++++++++++----------
 wps/wpsbuild.pl   |   8 +++++---
 3 files changed, 15 insertions(+), 13 deletions(-)
 delete mode 100644 icons/viewers.bmp

diff --git a/icons/viewers.bmp b/icons/viewers.bmp
deleted file mode 100644
index 9b039b12174d40abc83ea28de1d36257bf8609ec..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 2134
zcmd5)F;c`Z5Hk(K3>_6EB`@Fs^t8Nz|9>E-tSm*G<1m3MU=n++)k<=X_m9tqR)23;
zU-91K{la_Kp0T#}g8tJZ-vY2KZMk}YT_MZL1IC65&g#Uz^#Y#EcD2pSvl|5Cfp!vO
z5UqdIlgt{b>eOQsfh)7A!9(e%cs{AXtV$QwbLYr|0-#or`6O)*0V`QG_C9%h5Aw_A
zr+k!*%ruBunRn-&z0+#Tdvv`2@9MP1);+9u7GqROmPq<aHSZf3r-oU>gC06J#UcyK
zy)5Vm_T2<v2gXyZ#N4QkzrjRLY<mcx2TyCQG+W6Zk~}iVDyn@4OiCOdeQ~w$5PO1A
zJy!c-5OA%)g2DO(7D|VDgNw7+9f8E+ULEs{-H7+ySI!$c{m!PKP!^06u7dX%Fax6U
dZh+%Cf#q1$liB~o+sm?t@r$riweR*T>;mrs-H`wQ

diff --git a/wps/WPSLIST b/wps/WPSLIST
index c7fab9e6cf..52d81a6ab7 100644
--- a/wps/WPSLIST
+++ b/wps/WPSLIST
@@ -51,7 +51,7 @@ Foreground Color: 000000
 Background Color: B6C6E5
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
@@ -66,7 +66,7 @@ Foreground Color: 000000
 Background Color: B6C6E5
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
@@ -78,7 +78,7 @@ Font: nedore-8.fnt
 Statusbar: on
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
@@ -89,7 +89,7 @@ Author: Raymond Hoh
 Font: nimbus-12.fnt
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 Statusbar: on
 </wps>
@@ -102,7 +102,7 @@ Font: nedore-8.fnt
 Statusbar: on
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
@@ -149,7 +149,7 @@ Font: nimbus-13.fnt
 Statusbar: on
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
@@ -170,7 +170,7 @@ Foreground Color: 000000
 Background Color: B6C6E5
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
@@ -201,7 +201,7 @@ Background Color: FFFFFF
 Statusbar: on
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
@@ -223,7 +223,7 @@ Background Color: FFFFFF
 Statusbar: on
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
@@ -254,7 +254,7 @@ Background Color: FFC000
 Statusbar: on
 backdrop:
 iconset:
-viewers iconset: /.rockbox/icons/viewers.bmp
+viewers iconset:
 selector type: bar (inverse)
 </wps>
 
diff --git a/wps/wpsbuild.pl b/wps/wpsbuild.pl
index a07909ca77..cb9ccbfc80 100755
--- a/wps/wpsbuild.pl
+++ b/wps/wpsbuild.pl
@@ -144,8 +144,10 @@ sub copythemeviewericon
 {
     #copy the viewer icon specified by the theme
 
-    $viewericon =~ /\/(.*icons\/(.*))/i;
-    `cp $ROOT/icons/$2 $1`;
+    if ($viewericon ne '') {
+        $viewericon =~ /\/(.*icons\/(.*))/i;
+        `cp $ROOT/icons/$2 $1`;
+    }
 }
 
 sub copywps
@@ -244,7 +246,7 @@ MOO
     if(defined($iconset)) {
         push @out, "iconset: $iconset\n";
     }
-    if($viewericon) {
+    if(defined($viewericon)) {
         push @out, "viewers iconset: $viewericon\n";
     }
     if($lineselecttextcolor && $main_depth > 2 ) {