From 8db2cf0e93921dff5d8a712c82eaebe4ebbe0559 Mon Sep 17 00:00:00 2001
From: Jules Villard <jul@fb.com>
Date: Thu, 7 Jul 2016 04:02:11 -0700
Subject: [PATCH] unset MAKEFLAGS before calling buck

Summary:
`make` calls `buck`, who calls `make` again. The last `make` then gets confused
because it wasn't called with `make -n` as it should be for a call of
make-within-make. Unsetting this variable makes `make` forget that it's running
inside `make`.

The main benefit is that it removes annoying spamming of stderr by make/buck
about disabling jobserver.

The submake should also be more parallel now, although if other things are
running in parallel they will ignore any limit set with `-j`.

Reviewed By: jberdine

Differential Revision: D3522156

fbshipit-source-id: 45db21a
---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 3b589984a..b6cf0a895 100644
--- a/Makefile
+++ b/Makefile
@@ -97,7 +97,7 @@ ocaml_unit_test: test_this_build
 
 buck_test: infer
 	NO_BUCKD=1 buck clean
-	NO_BUCKD=1 buck test -j $(NCPU) -L $(NCPU) $(TARGETS_TO_TEST)
+	MAKEFLAGS= NO_BUCKD=1 buck test -j $(NCPU) -L $(NCPU) $(TARGETS_TO_TEST)
 	NO_BUCKD=1 ./infer/tests/build_systems/build_integration_tests.py
 
 buck_test_xml: infer