1267876 - Make hazard analysis runnable on developer desktops

Status: RESOLVED FIXED

(Core :: JavaScript: GC, defect)

Description

[Steve Fink [:sfink] [:s:]]

It would be useful to be able to run the rooting hazard analysis on developers' desktops.

MozReview-Commit-ID: K6kMX8WD15g

Comment 1

[Steve Fink [:sfink] [:s:]]

Attachment: Make usable on developer desktops

Note that this only mentions Docker in passing, because I didn't want to bother with documenting what image to use and things. But I'm running this successfully on my Fedora desktop without Docker. Works great. Curious to see if it'll work for you.

Comment 2

[Jason Orendorff [:jorendorff]]

Comment on attachment 8745760 [details] [diff] [review]
Make usable on developer desktops

Review of attachment 8745760 [details] [diff] [review]:
-----------------------------------------------------------------

Thank you so much for dropping everything and doing this. Ludicrously nice. Apart from me being dumb and setting SRCDIR to a relative dir, it ran great.

::: js/src/devtools/rootAnalysis/README.txt
@@ +1,1 @@
>  # Spidermonkey JSAPI rooting analysis

This file is really a markdown file; rename to README.md?

@@ +13,2 @@
>  
>  2.  Have the Gecko build prerequisites installed.

This also requires `tc-vcs`, which isn't a Gecko build prerequisite.

I was able to install it locally with

    npm install taskcluster-vcs
    export PATH="$PATH:$(pwd)/node_modules/.bin"

@@ +18,4 @@
>  
> +    mkdir work
> +    cd work
> +    GECKO_DIR=$SRCDIR $SRCDIR/testing/taskcluster/scripts/builder/build-haz-linux.sh $(pwd)

markdown nit: indent the line that says "run these commands:" to 4 spaces (I don't think it matters if the previous line is indented or not) and indent these three commands an additional 4 spaces.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/69b3bf72b9d1

Comment 4

[Pulsebot]

Backout:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fdd10b413bf2

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d55e3bc94309

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d55e3bc94309