1079579 - Move the spidermonkey automation build scripts into the tree

Status: RESOLVED FIXED

(Release Engineering :: Applications: MozharnessCore, defect)

Description

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

It's much easier to figure out what broke the tests if the build/run script is in the tree. And this seems to be the general direction of most things these days.

Comment 1

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

Attachment: Move the automation build scripts into the tree

Comment 2

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

Attachment: Call an in-tree script for spidermonkey jobs

Comment 3

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

Attachment: Call an in-tree script for spidermonkey jobs

Do the buildbot stuff, then run the script from the source tree.

Comment 4

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

Attachment: Move the automation build scripts into the tree

Move all the scripts into the tree.

Comment 5

[Terrence Cole [:terrence]]

Comment on attachment 8504827 [details] [diff] [review]
Move the automation build scripts into the tree

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

::: js/src/devtools/automation/autospider.sh
@@ +124,5 @@
> +fi
> +
> +$COMMAND_PREFIX $MAKE check || exit 1
> +$COMMAND_PREFIX $MAKE check-jit-test || exit 1
> +$COMMAND_PREFIX $MAKE check-jstests || exit 1

I guess this is the line that turns out to be a nop currently?

Comment 6

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

Comment on attachment 8504826 [details] [diff] [review]
Call an in-tree script for spidermonkey jobs

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

Then again, r- from me. I just realized that if I land this as-is, I will break all of the spidermonkey builds for the older trees.

Comment 7

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

Attachment: Call an in-tree script for spidermonkey jobs

Smaller patch. Call the in-tree version if it exists.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

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