713229 - configure only requires NSPR 4.8.8, but we require NSPR >= 4.9 beta 4

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Mike Hommey [:glandium]]

indexedDB code uses PR_UINT64, which is only available in NSPR 4.9 beta 4 and greater.

Comment 1

[Mike Hommey [:glandium]]

Attachment: Require NSPR 4.9 beta 4 or greater

Comment 3

[Mike Hommey [:glandium]]

Attachment: Require NSPR 4.9 beta 4 or greater

Comment 5

[Mike Hommey [:glandium]]

There's a patch in this bug already, let's duplicate the other way around.

Comment 6

[(not currently active) Ted Mielczarek]

Comment on attachment 584064 [details] [diff] [review]
Require NSPR 4.9 beta 4 or greater

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

We should really release an NSPR 4.9 :-/

Comment 7

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2a6cac678c94

Comment 8

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/2a6cac678c94

Comment 9

[Mike Hommey [:glandium]]

Comment on attachment 584064 [details] [diff] [review]
Require NSPR 4.9 beta 4 or greater

[Approval Request Comment]
The current codebase uses macros that are only present in NSPR 4.9 beta 4 and above.
This is a low risk, NPOTB patch that makes life easier for linux distributions using the --with-system-nspr flag. Making it clear if their NSPR is too old.

Comment 10

[Alex Keybl [:akeybl]]

Comment on attachment 584064 [details] [diff] [review]
Require NSPR 4.9 beta 4 or greater

[Triage Comment]
Low risk, NPOTB - approved for Aurora.

Comment 11

[Mike Hommey [:glandium]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/ccbae23f3d04

Comment 12

[Landry Breuil (:gaston)]

Fx 11.0b1 was released depending on an unreleased nspr version, thus breaks people using system nspr. Where's the tracking bug for 4.9.0 final release ?