Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 545508 - app-text/qpdfview : USE="fitz" app-text/mupdf-1.4 dependency required?
Summary: app-text/qpdfview : USE="fitz" app-text/mupdf-1.4 dependency required?
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-04 13:51 UTC by Jan Janecek
Modified: 2015-06-16 10:21 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Janecek 2015-04-04 13:51:06 UTC
Why does the fitz backend depend on app-text/mupdf:0/1.4 ?
I tested qpdfview 0.4.14 with mupdf 1.6 and everything seems to work fine.

Reproducible: Always
Comment 1 Ben de Groot (RETIRED) gentoo-dev 2015-04-11 08:21:04 UTC
Franz, can you shed some light on this?
Comment 2 Franz Trischberger 2015-04-12 06:12:13 UTC
I had a look at the CHANGES file from mupdf. It usually lists accurately what changed API-wise. There were some API-changes in 1.4, but since then (v1.5, v1.6) there were none listed. So IMHO qpdfview should also build with mupdf:1.6.
However there may be ABI breakage, so forcing a rebuild of qpdfview after a mupdf-bump is needed - which the subslot-dep should handle just fine.

So I personally don't see any issues to allow >=mupdf-1.4 as a dep. But we should then watch for API changes in future mupdf releases (and how/if qpdfview adapts to them).
Comment 3 Franz Trischberger 2015-04-17 17:03:02 UTC
mupdf:1.7 is out and there were API changes, again. If you e.g. look at zathura-pdf-mupdf they now added a "<mupdf:1.7" restriction. IMHO that should work for qpdfview, too.
Comment 4 Franz Trischberger 2015-04-25 11:52:31 UTC
Rev 1884 (from 12. April) introduced support for mupdf:1.7. It is then a hard requirement. So the next qpdfview-release will have to depend on mupdf:1.7.
Comment 5 Davide Pesavento (RETIRED) gentoo-dev 2015-04-25 19:38:55 UTC
(In reply to Franz Fellner from comment #4)
> So the next qpdfview-release will have to depend on
> mupdf:1.7.

IIUC, >=app-text/mupdf-1.7:= makes more sense
Comment 6 Franz Trischberger 2015-06-16 10:02:38 UTC
Just wanted to point out that the version bump to qpdfview-0.4.14 still depends on >=mupdf-1.4. Should be 1.7 now.
Comment 7 Ben de Groot (RETIRED) gentoo-dev 2015-06-16 10:21:15 UTC
Fixed in qpdfview-0.4.15.ebuild now. I have removed 0.4.14.