Closed
Bug 1414297
Opened 7 years ago
Closed 7 years ago
Use VS2017 in NSS Windows gyp build
Categories
(NSS :: Build, enhancement)
NSS
Build
Tracking
(Not tracked)
RESOLVED
FIXED
3.35
People
(Reporter: away, Assigned: away)
References
Details
Attachments
(1 file)
4.42 KB,
patch
|
Details | Diff | Splinter Review |
As a followup to switching Firefox builds over to VS2017, I'm hoping to update every use of "vs2015u3" I can find in the tree.
I couldn't find anything in the Gecko Try that would exercise this build, so I made these changes blindly, just based on the directory structure of the releng package. Tim, is this something that can be tested on NSS Try?
Assignee: nobody → dmajor
Flags: needinfo?(ttaubert)
Comment 2•7 years ago
|
||
Yeah, you can push it to nss-try [1], it's an L1-accessible service just like normal try. But you'll need an NSS checkout. Let me do that :) https://treeherder.mozilla.org/#/jobs?repo=nss-try&revision=2cf297730755bb99ae70fb16874770243dcfe8c1 [1] https://wiki.mozilla.org/NSS:TryServer
Flags: needinfo?(ttaubert)
Comment 3•7 years ago
|
||
Shall we go ahead and land this? Looks good to me.
Flags: needinfo?(ttaubert)
Comment 6•7 years ago
|
||
https://hg.mozilla.org/projects/nss/rev/ba3a668b2e3d Thanks David!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.35
Thanks for the review and landing! Any idea when this should reach mozilla-central? (No rush, just curious)
Comment 8•7 years ago
|
||
We'll land 3.35 after the m-c merge to beta. It should hit Nightly this week.
Comment 9•7 years ago
|
||
Oh and btw, this won't actually affect Firefox. This is only used on the NSS-specific Taskcluster infrastructure. You might know that already, just wanted to point it out :)
You need to log in
before you can comment on or make changes to this bug.
Description
•