Skip to content

Support for PoDoFo 1.x#847

Open
iczero wants to merge 4 commits intohorizon-eda:masterfrom
iczero:podofo-1.x
Open

Support for PoDoFo 1.x#847
iczero wants to merge 4 commits intohorizon-eda:masterfrom
iczero:podofo-1.x

Conversation

@iczero
Copy link
Copy Markdown

@iczero iczero commented Mar 15, 2026

I've updated API usage following https://github.com/podofo/podofo/blob/master/API-MIGRATION.md. Exporting a basic pdf seems to work fine and looks identical to one exported using 0.10.x. Unfortunately I am an extreme beginner to this so I have not tested more complex cases.

Fixes #815

meson.build Outdated
podofo = dependency('libpodofo')
if podofo.version() == 'unknown' or podofo.version() < '0.10.0'
podofo_legacy = true
elif podofo.version() < '1.0.0'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the version macros provided by podofo instead.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

undo whitespace changes or put then in their own commit

@carrotIndustries
Copy link
Copy Markdown
Member

carrotIndustries commented Mar 27, 2026

Probably also makes sense to have at least one of the CI jobs have podofo in a version that's new enough. IIRC opensuse should have podofo 1.0 available.

@iczero
Copy link
Copy Markdown
Author

iczero commented Apr 6, 2026

Fedora has also switched podofo-devel to 1.x, should I add this to CI? 0.10 is still available as podofo0.10-devel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compile error with libpodofo 1.0.1

2 participants