Bug 8576 - Included poppler is old
Summary: Included poppler is old
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Printing (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.19.0
Assignee: Pierre Ossman
URL:
Keywords: adaha_tester, relnotes
Depends on: 8573 8577
Blocks: 8361
  Show dependency treegraph
 
Reported: 2025-04-17 09:20 CEST by Pierre Ossman
Modified: 2025-04-28 16:22 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2025-04-17 09:20:52 CEST
The poppler we include in the ThinLinc client for thinlinc is rather old. We last updated it in 2017.

We should update to a current version to get upstream fixes.
Comment 1 Pierre Ossman cendio 2025-04-17 09:21:32 CEST
This is needed to resolve bug 8361 since the current version doesn't compile with modern gcc.
Comment 2 Pierre Ossman cendio 2025-04-17 09:25:43 CEST
Poppler has switched to CMake now, which we haven't used normally in the ThinLinc build before (TigerVNC uses CMake, but heavily modified for us).

We need to come up with some new methods to build everything correctly there.
Comment 3 Pierre Ossman cendio 2025-04-24 11:02:25 CEST
Apparently, we have a bit of a dependency loop with bug 8361. The old poppler doesn't compile with a new gcc, but the new poppler doesn't compile with an old gcc (because it wants better C++ and C versions).

Let's see if we can patch something to work.
Comment 4 Pierre Ossman cendio 2025-04-24 15:02:36 CEST
The current version of Poppler makes heavy use of new C++ features. It doesn't look feasible to adjust that.

The latest release that still uses C++11 is poppler 21.11.0. It still uses std::mutex, though, which is broken for Windows (bug 8419).

It seems like I can revert that, though, and get it to build. I can even get 21.12.0 running that way.

Let's see that this version of poppler still works with an updated gcc.
Comment 5 Pierre Ossman cendio 2025-04-24 16:47:51 CEST
Seems to work, so let's go ahead with that intermediate version of poppler.
Comment 6 Pierre Ossman cendio 2025-04-25 13:14:06 CEST
Did test prints on Fedora 42 and Windows 11, and the prints look the same as before, with one exception; the alpha/background issue on Linux seems to be resolved (mentioned in bug 5347).
Comment 11 Adam Halim cendio 2025-04-28 16:22:17 CEST
Tested client build 3920 on Windows 10, macOS 15 and Fedora 41.

Printed some of our test pdfs, could not see any issues.

Also tested bug 7455.

Commits look good, closing.

Note You need to log in before you can comment on or make changes to this bug.