Closed
Bug 769930
Opened 12 years ago
Closed 12 years ago
Split PSM and NSS builds so that we can build NSS early
Categories
(Core :: Security, defect)
Core
Security
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: ekr, Assigned: ekr)
References
(Depends on 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Some of the WebRTC code invokes NSS directly, so we need to make the NSS build happen earlier.
The following patch applies against alder/trunk. We will eventually submit one against m-c.
Original patch by glandium. Modifed by EKR.
Assignee | ||
Updated•12 years ago
|
Attachment #638123 -
Attachment is patch: true
Updated•12 years ago
|
Attachment #638123 -
Flags: review?(khuey)
Updated•12 years ago
|
Assignee: nobody → ekr
Status: NEW → ASSIGNED
Comment on attachment 638123 [details] [diff] [review]
Patch
This is fine, but slightly longer term we need to figure out how webrtc should interact with --disable-crypto.
Attachment #638123 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 2•12 years ago
|
||
Still waiting on try server.
Attachment #638123 -
Attachment is obsolete: true
Assignee | ||
Comment 3•12 years ago
|
||
Try server results:
https://tbpl.mozilla.org/?tree=Try&rev=1e7810b16780
Assignee | ||
Updated•12 years ago
|
Attachment #639778 -
Flags: review?(mh+mozilla)
Comment 4•12 years ago
|
||
Comment on attachment 639778 [details] [diff] [review]
Revised patch, applies on m-c
I can't review what largely is my own patch ;)
Attachment #639778 -
Flags: review?(mh+mozilla) → review?(khuey)
Comment on attachment 639778 [details] [diff] [review]
Revised patch, applies on m-c
Review of attachment 639778 [details] [diff] [review]:
-----------------------------------------------------------------
I don't think there's anything I need to review again.
Attachment #639778 -
Flags: review?(khuey)
Comment 6•12 years ago
|
||
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #5)
> Comment on attachment 639778 [details] [diff] [review]
> Revised patch, applies on m-c
>
> Review of attachment 639778 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> I don't think there's anything I need to review again.
So where do we go from here ?
Comment 7•12 years ago
|
||
(In reply to Ludovic Hirlimann [:Usul] from comment #6)
> So where do we go from here ?
We land it.
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed614ea130c0
Comment 8•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in
before you can comment on or make changes to this bug.
Description
•