Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 524376 - app-text/mupdf-1.5 - mupdf.desktop should not refer to mupdf-select-file
Summary: app-text/mupdf-1.5 - mupdf.desktop should not refer to mupdf-select-file
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Michael Weber (RETIRED)
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2014-10-03 11:55 UTC by Dirk Sondermann
Modified: 2014-10-10 13:51 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
patch for files/mupdf-1.4-old-debian-files.patch (desktop.patch,168 bytes, patch)
2014-10-03 11:59 UTC, Dirk Sondermann
Details | Diff
patch for files/mupdf-1.4-old-debian-files.patch (desktop.patch,376 bytes, patch)
2014-10-03 12:03 UTC, Dirk Sondermann
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Sondermann 2014-10-03 11:55:44 UTC
The mupdf.desktop file, which app-text/mupdf-1.5 installs in /usr/share/applications, refers to the script mupdf-select-file. This script was part of mupdf-1.4, but has been dropped in mupdf-1.5 (cf. https://bugs.gentoo.org/show_bug.cgi?id=508076).

As a consequence, mimeopen fails to open pdf files with mupdf.

Reproducible: Always

Steps to Reproduce:
1. Install app-text/mupdf-1.5
2. Make sure that dev-perl/File-MimeInfo (to which mimeopen belongs) is installed
3. Try to open a pdf file: mimeopen -a test.pdf
4. Select mupdf as application
Actual Results:  
mimeopen prints "Opening test.pdf with MuPDF (application/pdf)" and fails with exit status 1.

Expected Results:  
mimeopen starts mupdf.

mupdf.desktop is defined in the patch files/mupdf-1.4-old-debian-files.patch, which is applied by mupdf-1.5.ebuild and mupdf-9999.ebuild.

Replacing the two lines

  Exec=mupdf-select-file %f
  TryExec=mupdf-select-file

in that patch by

  Exec=mupdf %f
  TryExec=mupdf

solves the problem for me.
Comment 1 Dirk Sondermann 2014-10-03 11:59:20 UTC
Created attachment 385988 [details, diff]
patch for files/mupdf-1.4-old-debian-files.patch
Comment 2 Dirk Sondermann 2014-10-03 12:03:16 UTC
Created attachment 385990 [details, diff]
patch for files/mupdf-1.4-old-debian-files.patch
Comment 3 Michael Weber (RETIRED) gentoo-dev 2014-10-10 13:51:04 UTC
+*mupdf-1.5-r1 (10 Oct 2014)
+
+  10 Oct 2014; Michael Weber <xmw@gentoo.org>
+  +files/mupdf-1.5-old-debian-files.patch, +mupdf-1.5-r1.ebuild,
+  -mupdf-1.5.ebuild, mupdf-9999.ebuild:
+  Fix .desktop file (bug 524376, thanks Dirk Sondermann).
+