Ubuntu
kmatplot package

no .desktop file

Bug #47428 reported by Vassilis Pandis on 2006-05-30

Affects		Status	Importance	Assigned to	Milestone
	kmatplot (Ubuntu)	Invalid	Low	Unassigned

Bug Description

I'm attaching one. This is related to bug #47426. There is no icon, so I'm attaching an icon grabbed from http://kmatplot.sourceforge.net . It isn't excplicitly licensed under the GNU GPL, but the program itself is so it is safe (?) to assume this is valid for the icon as well. It's quite large, 20KB. I don't really know how to make the icon smaller.

Revision history for this message

Vassilis Pandis (pandisv) wrote on 2006-05-30: .desktop file

.desktop file Edit (237 bytes, text/plain)

Revision history for this message

Vassilis Pandis (pandisv) wrote on 2006-05-30: Icon

Icon Edit (19.4 KiB, image/x-xpixmap)

Revision history for this message

Phil Bull (philbull) wrote on 2006-05-30: Altered icon

Altered icon Edit (8.7 KiB, image/x-xpixmap)

8.7KB, seems identical to me. I used an indexed palette in the GIMP and re-saved it.

Revision history for this message

Vassilis Pandis (pandisv) wrote on 2006-05-30: Re: [ubuntu-science] [Bug 47428] no .desktop file

> Dear Vassilis,
>
> It seems that there are other icons which are already part of the
> kmatplot package. Only one of them is 32x32, and its format is png.
> However, you could convert it with a command such as
>
> convert kmatplot/pics/lo32-app-kmatplot.png
> debian/kmatplot/usr/share/pixmaps/lo32-app-kmatplot.xpm
>
> during the packaging process (you would then have to build-depend on
> imagemagick).
>
> Have a nice day,
>

Thanks a lot. I took a look but apparently I missed it. Phil took care of it
immediately though :)

___________________________________________________________
All New Yahoo! Mail Tired of Vi@gr@! come-ons? Let our SpamGuard protect you. http://uk.docs.yahoo.com/nowyoucan.html

Phil Bull (philbull) on 2006-05-30

Changed in kmatplot:
status:	Unconfirmed → Confirmed

Revision history for this message

Emmet Hikory (persia) wrote on 2006-05-30:

I have previously been encouraged not to depend on ImageMagick for this particular purpose. The provided PNG files should either be copied to /usr/share/pixmaps using dh_install magic, or if they need further work (transparency, resizing, etc.), an xpm should be created and put in the debian directory.

Revision history for this message

Vassilis Pandis (pandisv) wrote on 2006-05-31:

Emmet, thanks for the information. But isn't Phil's .xpm OK ?

Revision history for this message

Phil Bull (philbull) wrote on 2006-05-31:

Also, I'm not quite sure how I might go about creating a patch with a new binary file... will debdiff handle it gracefully?

Revision history for this message

Emmet Hikory (persia) wrote on 2006-05-31:

The latest xpm is fine, although some people prefer a transparent area around a non-square icon. Patches cannot handle binary files, but a xpm file is a text file (try less), and is fine. If you *really* need a binary file, I've seen a few packages that have a uuencoded patch, and uudecode in debian/rules.

Revision history for this message

Phil Bull (philbull) wrote on 2006-05-31:

I tried to find a way of patching it, but I don't understand CDBS. I asked on #ubuntu-motu but I still couldn't get a working patch. Any chance you could look at it for me, Emmet? Thanks :)

Revision history for this message

Emmet Hikory (persia) wrote on 2006-05-31: Example debdiff (do not upload)

#10

Example debdiff (do not upload) Edit (10.6 KiB, text/plain)

The attached debdiff is a demonstration of how to add a .desktop and icon to a CDBS package (kmatplot in this case). I don't believe it belongs in either Dapper or Edgy, for the following reasons:

1) Version 4.8 has been released (also in debian)
2) The package has been orphaned in debian
3) The package has been removed from debian
(see http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=285735)
4) Upstream is inactive, and replacement project (qmatplot) is also inactive.

On the other hand, I like CDBS, as it the interface is very clean for this type of change. When making this type of patch, please remember to update the menu file so that the Debian menu will also show the icon.

Revision history for this message

LaserJock (laserjock) wrote on 2006-08-15:

#11

I'm going to reject this because kmatplot has been removed from Edgy. I don't think we should bother with adding a .desktop to a non-existant package.