[Bug 208678] Review Request: SimGear - Simulation library components

bugzilla at redhat.com bugzilla at redhat.com
Wed Mar 28 14:20:52 UTC 2007


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: SimGear - Simulation library components


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=208678





------- Additional Comments From j.w.r.degoede at hhs.nl  2007-03-28 10:20 EST -------
Running rpmlint on the installed SimGear package results in lots of undefined
non weak symbols warnings. It looks like the .so's need to be linked against
other simgear libs.

Also I think its not wise to use .1.0.0 as soname versioning. Since upstream
only supports static libs, and this is a c++ library, ABI breakage with a new
upstream release is _extremely_ likely. Thus I would like to advice to use
"-release %{version}" flag to libtool when linking the .so files instead of
"-version xxx"

Using -release will result in sonames like this:
libSimGear-%{version}.so

Thus changing the soname with every upstream release, which (unfortunately) most
likely is a good thing todo in this case. If you're ok with me becoming a
co-maintainer and you can add me to the ACL, then I can fix both the
non-weak-symbols and the soname issue and do an import then. I think its
important to get the soname right from the start.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list