Roundup Robot added the comment:
New changeset 44fe1f5b07e3 by Serhiy Storchaka in branch '2.7':
Issue #17857: Prevent build failures with pre-3.5.0 versions of sqlite3,
http://hg.python.org/cpython/rev/44fe1f5b07e3
--
___
Python tracker
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset e31597b3bd15 by Ned Deily in branch '2.7':
Issue #14572: Prevent build failures with pre-3.5.0 versions of
http://hg.python.org/cpython/rev/e31597b3bd15
--
nosy: +python-dev
Ned Deily n...@acm.org added the comment:
OK, the patch, as originally suggested by Joakim, is applied for release in
2.7.4. Thanks everyone.
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker
Petri Lehtinen pe...@digip.org added the comment:
Isn't 3.2 or 3.3 affected by this?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14572
___
Ned Deily n...@acm.org added the comment:
Isn't 3.2 or 3.3 affected by this?
No, since the developer who made the original changes used sqlite3_int64 for
2.7 (37c824c3efe8) but chose to hardwire the type to PyLong_AsLongLong for 3.x
(2d16048b10cd).
--
Changes by Ned Deily n...@acm.org:
--
Removed message: http://bugs.python.org/msg161213
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14572
___
Ned Deily n...@acm.org added the comment:
Isn't 3.2 or 3.3 affected by this?
No, since the developer who made the original changes used sqlite3_int64 for
2.7 (789a3ea97083) but chose to hardwire the type to PyLong_AsLongLong for 3.x
(e67715b87131). [corrected ids]
--
Marc Abramowitz msabr...@gmail.com added the comment:
Ned, thanks for applying this patch!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14572
___
Marc Abramowitz msabr...@gmail.com added the comment:
OK, here's a patch for configure.ac which seems to fix this problem -- if folks
could review and test it that would be great.
--
keywords: +patch
Added file: http://bugs.python.org/file25634/sqlite3_int64.patch
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +dmalcolm
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14572
___
___
Python-bugs-list mailing
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
Marc Abramowitz msabr...@gmail.com added the comment:
My guess would be that the code was switched to use the new typedef because the
SQLite docs say they're preferred.
http://www.sqlite.org/c3ref/int64.html
Maybe they are planning to deprecate the old typedef at some point?
--
Marc Abramowitz msabr...@gmail.com added the comment:
Probably either approach will have the exact same effect for the foreseeable
future, so I don't feel strongly either way. It would be nice to have one of
them so folks can have a sqlite3 module without having to search around and
apply
Petri Lehtinen pe...@digip.org added the comment:
Changing the code to use sqlite_int64 (Ned's patch) sounds better to me.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14572
___
Marc Abramowitz msabr...@gmail.com added the comment:
Just to make this a tad easier, I put Joakim's patch into a gist:
[marca@logger01.prod1 Python-2.7.3]$ pwd
/home/marca/src/Python-2.7.3
[marca@logger01.prod1 Python-2.7.3]$ curl -sk
https://raw.github.com/gist/2727063/
Marc Abramowitz msabr...@gmail.com added the comment:
curl -sk https://raw.github.com/gist/2727063/ | patch -p1
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14572
___
Marc-Andre Lemburg m...@egenix.com added the comment:
Mac OS X 10.4 is also affected and for the same reason. SQLite builds fine for
Python 2.5 and 2.6, but not for 2.7.
--
nosy: +lemburg
title: 2.7.3: sqlite module does not build on centos 5 - 2.7.3: sqlite module
does not build on
Marc Abramowitz msabr...@gmail.com added the comment:
This patch worked for me as well. Thanks, Joakim!
$ cat /etc/redhat-release
CentOS release 5.5 (Final)
--
nosy: +Marc.Abramowitz
___
Python tracker rep...@bugs.python.org
New submission from Joakim Sernbrant serb...@gmail.com:
Python-2.7.3/Modules/_sqlite/connection.c: In function ‘_pysqlite_set_result’:
Python-2.7.3/Modules/_sqlite/connection.c:552: error: ‘sqlite3_int64’
undeclared (first use in this function)
The centos 5 version of sqlite3
19 matches
Mail list logo