Created attachment 176986 [details] patch libcrossguid.so lacks of a soname, so I added the soname to stop the qa stage complain: /usr/local/lib/libcrossguid.so which does not have a SONAME. devel/libcrossguid needs to be fixed. Furthermore, 'pkg check' shows: % pkg check -dn kodi is missing a required shared library: libcrossguid.so No output here: % objdump -p /usr/local/lib/libcrossguid.so | grep SONAME While I'm here, add some cosmetic fixes.
portlint -ac output looks fine. poudriere testport build logs are fine on 9.3/ia, 10.2/ia, 10.3/ia and 11.0/ia
This doesn't seem to be correct. The purpose of soname is to point linker to current ABI version of the library, in this case it should be libcrossguid.so.0. There's also no point in using soname in pkgconfig file, just add -lcrossguid there (and probably remove -lguid, seems like a typo).
Created attachment 176996 [details] patch v1
(In reply to Dmitry Marakasov from comment #2) You're right, Dmitry :) Thanks for checking the patch.
(In reply to Carlos J. Puga Medina from comment #4) > (In reply to Dmitry Marakasov from comment #2) > > You're right, Dmitry :) > > Thanks for checking the patch. -Wl,-soname,libcrossguid.so.0 ^^
Also I see no point in splitting post-install target - everything can be made in do-install.
Created attachment 176998 [details] patch v2 Done!
% objdump -p /usr/local/lib/libcrossguid.so | grep SONAME SONAME libcrossguid.so.0 % objdump -p /usr/local/lib/libcrossguid.so.0 | grep SONAME SONAME libcrossguid.so.0
Now 'pkg check -dn' looks fine. All required shared libraries are found. LGTM.
LGTM as well
Looks good to me too
Carlos, please commit.
A commit references this bug: Author: cpm Date: Thu Nov 24 11:31:33 UTC 2016 New revision: 427007 URL: https://svnweb.freebsd.org/changeset/ports/427007 Log: devel/libcrossguid: Add SONAME, cosmetic fixes - Add SONAME to libcrossguid.so - Cosmetic fixes - Bump PORTREVISION PR: 214499 Submitted by: cpm Reviewed by: Mickael Maillot <mickael.maillot@gmail.com> (maintainer), amdmi3 (mentor) Approved by: Mickael Maillot <mickael.maillot@gmail.com> (maintainer), amdmi3 (mentor) MFH: 2016Q4 Changes: head/devel/libcrossguid/Makefile head/devel/libcrossguid/files/libcrossguid.pc.in
(In reply to Dmitry Marakasov from comment #12) Committed, thanks!
A commit references this bug: Author: cpm Date: Fri Nov 25 00:40:40 UTC 2016 New revision: 427070 URL: https://svnweb.freebsd.org/changeset/ports/427070 Log: MFH: r427007 devel/libcrossguid: Add SONAME, cosmetic fixes - Add SONAME to libcrossguid.so - Cosmetic fixes - Bump PORTREVISION PR: 214499 Submitted by: cpm Reviewed by: Mickael Maillot <mickael.maillot@gmail.com> (maintainer), amdmi3 (mentor) Approved by: Mickael Maillot <mickael.maillot@gmail.com> (maintainer), amdmi3 (mentor) Approved by: ports-secteam (feld) Changes: _U branches/2016Q4/ branches/2016Q4/devel/libcrossguid/Makefile branches/2016Q4/devel/libcrossguid/files/libcrossguid.pc.in