Handle LOCAL_CPPFLAGS/CFLAGS/CXXFLAGS as claimed by the documentation.

The previous code did only use LOCAL_CFLAGS for both C and C++ sources.
This commit is contained in:
David 'Digit' Turner
2009-07-24 17:42:21 +02:00
parent f74c1d2fb7
commit 1715a1f877
2 changed files with 10 additions and 2 deletions

View File

@@ -16,7 +16,7 @@
#
# We use the GNU Make Standard Library
include build/gmsl/gmsl
include $(BUILD_SYSTEM)/../gmsl/gmsl
# This is the Android NDK version number as a list of three items:
# major, minor, revision
@@ -371,6 +371,7 @@ $$(_OBJ): PRIVATE_CFLAGS := $$($$(my)CFLAGS) \
$$($$(my)$(LOCAL_ARM_MODE)_$(LOCAL_BUILD_MODE)_CFLAGS) \
$$(LOCAL_C_INCLUDES:%=-I%) \
-I$$(LOCAL_PATH) \
$$(LOCAL_CPPFLAGS) \
$$(LOCAL_CFLAGS) \
$$(NDK_APP_CPPFLAGS) \
$$(NDK_APP_CFLAGS)
@@ -430,7 +431,8 @@ $$(_OBJ): PRIVATE_CXXFLAGS := $$($$(my)CXXFLAGS) \
$$($$(my)$(LOCAL_ARM_MODE)_$(LOCAL_BUILD_MODE)_CFLAGS) \
$$(LOCAL_C_INCLUDES:%=-I%) \
-I$$(LOCAL_PATH) \
$$(LOCAL_CFLAGS) \
$$(LOCAL_CPPFLAGS) \
$$(LOCAL_CXXFLAGS) \
$$(NDK_APP_CPPFLAGS) \
$$(NDK_APP_CXXFLAGS) \

View File

@@ -33,5 +33,11 @@ current version
- Fix compilation of assembler files (e.g. foo.S)
- Make LOCAL_CFLAGS / LOCAL_CXXFLAGS / LOCAL_CPPFLAGS work as advertized
by the documentation (previously, only LOCAL_CFLAGS did work for both C
*and* C++ sources, contrary to what the doc claimed).
Note that APP_CPPFLAGS / APP_CFLAGS / APP_CXXFLAGS continue to work.
-------------------------------------------------------------------------------
android-1.5_r1 released.