From 4189c0f9aaa7d4e0883bf30dd076394e44429f79 Mon Sep 17 00:00:00 2001 From: Yueh-Hsuan Chiang Date: Tue, 17 Nov 2015 13:50:54 -0800 Subject: [PATCH] Fix Java Makefile Summary: In case rocksdb java package is built using make rocksdbjavastaticrelease, then only those rocksdb binary built under the virtual environments is release build. This patch fix this issue. Test Plan: PORTABLE=1 V=2 make rocksdbjavastaticrelease -j32 and make sure -O2 and -NDEBUG is included when compiling all source files. Reviewers: sdong, anthony, IslamAbdelRahman, rven, kradhakrishnan, igor Reviewed By: igor Subscribers: dhruba, leveldb Differential Revision: https://reviews.facebook.net/D50895 --- Makefile | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/Makefile b/Makefile index 0fd1dae7f..6eef57260 100644 --- a/Makefile +++ b/Makefile @@ -73,6 +73,14 @@ ifeq ($(MAKECMDGOALS),rocksdbjavastatic) DEBUG_LEVEL=0 endif +ifeq ($(MAKECMDGOALS),rocksdbjavastaticrelease) + DEBUG_LEVEL=0 +endif + +ifeq ($(MAKECMDGOALS),rocksdbjavastaticpublish) + DEBUG_LEVEL=0 +endif + # compile with -O2 if debug level is not 2 ifneq ($(DEBUG_LEVEL), 2) OPT += -O2 -fno-omit-frame-pointer