Ticket #11136 (closed enhancement: fixed)
Upgrade sqlite to the newest upstream release (3.7.5)
| Reported by: | drkirkby | Owned by: | tbd |
|---|---|---|---|
| Priority: | major | Milestone: | sage-4.7 |
| Component: | packages: standard | Keywords: | |
| Cc: | fbissey, mariah | Work issues: | |
| Report Upstream: | N/A | Reviewers: | Mariah Lenox, François Bissey |
| Authors: | David Kirkby | Merged in: | sage-4.7.alpha5 |
| Dependencies: | Stopgaps: |
Description (last modified by drkirkby) (diff)
The version in sqlite of Sage is 3 years old, so an update is in order. Several recent release have only been bug-fix-releases, so this should be very stable.
A new .spkg has been placed at
http://boxen.math.washington.edu/home/kirkby/patches/sqlite-3.7.5.spkg
Using this, Sage was built from scratch on OpenSolaris 06/2009. All long doctests passed too. But it needs testing on other platforms.
I've tested this on the following platforms.
- 32-bit OpenSolaris on x86 (hawk) using GCC 4.5.0. Complete build of Sage from the start. All doctests pass.
- 32-bit Solaris 10 on SPARC (t2.math) using gcc 4.5.1 Test of package building only.
- 64-bit Solaris 10 on SPARC (t2.math). GCC 4.5.1. Test of package building only. (64-bit SPARC does not result in a very stable Sage, so many doctest would fail even if sqlite worked perfectly.)
- 64-bit Ubuntu Linux (sage.math) using GCC 4.2.4. Build of package - not complete doc tests, although devel/sage/sage/graphs/graph_database.py was tested and passed.
- OS X 10.6 (bsd.math) Build with GCC 4.2.1
Dave
Attachments
Change History
comment:1 Changed 2 years ago by drkirkby
- Status changed from new to needs_review
- Description modified (diff)
- Authors set to David Kirkby
comment:4 Changed 2 years ago by drkirkby
- Description modified (diff)
Given this builds on at least the following:
- Operating systems: Linux, OS X, Solaris 10, OpenSolaris? & Solaris 11 Express
- CPUs: SPARC and x86
- 32-bit and 64-bit
as well as the fact this version of sqlite has been used on Gentoo Linux for some time, I believe this should be merged.
comment:7 Changed 2 years ago by fbissey
Sorry I couldn't review today - got swamped with work. Hopefully I will able to do a review tomorrow.
comment:8 follow-up: ↓ 9 Changed 2 years ago by mariah
- Cc mariah added
- Status changed from needs_review to needs_work
SPKG.txt has entries:
=== sqlite-3.6.22 (David Kirkby, March 1st 2010) === === sqlite-3.7.5 (David Kirkby, 6th April 2011) === === sqlite-3.6.19.p0 (Mike Hansen, October 26th, 2009) ===
Suggest order be changed to:
=== sqlite-3.7.5 (David Kirkby, 6th April 2011) === === sqlite-3.6.22 (David Kirkby, March 1st 2010) === === sqlite-3.6.19.p0 (Mike Hansen, October 26th, 2009) ===
Tested with 'make testlong' on sage-4.7.alpha3 with sqlite-3.7.5.spkg (built with gcc-4.5.1) on the following skynet machines:
cleo (ia64-Linux-rhel) iras (ia64-Linux-suse) eno (x86_64-Linux-core2-fc) lena (x86_64-Linux-k10-fc) flavius (x86_64-Linux-k8-fc) taurus (x86_64-Linux-nehalem-fc) sextus2 (x86_64-Linux-netburst-fc) cicero (x86-Linux-pentium4-fc)
All tests passed.
With the change above, I would give this a positive review.
Changed 2 years ago by drkirkby
-
attachment
11136-update-sqlite.patch
added
Changes, (which are only to SPKG.txt) for review purposes only.
comment:9 in reply to: ↑ 8 Changed 2 years ago by drkirkby
- Status changed from needs_work to needs_review
Replying to mariah:
SPKG.txt has entries:
=== sqlite-3.6.22 (David Kirkby, March 1st 2010) === === sqlite-3.7.5 (David Kirkby, 6th April 2011) === === sqlite-3.6.19.p0 (Mike Hansen, October 26th, 2009) ===Suggest order be changed to:
=== sqlite-3.7.5 (David Kirkby, 6th April 2011) === === sqlite-3.6.22 (David Kirkby, March 1st 2010) === === sqlite-3.6.19.p0 (Mike Hansen, October 26th, 2009) ===
Thank you for spotting that error. I've now corrected it.
Tested with 'make testlong' on sage-4.7.alpha3 with sqlite-3.7.5.spkg (built with gcc-4.5.1) on the following skynet machines:
cleo (ia64-Linux-rhel) iras (ia64-Linux-suse) eno (x86_64-Linux-core2-fc) lena (x86_64-Linux-k10-fc) flavius (x86_64-Linux-k8-fc) taurus (x86_64-Linux-nehalem-fc) sextus2 (x86_64-Linux-netburst-fc) cicero (x86-Linux-pentium4-fc)All tests passed.
Thanks, that's a good range of machines to to check this on.
With the change above, I would give this a positive review.
I'll leave you to double-check it and give the positive. I attached a revised Mercurial patch for review purposes, but they are now in the right order
drkirkby@hawk:~/newdocs/sage-4.7.alpha3/spkg/standard/sqlite-3.7.5$ grep "=== sqlite-3." SPKG.txt === sqlite-3.7.5 (David Kirkby, 6th April 2011) === === sqlite-3.6.22 (David Kirkby, March 1st 2010) === === sqlite-3.6.19.p0 (Mike Hansen, October 26th, 2009) === === sqlite-3.6.19.p0 (Mike Hansen, October 26th, 2009) === === sqlite-3.6.17.p0 (Dan Drake, October 25th, 2009) === === sqlite-3.6.17 (David Kirkby, August 16th, 2009) === === sqlite-3.5.3.p4 (David Kirkby, June 13th, 2009) === === sqlite-3.5.3.p3 (Michael Abshoff, May 19th, 2008) === === sqlite-3.5.3.p2 (William Stein, May 16th, 2008) === drkirkby@hawk:~/newdocs/sage-4.7.alpha3/spkg/standard/sqlite-3.7.5$
comment:10 Changed 2 years ago by fbissey
- Status changed from needs_review to positive_review
- Reviewers set to Mariah Lenox, François Bissey
Just to add to this, I tested on OS X 10.5 and the long tests are fine. Thanks to Mariah for stepping in with such a wide range of coverage. I am setting this to positive review.
comment:11 Changed 2 years ago by jdemeyer
- Status changed from positive_review to closed
- Resolution set to fixed
- Merged in set to sage-4.7.alpha5
