1672428 - Merge the paths for running hazard analysis locally and in automation

Status: RESOLVED FIXED

(Core :: JavaScript: GC, task, P3)

Description

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

I implemented mach commands for running the hazard analysis, but the actual analysis is still controlled through separate shell scripts. That has resulted in them diverging, and creates duplicate code. I would like to switch all but a basic wrapper to mach.

Comment 1

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

Attachment: Bug 1672428 - Switch developer hazard builds gcc 8 to gcc 9. Otherwise they will see many hazards.

Comment 2

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

Attachment: Bug 1672428 - Use `mach hazards build-shell` for actual hazard job JS shell

Comment 3

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

Attachment: Bug 1672428 - Allow simultaneous browser and shell instances of the hazard analysis

Comment 4

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

Attachment: Bug 1672428 - Add `mach build` command-line arguments to `mach hazards gather`

I wanted this because I was running out of memory during a compile, and I wanted to pass -j4 to limit the number of parallel jobs.

Comment 5

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

Attachment: Bug 1672428 - Remove a layer of mapping from "project" to "application"

Comment 6

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

Attachment: Bug 1672428 - Add in partial support for finding tools in automation paths in addition to `mach hazard bootstrap` paths for developer builds

Comment 7

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

Attachment: Bug 1672428 - Reformatted some code with black

Comment 8

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

Attachment: Bug 1672428 - Switch the actual analysis to mach

Comment 9

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

Attachment: Bug 1672428 - Run a self-test before doing the actual analysis.

Comment 10

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

Attachment: Bug 1672428 - Dodge -Wtype-limits warning

Comment 11

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

Attachment: Bug 1672428 - Remove hazard-analysis.sh

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f4bfb3b97dd2
Use `mach hazards build-shell` for actual hazard job JS shell r=jonco
https://hg.mozilla.org/integration/autoland/rev/e2a7870358ac
Allow simultaneous browser and shell instances of the hazard analysis r=jonco
https://hg.mozilla.org/integration/autoland/rev/fa2d59a19bcf
Add `mach build` command-line arguments to `mach hazards gather` r=jonco
https://hg.mozilla.org/integration/autoland/rev/0c5af7decad2
Remove a layer of mapping from "project" to "application" r=jonco
https://hg.mozilla.org/integration/autoland/rev/b0132ea16a42
Add in partial support for finding tools in automation paths in addition to `mach hazard bootstrap` paths for developer builds r=jonco
https://hg.mozilla.org/integration/autoland/rev/036d1664eb62
Reformatted some code with black r=jonco
https://hg.mozilla.org/integration/autoland/rev/e41138cdc5ac
Switch the actual analysis to mach r=jonco
https://hg.mozilla.org/integration/autoland/rev/38cf4fe6e57d
Run a self-test before doing the actual analysis. r=jonco
https://hg.mozilla.org/integration/autoland/rev/5cb551f8bb22
Dodge -Wtype-limits warning r=jonco
https://hg.mozilla.org/integration/autoland/rev/f9332f40f355
Remove hazard-analysis.sh r=jonco

Comment 13

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/f4bfb3b97dd2
https://hg.mozilla.org/mozilla-central/rev/e2a7870358ac
https://hg.mozilla.org/mozilla-central/rev/fa2d59a19bcf
https://hg.mozilla.org/mozilla-central/rev/0c5af7decad2
https://hg.mozilla.org/mozilla-central/rev/b0132ea16a42
https://hg.mozilla.org/mozilla-central/rev/036d1664eb62
https://hg.mozilla.org/mozilla-central/rev/e41138cdc5ac
https://hg.mozilla.org/mozilla-central/rev/38cf4fe6e57d
https://hg.mozilla.org/mozilla-central/rev/5cb551f8bb22
https://hg.mozilla.org/mozilla-central/rev/f9332f40f355