kpriv
/
freebsd-ports
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

print/sioyek: unbreak with mupdf 0.23 and unbundle synctex 

- mupdf 0.23 dropped a symbol we need; apply a Debian patch to not
   require it
 - unbundle synctex

See also:	https://github.com/ahrm/sioyek/issues/804
Obtained from:	https://salsa.debian.org/viccie30/sioyek/-/blob/debian/latest/debian/patches/0007-fix-build-with-mupdf-0.23.0.patch
MFH:		2024Q1
This commit is contained in:
Robert Clausecker 2024-01-02 14:42:52 +01:00
parent 85b044858d
commit b7afa2280b
3 changed files with 108 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
print/sioyek/Makefile
View File

@ -1,7 +1,7 @@
PORTNAME= sioyek
DISTVERSIONPREFIX= v
DISTVERSION= 2.0.0
PORTREVISION= 1
PORTREVISION= 2
CATEGORIES= print
MAINTAINER= fuz@FreeBSD.org
@ -16,7 +16,8 @@ LIB_DEPENDS= libmupdf.so:graphics/mupdf \
libfreetype.so:print/freetype2 \
libjbig2dec.so:graphics/jbig2dec \
libmujs.so:lang/mujs \
libopenjp2.so:graphics/openjpeg
libopenjp2.so:graphics/openjpeg \
libsynctex.so:devel/tex-synctex
USES= desktop-file-utils jpeg gl qmake qt:5 sqlite
USE_GITHUB= yes

20
print/sioyek/files/patch-pdf__viewer__build__config.pro
View File

@ -9,7 +9,16 @@
pdf_viewer/ui.h \
pdf_viewer/path.h \
pdf_viewer/utf8.h \
@@ -55,7 +53,6 @@ SOURCES += pdf_viewer/book.cpp \
@@ -37,8 +35,6 @@ HEADERS += pdf_viewer/book.h \
pdf_viewer/utf8/checked.h \
pdf_viewer/utf8/core.h \
pdf_viewer/utf8/unchecked.h \
- pdf_viewer/synctex/synctex_parser.h \
- pdf_viewer/synctex/synctex_parser_utils.h \
pdf_viewer/RunGuard.h \
pdf_viewer/OpenWithApplication.h
@@ -55,12 +51,9 @@ SOURCES += pdf_viewer/book.cpp \
pdf_viewer/checksum.cpp \
pdf_viewer/new_file_checker.cpp \
pdf_viewer/coordinates.cpp \
@ -17,12 +26,17 @@
pdf_viewer/ui.cpp \
pdf_viewer/path.cpp \
pdf_viewer/utils.cpp \
@@ -82,7 +79,7 @@ unix:!mac {
- pdf_viewer/synctex/synctex_parser.c \
- pdf_viewer/synctex/synctex_parser_utils.c \
pdf_viewer/RunGuard.cpp \
pdf_viewer/OpenWithApplication.cpp
@@ -82,7 +75,7 @@ unix:!mac {
} else {
DEFINES += NON_PORTABLE
DEFINES += LINUX_STANDARD_PATHS
- LIBS += -ldl -lmupdf -lmupdf-third -lgumbo -lharfbuzz -lfreetype -ljbig2dec -ljpeg -lmujs -lopenjp2 -lz
+ LIBS += -ldl -lmupdf -lmupdf-third -lharfbuzz -lfreetype -ljbig2dec -ljpeg -lmujs -lopenjp2 -lz -lsqlite3
+ LIBS += -ldl -lmupdf -lmupdf-third -lharfbuzz -lfreetype -ljbig2dec -ljpeg -lmujs -lopenjp2 -lz -lsqlite3 -lsynctex
}
isEmpty(PREFIX){

88
print/sioyek/files/patch-pdf_parse_link_uri Normal file
View File

@ -0,0 +1,88 @@
From: Patrycja Rosa <alpine@ptrcnull.me>
Date: Fri, 25 Aug 2023 22:44:26 +0200
Subject: fix build with mupdf-0.23.0
Fix for disappeared symbol in mupdf 0.23.
Origin: vendor, https://git.alpinelinux.org/aports/commit/?id=86e913eccf19b97a16f25d9b6cdf0f50232f1226
Bug: https://github.com/ahrm/sioyek/issues/804
---
pdf_viewer/document_view.cpp | 2 +-
pdf_viewer/main_widget.cpp | 6 +++---
pdf_viewer/utils.cpp | 5 +++--
pdf_viewer/utils.h | 2 +-
4 files changed, 8 insertions(+), 7 deletions(-)
diff --git pdf_viewer/document_view.cpp pdf_viewer/document_view.cpp
index f48d05e..0bc3855 100644
--- pdf_viewer/document_view.cpp
+++ pdf_viewer/document_view.cpp
@@ -1121,7 +1121,7 @@ std::vector<DocumentPos> DocumentView::find_line_definitions() {
std::optional<PdfLink> pdf_link = current_document->get_link_in_page_rect(get_center_page_number(), line_rects[line_index]);
if (pdf_link.has_value()) {
- auto parsed_uri = parse_uri(mupdf_context, pdf_link.value().uri);
+ auto parsed_uri = parse_uri(mupdf_context, current_document->doc, pdf_link.value().uri);
result.push_back({ parsed_uri.page - 1, parsed_uri.x, parsed_uri.y });
return result;
}
diff --git pdf_viewer/main_widget.cpp pdf_viewer/main_widget.cpp
index 19b568b..335a93f 100644
--- pdf_viewer/main_widget.cpp
+++ pdf_viewer/main_widget.cpp
@@ -170,7 +170,7 @@ void MainWidget::set_overview_position(int page, float offset) {
void MainWidget::set_overview_link(PdfLink link) {
- auto [page, offset_x, offset_y] = parse_uri(mupdf_context, link.uri);
+ auto [page, offset_x, offset_y] = parse_uri(mupdf_context, doc()->doc, link.uri);
if (page >= 1) {
set_overview_position(page - 1, offset_y);
}
@@ -3178,7 +3178,7 @@ void MainWidget::handle_pending_text_command(std::wstring text) {
open_web_url(utf8_decode(selected_link->uri));
}
else{
- auto [page, offset_x, offset_y] = parse_uri(mupdf_context, selected_link->uri);
+ auto [page, offset_x, offset_y] = parse_uri(mupdf_context, doc()->doc, selected_link->uri);
long_jump_to_destination(page-1, offset_y);
}
}
@@ -3863,7 +3863,7 @@ void MainWidget::handle_link_click(const PdfLink& link) {
return;
}
- auto [page, offset_x, offset_y] = parse_uri(mupdf_context, link.uri);
+ auto [page, offset_x, offset_y] = parse_uri(mupdf_context, doc()->doc, link.uri);
// convert one indexed page to zero indexed page
page--;
diff --git pdf_viewer/utils.cpp pdf_viewer/utils.cpp
index 22d4265..3150e66 100644
--- pdf_viewer/utils.cpp
+++ pdf_viewer/utils.cpp
@@ -116,8 +116,9 @@ bool rects_intersect(fz_rect rect1, fz_rect rect2) {
return range_intersects(rect1.x0, rect1.x1, rect2.x0, rect2.x1) && range_intersects(rect1.y0, rect1.y1, rect2.y0, rect2.y1);
}
-ParsedUri parse_uri(fz_context* mupdf_context, std::string uri) {
- fz_link_dest dest = pdf_parse_link_uri(mupdf_context, uri.c_str());
+ParsedUri parse_uri(fz_context* mupdf_context, fz_document* fz_doc, std::string uri) {
+ pdf_document* doc = pdf_document_from_fz_document(mupdf_context, fz_doc);
+ fz_link_dest dest = pdf_resolve_link_dest(mupdf_context, doc, uri.c_str());
return { dest.loc.page + 1, dest.x, dest.y };
}
diff --git pdf_viewer/utils.h pdf_viewer/utils.h
index 2a12211..643b030 100644
--- pdf_viewer/utils.h
+++ pdf_viewer/utils.h
@@ -55,7 +55,7 @@ void get_flat_toc(const std::vector<TocNode*>& roots, std::vector<std::wstring>&
int mod(int a, int b);
bool range_intersects(float range1_start, float range1_end, float range2_start, float range2_end);
bool rects_intersect(fz_rect rect1, fz_rect rect2);
-ParsedUri parse_uri(fz_context* mupdf_context, std::string uri);
+ParsedUri parse_uri(fz_context* mupdf_context, fz_document* fz_doc, std::string uri);
char get_symbol(int key, bool is_shift_pressed, const std::vector<char>&special_symbols);
template<typename T>