Re: Gmail calendar information for Platform Engineering meeting

2014-12-19 Thread Axel Hecht
On 12/19/14 6:20 AM, Chris Peterson wrote: Here is the new Gmail calendar information for the weekly Platform Engineering meeting. If these calendar links don't work, please let me know. * iCal ics link: https://www.google.com/calendar/ical/mozilla.com_p51ksu9ddgqt72f0jl21vaq76o%40group.calend

Re: Git pushes to mercurial, early testers needed

2014-12-19 Thread Nicolas B. Pierron
On 12/18/2014 12:13 PM, Mike Hommey wrote: I just published initial support for pushing to mercurial from git. It's still experimental and as such I'm looking for volunteers who would want to try it and confirm that it doesn't break stuff. See http://glandium.org/blog/?p=3405 for more details.

Re: Git pushes to mercurial, early testers needed

2014-12-19 Thread Nicolas B. Pierron
On 12/18/2014 12:13 PM, Mike Hommey wrote: I just published initial support for pushing to mercurial from git. It's still experimental and as such I'm looking for volunteers who would want to try it and confirm that it doesn't break stuff. See http://glandium.org/blog/?p=3405 for more details.

Re: PSA: Flaky timeouts in mochitest-plain now fail newly added tests

2014-12-19 Thread Ehsan Akhgari
Let me try to rephrase the problem in different terms, to hopefully make it clearer why using timers like this is a bad idea. setTimeout(foo, 1000) may seem to suggest run foo after 1 second, but that is *not* what that function does, at all. What it does is, run foo after 1 second, or perhap

Re: Updating the policy for Talos performance regression in 2015

2014-12-19 Thread Ehsan Akhgari
This looks good overall. Two questions though: On 2014-12-18 6:47 AM, jmaher wrote: Mozilla - 2015 Talos performance regression policy Over the last year and a half the Talos tests have been rewritten to be more useful and meaningful. This means we need to take them seriously and cannot jus

Re: Updating the policy for Talos performance regression in 2015

2014-12-19 Thread Benjamin Smedberg
On 12/19/2014 10:05 AM, Ehsan Akhgari wrote: Acceptable outcomes: * A promise to attempt a fix at the bug is agreed upon, the bug is assigned to someone and put in a queue. How do we ensure that the follow-up bug actually does get fixed and it fixes the regression completely? Avi/Vladan

Re: PSA: Support for Visual C++ 2010 has been dropped

2014-12-19 Thread Neil
Neil wrote: Neil wrote: Mike Hommey wrote: On Wed, Dec 17, 2014 at 06:06:25PM +, Neil wrote: I downloaded the MSVC 2013 Community Edition, but there was no sign of an SDK, so I downloaded that separately. Is this expected? If so, I'll update MDN. The SDK comes with it. So you say,

Re: PSA: Flaky timeouts in mochitest-plain now fail newly added tests

2014-12-19 Thread Gijs Kruitbosch
On 19/12/2014 14:56, Ehsan Akhgari wrote: Logging sufficiently is almost always enough to not have to use these timers, as those tests have demonstrated in practice. Who's working on improving the log output from tinderbox? Timestamps get mashed [0], sometimes only the failing assertion is pri

Re: Gmail calendar information for Platform Engineering meeting

2014-12-19 Thread Chris Peterson
On 12/19/14 12:25 AM, Axel Hecht wrote: On 12/19/14 6:20 AM, Chris Peterson wrote: Here is the new Gmail calendar information for the weekly Platform Engineering meeting. If these calendar links don't work, please let me know. * iCal ics link: https://www.google.com/calendar/ical/mozilla.com_

Re: PSA: Flaky timeouts in mochitest-plain now fail newly added tests

2014-12-19 Thread Ehsan Akhgari
On Fri, Dec 19, 2014 at 11:34 AM, Gijs Kruitbosch wrote: > > On 19/12/2014 14:56, Ehsan Akhgari wrote: > >> Logging sufficiently is >> almost always enough to not have to use these timers, as those tests >> have demonstrated in practice. >> > > Who's working on improving the log output from tinder

Re: Updating the policy for Talos performance regression in 2015

2014-12-19 Thread jmaher
Great questions folks. :bsmedberg has answered the questions quite well, let me elaborate: Before a bug can be marked as resolved:fixed we need to verify the regression is actually fixed. In many cases we will fix a large portion of the regression and accept the small remainder. We do keep tra

Is anyone still using JS strict warnings?

2014-12-19 Thread Jason Orendorff
So if you go to about:config and set the javascript.options.strict pref, you'll get warnings about accessing undefined properties. js> Math.TAU undefined /!\ ReferenceError: reference to undefined property Math.TAU (It says "ReferenceError", but your code still runs normally; it reall

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread J. Ryan Stinnett
Some prior discussion of this feature happened in the platform thread "Disabling strict warnings as errors in xpcshell"[1]. A few people argued for the extra warnings to be removed, while one person said they were useful. No clear conclusion was reached. [1]: https://groups.google.com/d/topic/mo

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread William McCloskey
I was the person in the previous thread who found them useful, and I still do. Some of the extraWarnings stuff is of questionable value, but the undefined property stuff is really useful. I don't really know if other people use this stuff. extraWarnings are enabled by default in debug builds, and

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread Chris Peterson
On 12/19/14 12:45 PM, William McCloskey wrote: I don't really know if other people use this stuff. extraWarnings are enabled by default in debug builds, and I think a lot of people don't realize the difference between these "extra warnings" and normal JS errors. Writing front-end JS code for Fir

Re: PSA: Flaky timeouts in mochitest-plain now fail newly added tests

2014-12-19 Thread Nils Ohlmeier
> On Dec 19, 2014, at 6:56 AM, Ehsan Akhgari wrote: > > Let me try to rephrase the problem in different terms, to hopefully make it > clearer why using timers like this is a bad idea. > > setTimeout(foo, 1000) may seem to suggest run foo after 1 second, but that is > *not* what that function

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread Nick Fitzgerald
I generally don't find them useful, but instead annoying, and that they provide a lot of noise to filter out to find actual relevant errors. This is including the undefined property errors. It is a common JS style to pass around configuration/option objects that will be missing many properties that

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread Gijs Kruitbosch
On 19/12/2014 20:45, William McCloskey wrote: I was the person in the previous thread who found them useful, and I still do. Some of the extraWarnings stuff is of questionable value, but the undefined property stuff is really useful. Can you give an example of a useful undefined property warnin

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread Jim Porter
On 12/19/2014 02:19 PM, Jason Orendorff wrote: > So if you go to about:config and set the javascript.options.strict pref, > you'll get warnings about accessing undefined properties. > > Please speak up now, if you're still using it! I find these warnings quite useful (granted, I'm the sort of per

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread David Rajchenbach-Teller
I am going to suggest, once again, that warnings generally noise and should be replaced by actionable errors, at least when the code is executed in a test suite. See https://groups.google.com/forum/#!topic/mozilla.dev.platform/gqSIOc5b-BI Cheers, David On 19/12/14 21:19, Jason Orendorff wrote:

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread William McCloskey
On Fri, Dec 19, 2014 at 2:34 PM, Gijs Kruitbosch wrote: > > Can you give an example of a useful undefined property warning? Because my > experience is the same as fitzgen's in that they are basically never useful > to me. I can't cite any bugzilla bugs, no. They're more useful while still develo

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread Jim Blandy
The bug is surprising, in that it claims that the bytecode that consumes the value determines whether a warning is issued (SETLOCAL;CALL), rather than the bytecode doing the fetch. Is that the intended behavior? I can't see how that makes much sense. On Dec 19, 2014 2:55 PM, "David Rajchenbach-Tel

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread Jason Orendorff
On Fri, Dec 19, 2014 at 5:13 PM, Jim Blandy wrote: > The bug is surprising, in that it claims that the bytecode that consumes > the value determines whether a warning is issued (SETLOCAL;CALL), rather > than the bytecode doing the fetch. > > Is that the intended behavior? I can't see how that mak

Re: PSA: Flaky timeouts in mochitest-plain now fail newly added tests

2014-12-19 Thread Ehsan Akhgari
On 2014-12-19 4:40 PM, Nils Ohlmeier wrote: On Dec 19, 2014, at 6:56 AM, Ehsan Akhgari wrote: Let me try to rephrase the problem in different terms, to hopefully make it clearer why using timers like this is a bad idea. setTimeout(foo, 1000) may seem to suggest run foo after 1 second, but t

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread ISHIKAWA,Chiaki
(2014/12/20 5:19), Jason Orendorff wrote: So if you go to about:config and set the javascript.options.strict pref, you'll get warnings about accessing undefined properties. js> Math.TAU undefined /!\ ReferenceError: reference to undefined property Math.TAU (It says "ReferenceErro

Re: Is anyone still using JS strict warnings?

2014-12-19 Thread Jim Blandy
On Fri, Dec 19, 2014 at 2:22 PM, Nick Fitzgerald wrote: > I generally don't find them useful, but instead annoying, and that they > provide a lot of noise to filter out to find actual relevant errors. This > is including the undefined property errors. It is a common JS style to pass > around conf