Ticket #11136 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

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

11136-update-sqlite.patch Download (797 bytes) - added by drkirkby 2 years ago.
Changes, (which are only to SPKG.txt) for review purposes only.

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:2 Changed 2 years ago by drkirkby

  • Description modified (diff)

comment:3 Changed 2 years ago by drkirkby

  • Description modified (diff)

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:5 Changed 2 years ago by drkirkby

  • Cc bissey added

comment:6 Changed 2 years ago by drkirkby

  • Cc fbissey added; bissey removed

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

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
Note: See TracTickets for help on using tickets.