Bug 540307 - Request for newer version of libkml
Summary: Request for newer version of libkml
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: libkml
Version: 14
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Rakesh Pandit
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-11-23 01:01 UTC by Keith Sharp
Modified: 2012-02-08 15:29 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-12-29 08:18:11 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
spec file updated for 1.2.0 (5.23 KB, text/x-rpm-spec)
2012-02-06 00:31 UTC, Jean-Eudes ONFRAY
no flags Details
patch #1 (47.14 KB, patch)
2012-02-06 00:32 UTC, Jean-Eudes ONFRAY
no flags Details | Diff
patch #2 (18.83 KB, patch)
2012-02-06 00:32 UTC, Jean-Eudes ONFRAY
no flags Details | Diff
patch #3 (861 bytes, patch)
2012-02-06 00:33 UTC, Jean-Eudes ONFRAY
no flags Details | Diff
spec file updated for 1.2.0 (6.04 KB, text/x-rpm-spec)
2012-02-08 12:52 UTC, Jean-Eudes ONFRAY
no flags Details
spec file updated for 1.2.0 (5.97 KB, text/x-rpm-spec)
2012-02-08 15:29 UTC, Jean-Eudes ONFRAY
no flags Details

Description Keith Sharp 2009-11-23 01:01:04 UTC
The current version of libkml in Rawhide is 0.6.1, the version on Google Code is 1.1.  This new version has bindings for Python, could libkml be upgraded to the most recent version and the Python bindings packaged?

Comment 1 Bug Zapper 2010-03-15 13:17:36 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 13 development cycle.
Changing version to '13'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

Comment 2 Rakesh Pandit 2010-05-28 12:33:02 UTC
Finally looking into it. They have this big issue that they use a third party folder and show many dependencies into it and it is lot of rework to patch build system and get them depend upon what we already provide.

Updating the old patch ....

Regards,
Rakesh Pandit

Comment 3 Bug Zapper 2010-07-30 10:47:50 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 14 development cycle.
Changing version to '14'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

Comment 4 Rakesh Pandit 2010-12-29 08:18:11 UTC
There has been no change on this front. Upstream continues using third party folder and redoing the autotool patch is lot of work now for f14. I will take this up in some time for rawhide again.

Regards,
Rakesh Pandit

Comment 5 Jean-Eudes ONFRAY 2012-02-06 00:31:06 UTC
Created attachment 559517 [details]
spec file updated for 1.2.0

Here's a series of patches to upgrade to latest version (1.2.0)

 - spec file updated.
 - third_party removal patch updated
 - new patch to use our minizip instead of google's
 - swig-1.3.35-check patch updated

Comments:
 - in spec %files
       %{python_sitearch}/*.pyc
       %{python_sitearch}/*.pyo
   not created in my test build so I disabled these.
 - now %{_datadir}/java/LibKML.jar is build so I added it to spec
 - patch #2 should be reviewed and probably slightly reworked.
     Google has changed some of minizip internals
     This patch rework this so we can use ours.
     4 files created for this purpose. They were taken from
     http://code.trak.dk/ as per Google's minizip changes.
     They are missing appropriate headers (licence/copyright)

Comment 6 Jean-Eudes ONFRAY 2012-02-06 00:32:03 UTC
Created attachment 559518 [details]
patch #1

Comment 7 Jean-Eudes ONFRAY 2012-02-06 00:32:44 UTC
Created attachment 559519 [details]
patch #2

Comment 8 Jean-Eudes ONFRAY 2012-02-06 00:33:07 UTC
Created attachment 559520 [details]
patch #3

Comment 9 Jean-Eudes ONFRAY 2012-02-08 12:52:23 UTC
Created attachment 560256 [details]
spec file updated for 1.2.0

Here's a rework of spec file.

Items changed:
- Requires:       expat, uriparser, zlib, minizip
- package -devel Requires: boost-devel (see bug #787388)
- separate -python package (with appropriate requires)
- separate -java package (with appropriate requires)
- Remove this part
<<
# move libs to kml (for keeping third party libs away)
mkdir -p $RPM_BUILD_ROOT%{_libdir}/libkml/
mv $RPM_BUILD_ROOT%{_libdir}/*.so* $RPM_BUILD_ROOT%{_libdir}/libkml/
>>
because to me, it breaks ldd (IIUC it'd require adding libkml/ to ld.so.conf) swig libs are already created in this subdir.
- chrpath removal adjusted to appropriate packages: only python libs require this
- %files adjusted for new packages

Note:
I don't understand why %{_libdir}/libkml/*.so is included in -devel as it's already in main package (that's itself required by -devel) and symlinks to *.so.0.0.0
If left it as-is. You can adjust this in changing /.so.*/ to /.so*/ and removing /*.so/ from %files devel

Comment 10 Jean-Eudes ONFRAY 2012-02-08 15:29:04 UTC
Created attachment 560295 [details]
spec file updated for 1.2.0


Sorry, one more take. Package built with spec in #9 above fail to install:

error: Failed dependencies:
	libkmlbase_swig_java.so.0()(64bit) is needed by libkml-devel-1.2.0-1.fc16.x86_64
	libkmlbase_swig_python.so.0()(64bit) is needed by libkml-devel-1.2.0-1.fc16.x86_64
	libkmldom_swig_java.so.0()(64bit) is needed by libkml-devel-1.2.0-1.fc16.x86_64
	libkmldom_swig_python.so.0()(64bit) is needed by libkml-devel-1.2.0-1.fc16.x86_64
	libkmlengine_swig_java.so.0()(64bit) is needed by libkml-devel-1.2.0-1.fc16.x86_64
	libkmlengine_swig_python.so.0()(64bit) is needed by libkml-devel-1.2.0-1.fc16.x86_64

See Note: above. I removed these and everything works now. I'm compiling & linking fine now. Sorry for the noise.


Note You need to log in before you can comment on or make changes to this bug.