Ned Deily <n...@acm.org> added the comment:

Thanks for the patch to configure.ac. It appears to work on OS X 10.4 and it 
should on any other system with an older version of sqlite3 installed. However, 
I think a better approach is to just change the two problematic references in 
Modules/_sqlite/connection.c to the older backwards compatible form.  The 
current sqlite3.h file explicitly supports the older type for backward 
compatibility for exactly this kind of case.  If so, why add unnecessary magic 
to Python's configure?  Presumably the sqlite3 project will need to continue to 
support both definitions indefinitely.  Perhaps Petri can chime in here as he 
made the change to include the new types.  Attached is a new patch tested on OS 
X 10.4 and 10.7.

----------
nosy: +ned.deily, petri.lehtinen
stage:  -> patch review
Added file: http://bugs.python.org/file25647/sqlite3_int64_v2.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue14572>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to