Skip to content

libcupsfilters.pc: Add pdfio as public Requires dependency - #219

Open
princemittalr wants to merge 1 commit into
OpenPrinting:masterfrom
princemittalr:fix/pdfio-public-requirement
Open

libcupsfilters.pc: Add pdfio as public Requires dependency#219
princemittalr wants to merge 1 commit into
OpenPrinting:masterfrom
princemittalr:fix/pdfio-public-requirement

Conversation

@princemittalr

Copy link
Copy Markdown

Fixes #213

pdf.h is a public installed header that includes <pdfio.h> directly:

#include <pdfio.h>

This means downstream packages (e.g. libppd) that include pdf.h need
pdfio headers and types at compile time. However, pdfio was only listed
in Libs.private, which pkg-config does not propagate to consumers.

Fix: add pdfio to Requires and remove it from Libs.private to avoid
double-linking.

pdf.h (a public installed header) includes <pdfio.h> directly, which
means downstream packages (e.g. libppd) that include pdf.h need pdfio
headers and types at compile time. Moving pdfio from Libs.private to
Requires ensures pkg-config propagates this dependency correctly.

Fixes: OpenPrinting#213
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.

Missing requirements on pdfio for libcupsfilters.pc

1 participant