Bug 263221 - python-sqlite symbol collision back
Summary: python-sqlite symbol collision back
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: python
Version: rawhide
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ---
Assignee: James Antill
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F8Target
TreeView+ depends on / blocked
 
Reported: 2007-08-29 13:04 UTC by Joe Orton
Modified: 2007-11-30 22:12 UTC (History)
1 user (show)

Fixed In Version: 2.5-15.fc7
Clone Of:
Environment:
Last Closed: 2007-11-22 03:30:09 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
minimal fix (1.75 KB, patch)
2007-10-25 10:30 UTC, Joe Orton
no flags Details | Diff

Description Joe Orton 2007-08-29 13:04:58 UTC
bug 231848 is back with the sqlite module integrated into python.  symclash says:

# ./symclash.py /usr/lib64/python2.5/lib-dynload/*.so \
                /usr/lib64/httpd/modules/*.so

Clashes by library:

Clashes for /usr/lib64/python2.5/lib-dynload/timemodule.so:
  with /usr/lib64/python2.5/lib-dynload/datetime.so => inittime inittimezone

Clashes for /usr/lib64/python2.5/lib-dynload/_sqlite3.so:
  with /usr/lib64/httpd/modules/mod_mem_cache.so => cache_init

Clashes for /usr/lib64/httpd/modules/mod_mem_cache.so:
  with /usr/lib64/python2.5/lib-dynload/_sqlite3.so => cache_init

Clashes for /usr/lib64/python2.5/lib-dynload/datetime.so:
  with /usr/lib64/python2.5/lib-dynload/timemodule.so => inittime inittimezone

Clashes for /usr/lib64/python2.5/lib-dynload/_ctypes_test.so:
  with /usr/lib64/python2.5/lib-dynload/_ctypes.so => module_methods

Clashes for /usr/lib64/python2.5/lib-dynload/_ctypes.so:
  with /usr/lib64/python2.5/lib-dynload/_ctypes_test.so => module_methods

Comment 1 Joe Orton 2007-08-29 13:07:35 UTC
(reassigned, since I already have a bug to track getting this fixed in httpd; it
needs to be fixed in _sqlite3.so too)

Comment 2 Joe Orton 2007-10-25 10:30:31 UTC
Created attachment 237231 [details]
minimal fix

This is a minimal patch for the issue.	(tested to build only)

Note that this is a regression since F7 and it breaks several web applications.

Comment 3 Joe Orton 2007-10-25 10:31:56 UTC
... and hence, please fix for F8.

Comment 4 James Antill 2007-10-25 14:54:02 UTC
 How is this a regression, because we didn't ship sqlite in Fed-7 and do in Fed-8?

 The sqlite module in python looks like a complete namespace violation, AFAICS
the only thing that should be exported directly from the entire .so is
init_sqlite3() ... even the same file that exports "cache_init" also exports
"new_node" and "cache_get" *sigh*.
 So I'm hesitant to add just a tiny patch that fixes this one symbol clash.

 I can fairly easily get it down to a reasonable number of better named
functions (init_sqlite3 + one pysqlite_* for each file), but even then I don't
want to put it into our package unless upstream takes it -- as it's bound to
cause conflict as we update to newer upstreams.


Comment 5 Joe Orton 2007-10-25 18:07:32 UTC
python-sqlite2-2.3.3 is in F7.  And in fact, it's also still in F8!  Which is fubar.

I gave a minimal patch with minimal risk.  A complete fix would be upgrade the
copy of python-sqlite bundled in the Python tarball to 2.3.3.  Or to stop
shipping the old, broken copy of sqlite2 as well as the new, fixed one.

Comment 6 James Antill 2007-10-30 18:25:02 UTC
 Ok, I see that this patch is a minimal pysqlite2-2.3.3 update ... so I'm much
happier with the patch from comment#2.

 And a Fed-8 blocker issue came up that required a python update, so I added the
patch and it should be in GA.


Comment 7 Fedora Update System 2007-11-09 23:51:17 UTC
python-2.5-15.fc7 has been pushed to the Fedora 7 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update python'

Comment 8 Fedora Update System 2007-11-22 03:29:58 UTC
python-2.5-15.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.


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