837799 - Allow building of NSS against older sqlite

Status: RESOLVED FIXED

(NSS :: Libraries, defect, P1)

Description

[Kai Engert [:KaiE:]]

In bug 578561 we started to make use of the new SQLITE_FCNTL_TEMPFILENAME file control.

However, because that patch made use of that symbol, we currently have a build time dependency on newer NSS.

In some stable environments it might be required to still build against older sqlite - and we shouldn't block people from being able to build newer NSS.

I propose that we remove the build time dependency, and instead use the following code in NSS:

#ifndef SQLITE_FCNTL_TEMPFILENAME
#define SQLITE_FCNTL_TEMPFILENAME 16
#endif

Based on the description that Richard gave in bug 578561 comment 53 I conclude that sqlite never changes the numeric values of such codes, and that it's therefore fine to use the raw number.

Comment 1

[Kai Engert [:KaiE:]]

FYI, I'm proposing this addition to NSS, in order to allow building of newer NSS against older versions of sqlite on older maintained Linux branches, which want to pick up NSS for security reasons, but are not ready to pick up newer sqlite.

Comment 2

[Kai Engert [:KaiE:]]

Attachment: Patch v1

who is willing to bless this patch? thanks in advance

Comment 3

[Wan-Teh Chang]

Comment on attachment 709806 [details] [diff] [review]
Patch v1

Review of attachment 709806 [details] [diff] [review]:
-----------------------------------------------------------------

r=wtc. I agree we should do this.

Please add a comment:
  /* SQLITE_FCNTL_TEMPFILENAME was added in SQLite 3.7.15.*/
or
  /* Added in SQLite 3.7.15.*/

This will help future NSS maintainers know when this can be
removed.

Comment 4

[Kai Engert [:KaiE:]]

Wan-Teh, thank you very much for the quick review.

Checked in including comment.

Checking in sdb.c;
/cvsroot/mozilla/security/nss/lib/softoken/sdb.c,v  <--  sdb.c
new revision: 1.31; previous revision: 1.30
done