This component tracks
* Modifying or removing existing builds, tests and other jobs
* Adding support for new types of jobs, builds or tests (e.g. opt, pgo, debug, ASAN or code coverage builds; b2g device builds, new test suites; special builds like spidermonkey or valgrind)
* Scheduler changes: what jobs get run and when

Comment on attachment 8493146[details][diff][review]
runner-puppet.diff
Review of attachment 8493146[details][diff][review]:
-----------------------------------------------------------------
Looks good to me! Since all buildbot masters run on linux, it shouldn't be a problem that --no-run-if-empty is not mac-compatible. However, if we start using runner on non-linux machines, we may need to tweak it - but I don't think there are any plans to use it on anything other than buildbot masters at the moment, which are all linux, afaik.
If I'm wrong about this, let me know!
Thanks Rail!
Pete

Comment on attachment 8494160[details][diff][review]
update_shared_repos.diff
Review of attachment 8494160[details][diff][review]:
-----------------------------------------------------------------
::: modules/runner/templates/tasks/update_shared_repos.erb
@@ +28,5 @@
> fi
> + if [ -z $hgrc_found ]; then
> + echo "No valid mercurial repos found. Exiting early"
> + exit 0
> + fi
I think this might not work, because the if condition is inside the loop, and the unsetting of hgrc_found is also inside the loop, and both would need to be outside.
However, I think there may be another way, I'll attach a counter patch with my idea.

So out of curiosity, I tested on Mac, and it seems sed actually adds a line ending on mac, even though the input has no line ending. On linux this works fine, but also this does not work on Mac as desired, due to the difference in the sed implementation! For example:
Mac
===
pmoore@Elisandra:~ $ echo -n 'hi' | sed 's/hi/hello/'
hello
pmoore@Elisandra:~ $
Linux
=====
[cltbld@b-linux64-hp-0012.try.releng.scl3.mozilla.com ~]$ echo -n 'hi' | sed 's/hi/hello/'
hello[cltbld@b-linux64-hp-0012.try.releng.scl3.mozilla.com ~]$
So I'll tweak it again just so slightly to get it working on both.