From 876d3bd85aabe7fce71a025a89c6b3f6ddbf2053 Mon Sep 17 00:00:00 2001
From: Sam Roberts <sam@strongloop.com>
Date: Tue, 17 Dec 2013 16:04:20 -0800
Subject: [PATCH] cluster: do not synchronously emit 'setup' event

This is a problem present in both v0.10, and v0.11, where the 'setup'
event is synchronously emitted by `cluster.setupMaster()`, a mostly
harmless anti-pattern.
---
 lib/cluster.js                           |  4 +++-
 test/simple/test-cluster-setup-master.js | 12 ++++++------
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/lib/cluster.js b/lib/cluster.js
index f75fda5..4f8fb8c 100644
--- a/lib/cluster.js
+++ b/lib/cluster.js
@@ -269,7 +269,9 @@ function masterInit() {
         process._debugProcess(cluster.workers[key].process.pid);
     });
 
-    cluster.emit('setup');
+    process.nextTick(function() {
+      cluster.emit('setup');
+    });
   };
 
   var ids = 0;
diff --git a/test/simple/test-cluster-setup-master.js b/test/simple/test-cluster-setup-master.js
index b842228..d9f3879 100644
--- a/test/simple/test-cluster-setup-master.js
+++ b/test/simple/test-cluster-setup-master.js
@@ -39,6 +39,12 @@ if (cluster.isWorker) {
 
   var totalWorkers = 2;
 
+  // Setup master
+  cluster.setupMaster({
+    args: ['custom argument'],
+    silent: true
+  });
+
   cluster.once('setup', function() {
     checks.setupEvent = true;
 
@@ -51,12 +57,6 @@ if (cluster.isWorker) {
     }
   });
 
-  // Setup master
-  cluster.setupMaster({
-    args: ['custom argument'],
-    silent: true
-  });
-
   var correctIn = 0;
 
   cluster.on('online', function lisenter(worker) {
-- 
2.7.4