[Intrepid] TEG icon missing from Gnome Menu

Bug #78296 reported by Conrad Knauer
2
Affects Status Importance Assigned to Milestone
teg (Ubuntu)
Fix Released
Undecided
Daniel T Chen

Bug Description

Binary package hint: teg

The bug is back in Intrepid!

/usr/share/games/teg/gnome/apps/Games/teg.desktop has the line:
Icon=teg_icono.png
but...

the file in /usr/share/pixmaps/ is just teg.png

There are these:

/usr/share/games/teg/pixmaps/teg_icono.png
/usr/share/games/teg/pixmaps/teg_pix/teg_icono.png

(and BTW, all three are the same; MD5SUM = f4a5b58b9b40063f146dc89666bb5a7c )

but they're not picked up by Gnome.

Please fix again :)

kthxbai

Related branches

Revision history for this message
Emmet Hikory (persia) wrote :

Thanks for raising the bug. I've been meaning to restore the .desktop file. I'll upload a patch soon.

Changed in teg:
assignee: nobody → persia
status: Unconfirmed → Confirmed
Revision history for this message
Emmet Hikory (persia) wrote :

The attached debdiff adjusts the .desktop file to match the recent icon name change, modifies it for validation, and copies it into /usr/share/applications.

Revision history for this message
Sarah Kowalik (hobbsee-deactivatedaccount) wrote :

uploaded, thanks!

Changed in teg:
status: Confirmed → Fix Released
Revision history for this message
Conrad Knauer (atheoi) wrote :

Oh wow, that was fast! =D (The new version just downloaded)

The good news is that it does show up in the Gnome Menu now! :)

There is a bit of bad news however :/

The new desktop file has an entry:
Icon=teg

which gets it to use
/usr/share/pixmaps/teg.xpm
which is not very pretty (~8 colors?)

The good icon is still located at
/usr/share/games/teg/pixmaps/teg_icono.png

Should I file a new bug (since the one I submitted is technically resolved :) or will this do?

Revision history for this message
Emmet Hikory (persia) wrote :

The package installs the teg_icono.png file to two separate locations, neither of which is correct according to my understanding. The attached patch installs a third copy of the file (as teg.png) which makes the menus look pretty. At least for me, the launcher icon on the panel also automatically updated with package installation.

Revision history for this message
Daniel T Chen (crimsun) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Mon, 8 Jan 2007 18:56:53 +0900
Source: teg
Binary: teg
Architecture: source
Version: 0.11.1-3ubuntu4
Distribution: feisty
Urgency: low
Maintainer: Debian Games Team <email address hidden>
Changed-By: Emmet Hikory <email address hidden>
Description:
 teg - Turn based strategy game
Changes:
 teg (0.11.1-3ubuntu4) feisty; urgency=low
 .
   * Copied the .png icon into /usr/share/pixmaps
Files:
 49f4ca32fa5a5ef087fec53a1398c534 1016 games optional teg_0.11.1-3ubuntu4.dsc
 c9fd28481eafcd9c0d0ce71499e5e248 6791 games optional teg_0.11.1-3ubuntu4.diff.g
z

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFFoi2ke9GwFciKvaMRAkFAAJ9PVlRmZxG79cbzjPDRutOI7vg8OQCffoS8
PYxO9s9+jCb56FiypkmGOG4=
=nw9w
-----END PGP SIGNATURE-----

Changed in teg:
assignee: persia → crimsun
status: Fix Released → Fix Committed
Revision history for this message
Emmet Hikory (persia) wrote :

Built on all architectures

Changed in teg:
status: Fix Committed → Fix Released
Revision history for this message
Conrad Knauer (atheoi) wrote :

w00t!

All fixed; thanks so much! :)

Conrad Knauer (atheoi)
description: updated
Changed in teg:
status: Fix Released → Confirmed
description: updated
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package teg - 0.11.2+debian-1ubuntu1

---------------
teg (0.11.2+debian-1ubuntu1) intrepid; urgency=low

  * debian/rules: Make configure-stamp depend on $(QUILT_STAMPFN) as making
    configure depend on patch is no good if nothing actually depends on
    configure. (LP: #78296)
    - Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=502894

 -- James Westby <email address hidden> Mon, 20 Oct 2008 18:41:30 +0100

Changed in teg:
status: Confirmed → Fix Released
Revision history for this message
Conrad Knauer (atheoi) wrote :

---
Commit Log for Mon Oct 20 19:03:02 2008
[...]
teg (0.11.2+debian-1) to 0.11.2+debian-1ubuntu1
---

Confirmed fixed now, thank you!

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.