Bug 672360 – app-text/llpp with app-text/mupdf-1.14.0-r2 - link.c: In function ‘trimctm’: link.c:657:20: error: incompatible type for argument 1 of ‘fz_rotate’

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description fkater 2018-12-01 22:30:19 UTC]

llpm has fatal code issues when compiled against (stable) mupdf-1.14.x
(unexpected function arguments in mupdf headers etc).

Workaround: Mask mupdf-1.14 and compile llpp against mupdf-1.13.x

Thanks

[Comment 1 Franz Trischberger 2018-12-02 10:12:09 UTC]

I think llpp needs a maintainer. Currently in Gentoo is a more recent snapshot of a release from 21 months ago. llpp is on v30 ATM. Intermediate versions never made it into the tree...

[Comment 2 fkater 2018-12-02 10:43:43 UTC]

To help finding a maintainer ;-) here some features of llpp:

- llpp doesn't replace the excellent mupdf but sits on top

- llpp also aims to be lightweight and keyboard driven

- llpp adds continuous scrolling (no annoying page flip at page end)

- llpp has excellent text selection (see issues with mupdf's own
  freeglut changes not present in gentoo:
  https://bugs.gentoo.org/show_bug.cgi?id=653298)

Thanks

[Comment 3 Jeroen Roovers (RETIRED) 2018-12-02 14:09:29 UTC]

Please attach the entire build log to this bug report.

[Comment 4 Jeroen Roovers (RETIRED) 2018-12-02 14:14:00 UTC]

Created attachment 556946 [details]
app-text:llpp-26b_p20180114:20181202-141201.log

[Comment 5 doj 2018-12-02 21:39:34 UTC]

I have fixed the ebuild file (see attachment), which now builds v30 of llpp with all the new ocaml files. However it fails with the ocaml linking of the OpenGL library:

ocamlopt.opt -c -o main.cmx -I +lablGL -I wsi/x11 main.ml
File "main.ml", line 1:
Error: The files glutils.cmi and /usr/lib64/ocaml/lablGL/glClear.cmi
       make inconsistent assumptions over interface Gl

I understand nothing about ocaml and am not able to find out what OpenGL function call in the glutils.ml file is different than the ocaml lablGL library.

[Comment 6 doj 2018-12-02 21:42:09 UTC]

Created attachment 556972 [details, diff]
ebuild for llpp v30

This ebuild file builds the llpp source code, but still fails while linking. it will likely require a source code change/patch.

[Comment 7 Anestis 2018-12-04 16:24:33 UTC]

Created attachment 557042 [details]
Working ebuild for llpp v30

[Comment 8 doj 2018-12-13 13:35:13 UTC]

attachment 557042 [details] works for me.

[Comment 9 Jouni Kosonen 2018-12-16 23:30:41 UTC]

(In reply to Anestis from comment #7)
> Created attachment 557042 [details]
> Working ebuild for llpp v30

Thanks, that worked for me as llpp-30_p20181003.ebuild. This was very helpful now that mupdf-1.13 got removed from the tree.

[Comment 10 Larry the Git Cow 2018-12-22 07:49:39 UTC]

The bug has been closed via the following commit(s):

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

commit 045561eb7c94dc7d4acf061a421d541c3a3d3e58
Author:     Tim Harder <[e-mail]>
AuthorDate: 2018-12-22 07:41:29 +0000
Commit:     Tim Harder <[e-mail]>
CommitDate: 2018-12-22 07:49:04 +0000

    app-text/llpp: version bump to 30
    
    Closes: https://bugs.gentoo.org/662206
    Closes: https://bugs.gentoo.org/672360
    
    Signed-off-by: Tim Harder <[e-mail]>

 app-text/llpp/Manifest                   |  1 +
 app-text/llpp/files/Makefile             | 90 ++++++++++++++++++++++++++++++++
 app-text/llpp/files/llpp-30-keysym.patch | 55 +++++++++++++++++++
 app-text/llpp/files/llpp.desktop         | 14 +++++
 app-text/llpp/llpp-30.ebuild             | 70 +++++++++++++++++++++++++
 5 files changed, 230 insertions(+)