[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: am_edit/KDE_ICON bug
From:       Stephan Kulow <coolo () kde ! org>
Date:       2000-04-25 12:49:34
[Download RAW message or body]

David Faure wrote:
> 
> On Tue, Apr 25, 2000 at 02:27:22PM +0200, Stephan Kulow wrote:
> > David Faure wrote:
> > > 
> > > On Tue, Apr 25, 2000 at 02:07:10PM +0200, Stephan Kulow wrote:
> > > > David Faure wrote:
> > > > > 
> > > > > On Tue, Apr 25, 2000 at 01:32:31PM +0200, Simon Hausmann wrote:
> > > > > > Hi,
> > > > > > 
> > > > > > viewmag-.png in kdelibs/pics/locolor is not being installed.
> > > > > > 
> > > > > > am_edit generates the following rule for it:
> > > > > > 
> > > > > > 
> > > > > > $(mkinstalldirs) $(DESTDIR)$(kde_icondir)/locolor/16x16/actions/viewmag
> > > > > > $(INSTALL_DATA) $(srcdir)/lo16-action-viewmag-.png \
> > > > > > $(DESTDIR)$(kde_icondir)/locolor/16x16/actions/viewmag/.png 
> > > > > > So it seems the "-" is a problem.
> > > > > Yup.
> > > > > 
> > > > > > (shouldn't be difficult to fix for someone knowing am_edit, I guess :)
> > > > > Well this would be a major hack...
> > > > > '-' is the separator, and making a special case for this icon is
> > > > > probably not a good idea. What about renaming the icon ?
> > > > > 
> > > > Well, it's not really a problem as only two separators are valid.
> > > 
> > > Hmm, not anymore
> > > hi48-action-kde-0043.png
> > > 
> > Hmm, but this should be installed as kde-0043.png, no?
> 
> No, the goal is to put them in a subdir, kde/.
> 
> I've fixed the viewmag-.png problem anyway.
> 
Ah, then this is a feature :)

Greetings, Stephan

-- 
It said Windows 95 or better, so in theory Linux should run it
                                                GeorgeH on /.


[prev in list] [next in list] [prev in thread] [next in thread]