Bug 42590 - kdeadmin-2.2A2: kuser missing hicolor toolbar icons
Summary: kdeadmin-2.2A2: kuser missing hicolor toolbar icons
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Raw Hide
Classification: Retired
Component: kdeadmin
Version: 1.0
Hardware: i386
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Bernhard Rosenkraenzer
QA Contact: Aaron Brown
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2001-05-28 16:01 UTC by j. alan eldridge
Modified: 2007-04-18 16:33 UTC (History)
0 users

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2001-05-28 16:01:30 UTC
Embargoed:


Attachments (Terms of Use)

Description j. alan eldridge 2001-05-28 16:01:26 UTC
Description of Problem:

The icons on the kuser toolbar are missing. That's 'cause they're 
missing from the source tarball.

It could be patched up in the source dirs, but frankly, it's easier to 
fix this in the spec file, considering sooner or later somebody at KDE's 
hideout will notice this and deal with it officially.

Watch the web-form-mangled lines below. It's easy enough to see what to 
do though. Just dupe the directory locolor to hicolor. You could even do 
it with a symlink, but that might confuse rpm down the road when a new 
package comes out that has dirs.

[alane@wwweasel SPECS]$ diff -ru kdeadmin.spec.or kdeadmin.spec
--- kdeadmin.spec.or    Tue May 22 08:47:07 2001
+++ kdeadmin.spec       Mon May 28 11:52:38 2001
@@ -94,6 +94,10 @@
 mv $RPM_BUILD_ROOT/usr/bin/*kontrol-panel 
$RPM_BUILD_ROOT/usr/bin/kontrol-panel
 mv $RPM_BUILD_ROOT/etc/X11/applnk/System/kontrol-panel.desktop 
$RPM_BUILD_ROOT/usr/share/applnk/System
 
+#  fix missing icons
+cp -a $RPM_BUILD_ROOT/usr/share/apps/kuser/icons/locolor \
+        $RPM_BUILD_ROOT/usr/share/apps/kuser/icons/hicolor
+
 # remove dupes from kdebase
 rm -f 
$RPM_BUILD_ROOT/%{prefix}/share/icons/locolor/16x16/apps/{package,printer}.png 

Shit. I mean, you run the app, the toolbar's all whacked out. How do you 
*not* notice this? (A. When you're not wiping clean between test 
installs.)

Comment 1 Bernhard Rosenkraenzer 2001-08-09 13:40:23 UTC
This was fixed some time between then and now.



Note You need to log in before you can comment on or make changes to this bug.