Remove references to sconsbuild.

Remove references to sconsbuild.

scons hasn't been used in a long time.

Also remove support for using make on Mac, it's no longer supported. All bots
that were using make on mac are using ninja these days.

BUG=305702
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=227778

[Lei Zhang, 2013-10-09 18:56:37]

lgtm

Last time I tried to do this, I failed miserably:
https://codereview.chromium.org/7306001/

Tread carefully.

https://codereview.chromium.org/26672003/diff/4001/build/scripts/slave/compil...
File build/scripts/slave/compile.py (left):

https://codereview.chromium.org/26672003/diff/4001/build/scripts/slave/compil...
build/scripts/slave/compile.py:581: # The default process limit on 10.6 is 266
(`sysctl kern.maxprocperuid`),
This seems useful to know. Keep it?

[Nico, 2013-10-09 19:21:35]

Thanks!

I checked
http://build.chromium.org/p/chromium.perf/builders/Linux%20Perf%20%282%29/bui...
, the perf bots use out/Release these days.

Yang: I'm going to land this, but please check that the v8 bits make sense (or
find someone who can check that).

https://codereview.chromium.org/26672003/diff/4001/build/scripts/slave/compil...
File build/scripts/slave/compile.py (left):

https://codereview.chromium.org/26672003/diff/4001/build/scripts/slave/compil...
build/scripts/slave/compile.py:581: # The default process limit on 10.6 is 266
(`sysctl kern.maxprocperuid`),
On 2013/10/09 18:56:37, Lei Zhang wrote:
> This seems useful to know. Keep it?

This is only true on mac, and this function doesn't run on mac

[Lei Zhang, 2013-10-09 19:33:40]

Maybe split of the master change to a separate CL and give the V8 folks time to
check it? Maybe there are internal bots that use V8Factory? Plus you won't be
able to observe the effects until you restart the master.

https://codereview.chromium.org/26672003/diff/4001/build/scripts/slave/compil...
File build/scripts/slave/compile.py (left):

https://codereview.chromium.org/26672003/diff/4001/build/scripts/slave/compil...
build/scripts/slave/compile.py:581: # The default process limit on 10.6 is 266
(`sysctl kern.maxprocperuid`),
On 2013/10/09 19:21:35, Nico wrote:
> On 2013/10/09 18:56:37, Lei Zhang wrote:
> > This seems useful to know. Keep it?
> 
> This is only true on mac, and this function doesn't run on mac

Ah, right.

[Nico, 2013-10-09 19:36:53]

On 2013/10/09 19:33:40, Lei Zhang wrote:
> Maybe split of the master change to a separate CL and give the V8 folks time
to
> check it? Maybe there are internal bots that use V8Factory? Plus you won't be
> able to observe the effects until you restart the master.

Good idea, moved that to https://codereview.chromium.org/26399003/

[commit-bot: I haz the power, 2013-10-09 19:37:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thakis@chromium.org/26672003/11001

[commit-bot: I haz the power, 2013-10-09 19:40:02]

Presubmit check for 26672003-11001 failed and returned exit status 1.

Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh

** Presubmit ERRORS **
Pylint (401 files) failed
************* Module slave.compile
W0611: 20,0: Unused import shutil


Missing LGTM from an OWNER for these files:
    depot_tools/presubmit_support.py

Missing LGTM from an OWNER for these files:
    depot_tools/presubmit_support.py

Presubmit checks took 159.9s to calculate.

[Nico, 2013-10-09 19:44:25]

maruel: depot_tools/OWNERS stamp please

[M-A Ruel, 2013-10-09 19:46:48]

lgtm

[commit-bot: I haz the power, 2013-10-09 19:47:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thakis@chromium.org/26672003/19001

[commit-bot: I haz the power, 2013-10-09 19:50:28]

Change committed as 227778