From da4dbf85f95051345f147cfe69d909d85e81a429 Mon Sep 17 00:00:00 2001
From: Takuto Ikuta <tikuta@chromium.org>
Date: Mon, 4 Mar 2019 03:21:58 +0000
Subject: [PATCH] autoninja: limit j value for mac

Large parallelism causes too many open files error.
Let me limit to 500 for Mac now.

Bug: 936864
Change-Id: I2fccc8cf14483c6f34d84c84d82c44df6e4f3177
Reviewed-on: https://chromium-review.googlesource.com/c/1496675
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
---
 autoninja.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/autoninja.py b/autoninja.py
index a09e8399d..8cbb67f10 100755
--- a/autoninja.py
+++ b/autoninja.py
@@ -85,6 +85,10 @@ if not j_specified and not t_specified:
     if sys.platform.startswith('win'):
       # On windows, j value higher than 1000 does not improve build performance.
       j_value = min(j_value, 1000)
+    elif sys.platform == 'darwin':
+      # On Mac, j value higher than 500 causes 'Too many open files' error
+      # (crbug.com/936864).
+      j_value = min(j_value, 500)
 
     args.append('%d' % j_value)
   else: