Merge lp:~nik90/ubuntu/precise/evince/keywords into lp:ubuntu/precise/evince

[Nekhelesh Ramananthan]

Updated .desktop file to include keyword specification as mentioned in XDG.

[Didier Roche-Tolomelli (didrocks)]

Hey Nekhelesh,
thanks for your contribution to ubuntu and for this merge requests :)

Some notes about it:
- we normally comment the changes in the debian/changelog file. For that, we are using debchange -i and write something like:
- move the .desktop file change into something the patch system of the package can deal with
- the correct branch to target is the one in debian/control: Vcs-Bzr: (which is in that case lp:~ubuntu-desktop/…)

http://developer.ubuntu.com/packaging/html/udd-patchsys.html#edit-patch is something which might interest you in this regard. This is the end result: http://paste.ubuntu.com/855106/

Also, we try to forward the change to Upstream directly. I open a bug in the GNOME bugzilla tracker with this patch attached. We would really appreciate if you can do that yourself next time and stating it on the merge request).

Thanks again for your contribution there! That's really appreciated :)

review: Approve

[Didier Roche-Tolomelli (didrocks)]

FYI, I set the proposal as "merged" but I in fact merged the ubuntu-desktop branch :)

[Jeremy Bícha (jbicha)]

This is wrong. "Document Viewer" is already in the .desktop so it's already searchable.

review: Disapprove

=== modified file 'data/evince.desktop.in.in'
--- data/evince.desktop.in.in	2011-11-24 18:38:42 +0000
+++ data/evince.desktop.in.in	2012-02-29 02:01:18 +0000
@@ -14,3 +14,4 @@
 X-GNOME-Bugzilla-Version=@VERSION@
 Categories=GNOME;GTK;Graphics;VectorGraphics;Viewer;
 MimeType=@EVINCE_MIME_TYPES@
+_Keywords=Document;Viewer;