On 2017-09-13 13:58:37, Andreas Tille wrote: > Hi Sebastian, > > On Wed, Sep 13, 2017 at 01:08:54PM +0200, Sebastian Ramacher wrote: > > > > not sure why you CC me, but … > > Since the patch for the previous version[1] was crafted by you. :-) > > > On 2017-09-13 12:46:40, Andreas Tille wrote: > > > > > > I tried to upgrade TeXMaker to the latest upstream version. When doing > > > so I migrated the packaging from SVN to Git - so you can find the > > > packaging here > > > > > > https://anonscm.debian.org/git/debian-science/packages/texmaker.git > > > > > > Unfortunately I have no idea how to include synctex properly into the > > > new build system. Could anybody have a look please? > > > > Remove the synctex file from HEADERS and SOURCES and add -lsynctex to LIBS > > Synctex files are not mentioned in HEADERS nor SOURCES. % grep -n synctex texmaker.pro 111: synctex_parser.h \ 112: synctex_parser_utils.h \ 496: synctex_parser.c \ 497: synctex_parser_utils.c \ But thos lines are removed by debian/patches/use-system-synctex.patch. > My try to add it to > LIBS which looked like this: > > > texmaker(master) $ git diff > diff --git a/debian/patches/use-system-synctex.patch b/debian/patches/use-system-synctex.patch > index 8ad99bf..dbd79dd 100644 > --- a/debian/patches/use-system-synctex.patch > +++ b/debian/patches/use-system-synctex.patch > @@ -33,6 +33,15 @@ Last-Update: 2014-07-05 > usertagslistwidget.cpp \ > addtagdialog.cpp \ > versiondialog.cpp \ > +@@ -1225,6 +1221,8 @@ UI_DIR = .ui > + MOC_DIR = .moc > + OBJECTS_DIR = .obj > + > ++LIBS += -lsynctex > ++ > + LIBS_PRIVATE += -framework AppKit -framework CoreFoundation > + > + QMAKE_MAC_SDK=macosx As the last line indicates, this part of the pro file is for Mac OS X. You need add that in the unix:!macx part. But by the looks of it, Leopold's patch does the correct thing. Cheers > --- a/pdfviewer.h > +++ b/pdfviewer.h > @@ -37,7 +37,7 @@ > > > did not changed anything. The build ends with: > > ... > .obj/pdfviewerwidget.o: In function `PdfViewerWidget::openFile(QString, QString, QString)': > ./pdfviewerwidget.cpp:317: undefined reference to `synctex_scanner_free' > ./pdfviewerwidget.cpp:347: undefined reference to `synctex_scanner_new_with_output_file' > .obj/pdfviewer.o: In function `PdfViewer::jumpToEditor(int, QPointF const&)': > ./pdfviewer.cpp:1019: undefined reference to `synctex_edit_query' > ./pdfviewer.cpp:1022: undefined reference to `synctex_next_result' > ./pdfviewer.cpp:1024: undefined reference to `synctex_node_tag' > ./pdfviewer.cpp:1024: undefined reference to `synctex_scanner_get_name' > ./pdfviewer.cpp:1026: undefined reference to `synctex_node_line' > .obj/pdfviewer.o: In function `PdfViewer::~PdfViewer()': > ./pdfviewer.cpp:308: undefined reference to `synctex_scanner_free' > .obj/pdfviewer.o: In function `PdfViewer::jumpToPdfFromSource(QString, int, int)': > ./pdfviewer.cpp:542: undefined reference to `synctex_scanner_input' > ./pdfviewer.cpp:554: undefined reference to `synctex_node_sibling' > ./pdfviewer.cpp:547: undefined reference to `synctex_node_tag' > ./pdfviewer.cpp:547: undefined reference to `synctex_scanner_get_name' > ./pdfviewer.cpp:561: undefined reference to `synctex_display_query' > ./pdfviewer.cpp:565: undefined reference to `synctex_next_result' > ./pdfviewer.cpp:568: undefined reference to `synctex_node_page' > ./pdfviewer.cpp:572: undefined reference to `synctex_node_box_visible_height' > ./pdfviewer.cpp:572: undefined reference to `synctex_node_box_visible_depth' > ./pdfviewer.cpp:571: undefined reference to `synctex_node_box_visible_width' > ./pdfviewer.cpp:570: undefined reference to `synctex_node_box_visible_v' > ./pdfviewer.cpp:570: undefined reference to `synctex_node_box_visible_height' > ./pdfviewer.cpp:569: undefined reference to `synctex_node_box_visible_h' > ./pdfviewer.cpp:567: undefined reference to `synctex_node_page' > .obj/pdfviewer.o: In function `PdfViewer::openFile(QString, QString, QString)': > ./pdfviewer.cpp:349: undefined reference to `synctex_scanner_free' > ./pdfviewer.cpp:379: undefined reference to `synctex_scanner_new_with_output_file' > collect2: error: ld returned 1 exit status > Makefile:2240: recipe for target 'texmaker' failed > > > But may be I added LIBS += -lsynctex at the wrong place? It does not > appear in the log. So may be some other place than texmaker.pro might > be what you intended to tell me. > > Kind regards > > Andreas. > > > [1] https://anonscm.debian.org/git/debian-science/packages/texmaker.git/tree/debian/patches/use-system-synctex.patch > > -- > http://fam-tille.de -- Sebastian Ramacher
Attachment:
signature.asc
Description: PGP signature