Merge change 8501 into donut

* changes:
  Handle LOCAL_CPPFLAGS/CFLAGS/CXXFLAGS as claimed by the documentation.
This commit is contained in:
Android (Google) Code Review
2009-07-25 09:10:27 -07:00
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.