Please update bundled synctex
Bug #2006446 reported by
Florian Weimer
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
qpdfview |
Fix Committed
|
Medium
|
Adam Reichold |
Bug Description
Please include this commit in the bundled synctex sources:
commit 2897465154892a7
Author: Germán Poo-Caamaño <email address hidden>
Date: Sat May 30 16:15:27 2020 -0400
Fix implicit declaration of vasprintf
Fix build when compiling with `-Werror=
https:/
It is needed for C99 compatibility.
Related to:
<https:/
<https:/
Changed in qpdfview: | |
milestone: | none → 0.5.1 |
To post a comment you must log in.
Hello Florian,
thank you for taking the time to report this! While version 0.5.0 was just released, I will update the bundled sources for 0.5.1. Just two questions so that I do the right thing:
* I usually import those sources from the TeXLive Subversion repository. Is the linked commit specific to the GitHub fork or can I just import the latest code from TeXLive as usual?
* Since this is referencing the Fedora change: If present at build time, qpdfview should prefer to link against the system-provided SyncTeX and the vendored sources are considered a fallback for more exotic platforms which do not ship that library. Does using pkg-config not work for the Fedora package or does the package need an additional build-time dependency on the SyncTeX library? Ideally, the vendored should not be built on a modern Linux distribution like Fedora.
Regards,
Adam