Makefile - Issue 29823575: Noissue - fix the switching to clang++/libc++

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Side by Side 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.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
1 HOST_ARCH :=$(shell python third_party/detect_v8_host_arch.py)	1 HOST_ARCH :=$(shell python third_party/detect_v8_host_arch.py)

2	2

3 ifndef HOST_OS	3 ifndef HOST_OS

4 raw_OS = $(shell uname -s)	4 raw_OS = $(shell uname -s)

5 ifeq (${raw_OS},Linux)	5 ifeq (${raw_OS},Linux)

6 HOST_OS=linux	6 HOST_OS=linux

7 TRAVIS_OS_NAME=linux	7 TRAVIS_OS_NAME=linux

8 CXX=clang++	8 CXX=clang++

9 else ifeq (${raw_OS},Darwin)	9 else ifeq (${raw_OS},Darwin)

10 HOST_OS=mac	10 HOST_OS=mac

(...skipping 82 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
93 APP_PLATFORM=${ANDROID_PLATFORM_LEVEL} \	93 APP_PLATFORM=${ANDROID_PLATFORM_LEVEL} \

94 APP_PIE=true \	94 APP_PIE=true \

95 APP_STL=c++_shared \	95 APP_STL=c++_shared \

96 APP_BUILD_SCRIPT=Makefile \	96 APP_BUILD_SCRIPT=Makefile \

97 NDK_PROJECT_PATH=. \	97 NDK_PROJECT_PATH=. \

98 NDK_OUT=. \	98 NDK_OUT=. \

99 ${ANDROID_FIXES} \	99 ${ANDROID_FIXES} \

100 NDK_APP_DST_DIR=android-$(ABP_TARGET_ARCH).release	100 NDK_APP_DST_DIR=android-$(ABP_TARGET_ARCH).release

101 else	101 else

102 SUB_ACTION ?= all	102 SUB_ACTION ?= all

	103 ifdef CXX

	104 CXX_PARAM:=CXX=${CXX}

	105 endif

103 all: ensure_dependencies	106 all: ensure_dependencies

104 GYP_DEFINES="${GYP_PARAMETERS}" third_party/gyp/gyp --depth=. -f make -I libadblockplus.gypi --generator-output=${BUILD_DIR} libadblockplus.gyp	107 GYP_DEFINES="${GYP_PARAMETERS}" third_party/gyp/gyp --depth=. -f make -I libadblockplus.gypi --generator-output=${BUILD_DIR} libadblockplus.gyp

105 » $(MAKE) -C ${BUILD_DIR} ${SUB_ACTION}	108 » $(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 It's not enough to define a local variable CXX, we should also pass it to $(MAKE), however if CXX is not defined, e.g. on another platform we should rather pass nothing. Therefore I have added CXX_PARAM which has CXX=${CXX} value when CXX is defined and is empty when CXX is not defined.
106	109

107 endif	110 endif

OLD	NEW

« no previous file with comments | « no previous file | no next file » | no next file with comments »