Beginning on October 25th, 2016, Persona will no longer be an option for authentication on BMO. For more details see Persona Deprecated.
Last Comment Bug 749768 - Disable --valgrind flag for when running "make check" with AddressSanitizer
: Disable --valgrind flag for when running "make check" with Addres...
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: All Linux
: -- normal (vote)
: mozilla15
Assigned To: Christian Holler (:decoder)
: Jason Orendorff [:jorendorff]
Depends on:
Blocks: 664901
  Show dependency treegraph
Reported: 2012-04-27 13:17 PDT by Christian Holler (:decoder)
Modified: 2012-05-02 21:05 PDT (History)
5 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch (2.30 KB, patch)
2012-04-30 05:26 PDT, Christian Holler (:decoder)
khuey: review+
Details | Diff | Splinter Review
Updated patch (1.29 KB, patch)
2012-04-30 05:54 PDT, Christian Holler (:decoder)
choller: review+
gary: checkin+
Details | Diff | Splinter Review

Description Christian Holler (:decoder) 2012-04-27 13:17:40 PDT
In js/src/ the script is called with "--valgrind" which causes some tests to timeout because running an ASan binary under valgrind doesn't work.

I'm writing a patch that only passes --valgrind in the Makefile when MOZ_ASAN is not set.
Comment 1 Christian Holler (:decoder) 2012-04-30 05:26:36 PDT
Created attachment 619528 [details] [diff] [review]
Comment 2 Kyle Huey [:khuey] (Exited; not receiving bugmail, email if necessary) 2012-04-30 05:28:33 PDT
Comment on attachment 619528 [details] [diff] [review]

Review of attachment 619528 [details] [diff] [review]:

::: js/src/
@@ +605,5 @@
>  ifneq ($(OS_ARCH),WINNT) # FIXME: this should be made work on Windows too.
>  #check:: check-malloc-function-usage FIXME: disable on JM until closer to merge time.
>  endif
> +ifneq ($(MOZ_ASAN),1)

ifndef MOZ_ASAN
Comment 3 Christian Holler (:decoder) 2012-04-30 05:54:53 PDT
Created attachment 619535 [details] [diff] [review]
Updated patch

Updated patch, addresses review comment. Keeping r+ from previous patch version.
Comment 4 Christian Holler (:decoder) 2012-04-30 05:55:59 PDT
Marking also as a build blocker because this causes "make check" to run for the maximum allowed time per test using valgrind, rendering the builds very slow.
Comment 5 Gary Kwong [:gkw] [:nth10sd] 2012-04-30 07:24:21 PDT
Comment on attachment 619535 [details] [diff] [review]
Updated patch
Comment 6 :Ehsan Akhgari (Away Oct 25 - Nov 9) 2012-05-02 21:05:10 PDT

Note You need to log in before you can comment on or make changes to this bug.