A year ago textproc/clucene-contrib was integrated into textproc/clucene, but the pkgconfig patch was not updated to include the new library. Fix: This patch replaces patch-src__core__libclucene-core.pc.cmake --- src/core/libclucene-core.pc.cmake | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) How-To-Repeat: # pkg-config --libs "libclucene-core" -L/usr/local/lib/ -lclucene-core -lclucene-shared
Responsible Changed From-To: freebsd-ports-bugs->office Over to maintainer (via the GNATS Auto Assign Tool)
A commit references this bug: Author: robak Date: Sun Mar 15 08:50:23 UTC 2015 New revision: 381307 URL: https://svnweb.freebsd.org/changeset/ports/381307 Log: textproc/clucene: fix missing lib in patch PR: 185567 Submitted by: Olivier <bugs-freebsd@interfasys.ch> Changes: head/textproc/clucene/Makefile head/textproc/clucene/files/patch-src__core__libclucene-core.pc.cmake
Committed (adjusting the patch as necessary), thanks for your work!
I don't understand the reason for this change. As you can see from the file name, libclucene-core.pc is for *libclucene-core*. Maybe you wanted to add libclucene-contrib.pc? Anyway, this change must be reverted.
% ldd /usr/local/lib/libclucene-core.so.1 /usr/local/lib/libclucene-core.so.1: libclucene-shared.so.1 => /usr/local/lib/libclucene-shared.so.1 (0x8017e8000) libz.so.6 => /lib/libz.so.6 (0x801a09000) libc++.so.1 => /usr/lib/libc++.so.1 (0x801c1f000) libcxxrt.so.1 => /lib/libcxxrt.so.1 (0x801edf000) libm.so.5 => /lib/libm.so.5 (0x8020fc000) libgcc_s.so.1 => /lib/libgcc_s.so.1 (0x802325000) libthr.so.3 => /lib/libthr.so.3 (0x802533000) libc.so.7 => /lib/libc.so.7 (0x800822000) As you can see, patch-src__core__libclucene-core.pc.cmake was added by me to note that libclucene-core.so requires libclucene-shared.so. https://svnweb.freebsd.org/changeset/ports/315888 However, libclucene-core.so does NOT require libclucene-contribs-lib.so.
Sorry but I had to revert the commit. https://svnweb.freebsd.org/changeset/ports/381438 Please let us know if you disagree with the decision.
@Jung-uk Kim, on January 28th 2013, you wrote: "Remove textproc/clucene-contrib because it is now integrated into textproc/clucene." So where is it? # pkg-config --libs "libclucene-contrib" Package libclucene-contrib was not found... # pkg-config --libs "libclucene-core" -L/usr/local/lib/ -lclucene-core -lclucene-shared My patch was simply to add the missing component to the location where it would be looked for, but you can add it wherever you want, as long as it's added somewhere.
(In reply to Olivier - interfaSys sàrl from comment #7) > @Jung-uk Kim, on January 28th 2013, you wrote: "Remove textproc/clucene-contrib because it is now integrated into textproc/clucene." > > So where is it? > > # pkg-config --libs "libclucene-contrib" > Package libclucene-contrib was not found... It never existed because the upstream never released one. Also, we didn't have one when textproc/clucene-contrib was available. https://svnweb.freebsd.org/ports/head/textproc/clucene-contrib/pkg-plist?revision=300897&pathrev=311136 > # pkg-config --libs "libclucene-core" > -L/usr/local/lib/ -lclucene-core -lclucene-shared As I said, I don't see anything wrong here, i.e., it correctly lists all necessary linker options. In fact, we don't want to link everything with -lclucene-contribs-lib. Overlinking is evil. > My patch was simply to add the missing component to the location where it > would be looked for, but you can add it wherever you want, as long as it's > added somewhere. You may write a patch to add libcluene-contrib-libs.pc as I said. However, it won't be useful unless the upstream accepts the patch and releases it, which is very unlikely to happen at this point.
(In reply to Jung-uk Kim from comment #8) > You may write a patch to add libcluene-contrib-libs.pc as I said. Of course, I meant to say "libcluene-contribs-lib.pc". Sorry.