Re: Visual Studio 2017 coming soon

2017-10-27 Thread Xidorn Quan
On Thu, Oct 26, 2017, at 12:24 PM, Ted Mielczarek wrote: > On the other hand, it's easier to > justify dropping support if VS is the last compiler holding us back from > being able to use new C++ features. FWIW, it's not at the moment. Currently we are really only blocked on GCC for new C++ featur

Re: Visual Studio 2017 coming soon

2017-10-27 Thread Kevin Brosnan
On Fri, Oct 27, 2017 at 12:03 AM, Xidorn Quan wrote: > On Thu, Oct 26, 2017, at 12:24 PM, Ted Mielczarek wrote: >> On the other hand, it's easier to >> justify dropping support if VS is the last compiler holding us back from >> being able to use new C++ features. > > FWIW, it's not at the moment.

Re: More ThinkStation P710 Nvidia tips (was Re: Faster gecko builds with IceCC on Mac and Linux)

2017-10-27 Thread Robert O'Callahan
On Fri, Oct 27, 2017 at 2:34 AM, Henri Sivonen wrote: > And the downsides don't even end there. rr didn't work. Plus other > stuff not worth mentioning here. > Turns out that rr not working with Nvidia on Ubuntu 17.10 was actually an rr issue triggered by the Ubuntu libc upgrade, not Nvidia's fa

Stylo is built as default even if Fennec/Android

2017-10-27 Thread Makoto Kato
Hi, all. You know, stylo (Quantum CSS) is turned on Firefox Desktop only. Stylo team is working very hard for Android too, then all reftests and mochitests are passed now even if Fennec/Android. So I would like to turn on stylo build on Android of Nightly channel for feedback. Although the prefe

Re: Better triage for intermittent leaks in tests?

2017-10-27 Thread Andrew McCreight
I half-heartedly keep an eye on intermittent leaks, so I can more active in looking at them. We'll need more logging for them to be actionable, but I guess the high frequency ones are easier to catch. Andrew On Thu, Oct 26, 2017 at 10:20 AM, Geoffrey Brown wrote: > Some of our most troublesome

Re: Stylo is built as default even if Fennec/Android

2017-10-27 Thread Nicholas Alexander
On Fri, Oct 27, 2017 at 6:24 AM, Makoto Kato wrote: > Hi, all. > > You know, stylo (Quantum CSS) is turned on Firefox Desktop only. > Stylo team is working very hard for Android too, then all reftests and > mochitests are passed now even if Fennec/Android. > > So I would like to turn on stylo bui

Re: WebIDL consts now generate C++ definitions

2017-10-27 Thread Steve Fink
On 10/26/17 8:22 PM, Kyle Machulis wrote: This addition also creates the new [NeedsWindowsUndef] extended attribute, as some WebIDL constant names conflict with windows.h macros, and undef'ing them in the binding generation is easier than tracking down include order issues. Gecko seems to handl

Re: WebIDL consts now generate C++ definitions

2017-10-27 Thread Boris Zbarsky
On 10/27/17 12:33 PM, Steve Fink wrote: Within Spidermonkey, our rule is to never #include windows.h directly, but always via jswin.h Ah, I thought I'd seen this sort of thing somewhere; I had recalled a header that you could include to under "all the bad windows.h stuff"... This approach do

Re: More ThinkStation P710 Nvidia tips (was Re: Faster gecko builds with IceCC on Mac and Linux)

2017-10-27 Thread Steve Fink
Not necessarily relevant to this specific discussion, but I'm on a Lenovo P50 running Linux, and wanted to offer up my setup as a datapoint. (It's not quite either a recommendation or a word of warning. A combination.) I use Linux (Fedora 25) as the host OS, with two external monitors plus th

Re: More ThinkStation P710 Nvidia tips (was Re: Faster gecko builds with IceCC on Mac and Linux)

2017-10-27 Thread Gabriele Svelto
On 27/10/2017 01:02, Gregory Szorc wrote: > Sophana (CCd) is working on a new system build right now. It will be based > on the i9's instead of dual socket Xeons and should be faster and cheaper. ... and lacking ECC memory. Please whatever CPU is chosen make sure it has ECC support and the machine

Intent to remove client.mk

2017-10-27 Thread Gregory Szorc
client.mk has existed since 1998 ( https://hg.mozilla.org/experimental/mozilla-central-cvs/rev/0a5e1fa423bd). Before `mach`, client.mk was the recommended way to build Firefox and perform other build tasks. client.mk has rarely changed in recent years because the build system maintainers have been

Re: More ThinkStation P710 Nvidia tips (was Re: Faster gecko builds with IceCC on Mac and Linux)

2017-10-27 Thread Gregory Szorc
Yeah. Only the Xeons and ThreadRipper (as our potential high core count machines) support ECC. rr, ECC, or reasonable costs: pick at most two :/ On Fri, Oct 27, 2017 at 4:08 PM, Sophana "Soap" Aik wrote: > Thanks Gabriele, that poses a problem then for the system build we have in > mind here as

Re: More ThinkStation P710 Nvidia tips (was Re: Faster gecko builds with IceCC on Mac and Linux)

2017-10-27 Thread Gabriele Svelto
On 28/10/2017 01:08, Sophana "Soap" Aik wrote: > Thanks Gabriele, that poses a problem then for the system build we have > in mind here as the i9's do not support ECC memory. That may have to be > a separate system with a Xeon. Xeon-W processors are identical to the i9 but come with more workstati

Re: Intent to remove client.mk

2017-10-27 Thread Kris Maglione
On Fri, Oct 27, 2017 at 04:16:01PM -0700, Gregory Szorc wrote: client.mk has existed since 1998 ( https://hg.mozilla.org/experimental/mozilla-central-cvs/rev/0a5e1fa423bd). Before `mach`, client.mk was the recommended way to build Firefox and perform other build tasks. client.mk has rarely change

Re: More ThinkStation P710 Nvidia tips (was Re: Faster gecko builds with IceCC on Mac and Linux)

2017-10-27 Thread Robert O'Callahan
BTW can someone forward this entire thread to their friends at AMD so AMD will fix their CPUs to run rr? They're tantalizingly close :-/. Rob -- lbir ye,ea yer.tnietoehr rdn rdsme,anea lurpr edna e hnysnenh hhe uresyf toD selthor stor edna siewaoeodm or v sstvr esBa kbvted,t rdsme,aoreseo

Re: Intent to remove client.mk

2017-10-27 Thread Nicholas Nethercote
This is excellent news. Relatedly, I want to particularly say that I think moz.build files are great. The syntax and semantics are very clear. They're easy to modify. They handle both simple cases and complex cases well. They pretty much never get in the way, which is exactly what a developer want