Re: .desktop file for emacs

[Dan Nicolaescu <dann@ics.uci.edu>]

Ulrich Mueller <ulm@gentoo.org> writes:

  > >>>>> On Fri, 07 Dec 2007, Dan Nicolaescu wrote:
  > 
  > > I added an emacs.desktop file.
  > > Reviewing for correctness would be greatly appreciated
  > 
  > Checking the file with "desktop-file-validate" (available at
  > <http://www.freedesktop.org/software/desktop-file-utils/>) results in
  > the following output:
  > 
  >    $ desktop-file-validate emacs.desktop
  >    emacs.desktop: error: value "emacs_32.png" for key "Icon" in group
  >       "Desktop Entry" is an icon name with an extension, but there
  >       should be no extension as described in the Icon Theme
  >       Specification if the value is not an absolute path
  >    emacs.desktop: warning: value "Application;Development;TextEditor;"
  >       for key "Categories" in group "Desktop Entry" contains a
  >       deprecated value "Application"
  >    emacs.desktop: warning: key "Encoding" in group "Desktop Entry" is
  >       deprecated
  > 
  > Also, the argument given to the "Exec" command should probably be "%F"
  > instead of "%f" since more than one filename argument is allowed.
  > Patch for the above is attached.

Thanks, I installed those fixes.

  > One could also change the tooltip to say "Emacs is the extensible,
  > customizable, self-documenting real-time display editor" instead of
  > "Edit text", but this is of course a matter of taste.

Agreed. I hope someone can come up with something better than "Edit
Text" and check that in.

  > > as would figuring out how to install it correctly.
  > 
  > Basically, the desktop entry should go to /usr/share/applications/ and
  > the icon(s) either to /usr/share/pixmaps/emacs_32.png or (preferred)
  > to $XDG_DATA_DIRS/icons/hicolor/{16x16,24x24,32x32,48x48}/emacs.png
  > (where XDG_DATA_DIRS is most likely just /usr/share), as described in
  > <http://standards.freedesktop.org/icon-theme-spec/latest/ar01s03.html>.

Does this mean the icons are not installed in the correct directory
right now?

Given that you seem to understand this, do you want to provide a patch?