Created attachment 273449 [details] Fix xlnt.pc Without this patch, xlnt.pc is generated with the following lines: includedir=include libdir=lib and when a program uses it with pkg-config, it returns -Iinclude -Llib which of course is invalid. The proposed patch adds the missing prefix, and now pkg-config --libs xlnt returns "-L/usr/local/lib -lxlnt" and pkg-config --cflags xlnt returns "-I/usr/local/include" for my case with e.g. $PREFIX=/usr/local. PR submitted upstream: see https://github.com/tfussell/xlnt/pull/765
Thanks for the patch! LGTM.
Committed, thanks!
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=86bd6000e673f2ad01ef9d8357699e507b251587 commit 86bd6000e673f2ad01ef9d8357699e507b251587 Author: Thierry Thomas <thierry@FreeBSD.org> AuthorDate: 2026-08-04 15:09:59 +0000 Commit: Thierry Thomas <thierry@FreeBSD.org> CommitDate: 2026-08-04 17:33:19 +0000 textproc/xlnt: fix xlnt.pc The corresponding PR has been submitted upstream: see https://github.com/tfussell/xlnt/pull/765 PR: 297275 Approved by: yuri (maintainer) textproc/xlnt/Makefile | 1 + textproc/xlnt/files/patch-source_CMakeLists.txt (new) | 13 +++++++++++++ 2 files changed, 14 insertions(+)
Yuri, I received a comment at https://github.com/tfussell/xlnt/pull/765#issuecomment-5183294594 They say that the repo tfussell/xlnt is unmaintained, and that there is an active fork at https://github.com/xlnt-community/xlnt Maybe you should modify the port to switch to this one?