Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 667788 - app-text/mupdf building with system liblcms not supported
Summary: app-text/mupdf building with system liblcms not supported
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:
Depends on:
Blocks:
 
Reported: 2018-10-05 09:32 UTC by fiesh
Modified: 2018-10-07 17:43 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 fiesh 2018-10-05 09:32:52 UTC
Building mupdf with the standard liblcms is not supported.  This can lead to assertions failing, see:

https://bugs.ghostscript.com/show_bug.cgi?id=699262
https://bugs.ghostscript.com/show_bug.cgi?id=699887

Adding the lcms use flag fixes the issue.

I think the lcms use flag should be removed and its behavior the default.

Reproducible: Always

Steps to Reproduce:
See mupdf bug reports
Comment 1 Virgil Dupras (RETIRED) gentoo-dev 2018-10-05 11:54:08 UTC
Thanks for this report. To add upon this, the clearest summary of the crux of the problem is https://bugzilla.redhat.com/show_bug.cgi?id=1553915#c7

So yeah, we should probably remove the lcms flag.
Comment 2 Virgil Dupras (RETIRED) gentoo-dev 2018-10-06 15:05:14 UTC
Further information: this flag is rather new, introduced in bug 641862
Comment 3 Larry the Git Cow gentoo-dev 2018-10-07 17:43:10 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=da7c28ccf0ad7b3a35d0adbd0ed2ad50640e718f

commit da7c28ccf0ad7b3a35d0adbd0ed2ad50640e718f
Author:     Virgil Dupras <vdupras@gentoo.org>
AuthorDate: 2018-10-07 13:39:48 +0000
Commit:     Virgil Dupras <vdupras@gentoo.org>
CommitDate: 2018-10-07 17:40:50 +0000

    app-text/mupdf: bump to 1.14.0
    
    Many things changed upstream in this release, which prompts many changes
    in the ebuild.
    
    1. Drop the "static" USE flag. It was already broken in 1.13 because
    options weren't passed to the static `emake` call like is was in the
    main `emake` invocation. In addition to that, building a static binary
    requires "media-gfx/graphite2[+static-libs]" which doesn't exist.
    
    2. Drop "lcms" USE flag. See bug #667788.
    
    3. Always link to dev-lang/mujs. mupdf can't build without being linked
    to mujs now. The best we can do it to disable JS with the "FZ_ENABLE_JS"
    flag, which is what the "javascript" USE flag does.
    
    4. Make "javascript" USE flag enabled by default, which is in line with
    upstream's behavior.
    
    5. Make "vanilla" USE flag enabled by default. The patch it enables
    doesn't seem very useful and I couldn't track it's *raison d'être* in
    git/cvs history: it's been there since the first commit.
    
    6. Simplify ebuild and remove obsolete seds. The "sed" invocations I've
    removed have been verified and either point to absent files or replace
    nothing.
    
    7. Don't remove third party libs. The new "USE_SYSTEM_LIBS" flag being
    used is a reliable way to ensure that the correct libraries are used.
    
    8. Fix "curl" USE flag. See bug #664646
    
    Closes: https://bugs.gentoo.org/664646
    Closes: https://bugs.gentoo.org/667788
    
    Signed-off-by: Virgil Dupras <vdupras@gentoo.org>
    Package-Manager: Portage-2.3.50, Repoman-2.3.11

 app-text/mupdf/Manifest                            |   1 +
 app-text/mupdf/files/mupdf-1.14-CFLAGS.patch       |  13 +++
 app-text/mupdf/files/mupdf-1.14-debug-build.patch  |  13 +++
 .../mupdf/files/mupdf-1.14-openssl-curl-x11.patch  |  52 +++++++++
 app-text/mupdf/mupdf-1.14.0.ebuild                 | 123 +++++++++++++++++++++
 5 files changed, 202 insertions(+)