Closed Bug 837799 Opened 11 years ago Closed 11 years ago

Allow building of NSS against older sqlite

Categories

(NSS :: Libraries, defect, P1)

3.14.2
defect

Tracking

(Not tracked)

RESOLVED FIXED
3.14.3

People

(Reporter: KaiE, Assigned: KaiE)

Details

Attachments

(1 file)

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.
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.
Attached patch Patch v1Splinter Review
who is willing to bless this patch? thanks in advance
Assignee: nobody → kaie
Attachment #709806 - Flags: superreview?(rrelyea)
Attachment #709806 - Flags: review?(wtc)
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.
Attachment #709806 - Flags: review?(wtc) → review+
Attachment #709806 - Flags: superreview?(rrelyea)
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
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.14.3
Priority: -- → P1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: