Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: Makefile

Issue 29823575: Noissue - fix the switching to clang++/libc++ (Closed) Base URL: https://github.com/adblockplus/libadblockplus@ca063256d589c9e950a5b14a16bc266e3b5072e8
Patch Set: Created July 5, 2018, 10:10 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Makefile
diff --git a/Makefile b/Makefile
index 399c64d599d7230c40f2479585cd164228d02cf1..af8857638e609e181960de7ba7e9b03e7972c0aa 100644
--- a/Makefile
+++ b/Makefile
@@ -100,8 +100,11 @@ all: ensure_dependencies
NDK_APP_DST_DIR=android-$(ABP_TARGET_ARCH).release
else
SUB_ACTION ?= all
+ifdef CXX
+CXX_PARAM:=CXX=${CXX}
+endif
all: ensure_dependencies
GYP_DEFINES="${GYP_PARAMETERS}" third_party/gyp/gyp --depth=. -f make -I libadblockplus.gypi --generator-output=${BUILD_DIR} libadblockplus.gyp
- $(MAKE) -C ${BUILD_DIR} ${SUB_ACTION}
+ $(MAKE) -C ${BUILD_DIR} ${SUB_ACTION} ${CXX_PARAM}
sergei 2018/07/05 10:29:34 It's not enough to define a local variable CXX, we
endif
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld