Summary: | Paths to header files need updating in Makefile's install target. | ||
---|---|---|---|
Product: | MuPDF | Reporter: | Gaetan Bisson <bisson> |
Component: | fitz | Assignee: | Tor Andersson <tor.andersson> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | bisson, michael, robin.watts |
Priority: | P4 | ||
Version: | master | ||
Hardware: | PC | ||
OS: | Linux | ||
Customer: | Word Size: | --- | |
Attachments: | Patch updating the paths to header files in Makefile's install. |
Description
Gaetan Bisson
2013-06-20 07:32:22 UTC
This results in /usr/include/{memento.h,fitz.h,pdf.h,xps.h,cbz.h,image.h}. At least /usr/include/image.h is very unspecific and might result in file collisions. I think the intentions was to install these files into /usr/include/mupdf/. This restructuring breaks llpp, which includes mupdf-internal.h muxps-internal.h (but, to my disappointment, build it's own copy of mupdf, specified by a git commit) http://repo.or.cz/w/llpp.git/blob/HEAD:/link.c zathura-pdf-mupdf will miss it, too. https://git.pwmt.org/?p=zathura-pdf-mupdf.git;a=blob;f=pdf.c;h=dad3c7985011e5e128bf5766ea83978f7bedf851;hb=HEAD and the mupdf/fitz.h does include mupdf/fitz/<a lot of stuff>.h as does mupdf/pdf.h, acutally, you need the whole include tree from the tarball. I added this for Gentoo http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/app-text/mupdf/files/mupdf-9999-install-headers.patch?revision=1.1&view=markup Thanks for the report guys. As you have no doubt noticed, MuPDF is undergoing a big reshuffle of its source code at the moment. Give us a few days to complete it, and hopefully it'll all shake out to something that works as you expect. The plan is that we should install headers into /usr/include/mupdf/ |