[hardy] please include icons

Bug #196194 reported by Morten Kjeldgaard
6
Affects Status Importance Assigned to Milestone
rasmol (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Binary package hint: rasmol

Please include icons to populate menu and panel in KDE.

Revision history for this message
Morten Kjeldgaard (mok0) wrote :

This patch includes and installs icons for rasmol, to be applied to rasmol_2.7.3.1-3 .

Changed in rasmol:
assignee: nobody → mok0
status: New → Confirmed
Revision history for this message
Jonathan Riddell (jr) wrote :

An easier work-around is

ln -s /usr/share/pixmaps/rasmol_48x48.xpm /usr/share/pixmaps/rasmol.xpm

Revision history for this message
Morten Kjeldgaard (mok0) wrote :

My png icons are prettier.

Revision history for this message
Morten Kjeldgaard (mok0) wrote :

After talking to Jordan Mantha on IRC, I was persuaded to keep it simple. Enclosed is a 19 line debdiff that creates the symlink Jonthan Riddell mentions above.

Changed in rasmol:
assignee: mok0 → rainct
importance: Undecided → Wishlist
status: Confirmed → In Progress
Revision history for this message
Siegfried Gevatter (rainct) wrote :

You have to update the Maintainer field, please subscribe ubuntu-universe-sponsors again once you have a new debdiff.

Also, I'm not sure if I understand this correctly... The problem is that GNOME's menu understands the "_32x32" suffix but KDE doesn't?

Thanks!

Changed in rasmol:
assignee: rainct → mok0
Revision history for this message
Siegfried Gevatter (rainct) wrote :

Err, the package is in main, so subscribe ubuntu-main-sponsors instead.

Also, forget my question, already got an answer to that:
< LaserJock> it doesn't work in Gnome either, btw
< LaserJock> they changed the icon name and forgot to change the .desktop ;-)

Revision history for this message
Morten Kjeldgaard (mok0) wrote :

Ydrk. Sorry about that. Maintainer field now points to u-m-d list. I also added a call to dh_desktop in debian/rules, which gave a lintian warning.

Revision history for this message
Siegfried Gevatter (rainct) wrote :

No problem :).

LaserJock mentioned that they forgot to change the .desktop; perhaps it would be preferable to update the name in the .desktop file (which if I remember correctly could need some clean-up anyways)?

Revision history for this message
Morten Kjeldgaard (mok0) wrote : Re: [Bug 196194] Re: [hardy] please include icons

The combination of the "link fix" described above, and the fact that
the desktop files are now actually *installed* (via dh_desktop) is an
acceptable solution. Alternatively, one could remove the link again,
and change the icon name in the two .desktop files. That would work
also, but I don't think one solution is better than the other. Why
not go with the fix that's already there?

Revision history for this message
Siegfried Gevatter (rainct) wrote :

Just to clarify, I'm not against to the currently proposed fix :).

I just suggest that the other way might be better, but I'm also happy
with the current one being uploaded (and anyway, it will be
overwritten by the Debian package once the problem is fixed there, so
it doesn't really matter).

Morten Kjeldgaard (mok0)
Changed in rasmol:
assignee: mok0 → nobody
status: In Progress → Confirmed
Revision history for this message
Daniel Holbach (dholbach) wrote :

Oliver: could you please take a look at it?

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package rasmol - 2.7.3.1-3ubuntu1

---------------
rasmol (2.7.3.1-3ubuntu1) hardy; urgency=low

  * debian/rasmol.link: Added link to include icon in kde
    environment (LP: #196194)
  * debian/rules: added call to dh_desktop

 -- Morten Kjeldgaard <email address hidden> Thu, 28 Feb 2008 13:49:00 +0100

Changed in rasmol:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.