Bug 84917 - [patch] audio/id3lib - fix shared lib linking
Summary: [patch] audio/id3lib - fix shared lib linking
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
Depends on:
Reported: 2005-08-14 19:00 UTC by jflemer
Modified: 2005-09-08 09:35 UTC (History)
1 user (show)

See Also:

audio-id3lib.diff (1.85 KB, patch)
2005-08-14 19:00 UTC, jflemer
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description jflemer 2005-08-14 19:00:27 UTC
	As currently built, libid3.so is linked using cc, not c++.  Since
	ID3lib is C++ code, this leaves many undefined C++ symbols in the
	shared lib.  Similarly, libz is not linked when building libid3.so,
	leaving those symbols undefined as well.

	The responsibility to link with c++ and libz now falls on the
	application using ID3lib.  Since ID3lib is supposed to provide
	a C interface, having to link with c++ is not acceptable.

Fix: #
# o Use libtool 1.5 so that the shared lib is linked with c++
#   not cc.
# o Using libtool 1.5 generates one more symlink for the shared
#   libs, add it to the plist.
# o Remove libtool 1.3 hacks from the configure script.
# o Patch configure script to link libz with libid3 so that
#   the responsibility does not fall on the application using
#   id3lib.
How-To-Repeat: 	Observe all the missing symbols with the following test.

	--- broke.c ---
	#include <id3.h>
	int main() {
		return 0;
	--- end ---
	cc -I/usr/local/include -L/usr/local/lib broke.c -lid3
Comment 1 Sergey Matveychuk freebsd_committer 2005-08-18 21:05:48 UTC
Dear maintainer!

Please, take the PR into account.

Comment 2 Kirill Ponomarev freebsd_committer 2005-09-08 09:35:04 UTC
State Changed
From-To: open->closed

Committed, thanks!