Skip to content

Commit 8c357b9

Browse files
committed
Don't add include dir in CFLAGS for all libraries.
Causes already installed headers to be used when re-building, which might be out of date.
1 parent 3238421 commit 8c357b9

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

env/toolchain.sh

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,6 @@ export RANLIB="${TOOLCHAIN}"/bin/${ANDROID_TARGET_BINUTILS}-ranlib
1616
export STRIP="${TOOLCHAIN}"/bin/${ANDROID_TARGET_BINUTILS}-strip
1717
export PKG_CONFIG_PATH="${INSTALL_PREFIX}/lib/pkgconfig"
1818

19-
# some libraries require this
20-
export CFLAGS="-I${INSTALL_PREFIX}/include"
21-
export CPPFLAGS="-I${INSTALL_PREFIX}/include"
22-
2319
# -fuse-ld=gold required to work around BFD ld linker bugs on arm64 with gnustep-2.0 libobjc runtime
2420
# -rpath-link required for linker to find libcxxrt dependency of libobjc
2521
# --build-id=sha1 required for Android Studio to locate debug information

phases/40-gnustep-corebase.sh

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,10 @@ echo "### Source GNUstep.sh"
1616
echo -e "\n### Running configure"
1717
./configure \
1818
--host=${ANDROID_TARGET} \
19-
--prefix="${INSTALL_PREFIX}"
19+
--prefix="${INSTALL_PREFIX}" \
20+
`# for some reason we need to manually specify the include dir` \
21+
CFLAGS="${CFLAGS} -I${INSTALL_PREFIX}/include" \
22+
CPPFLAGS="${CPPFLAGS} -I${INSTALL_PREFIX}/include" \
2023

2124
echo -e "\n### Building"
2225
make -j${MAKE_JOBS} ${GNUSTEP_MAKE_OPTIONS}

0 commit comments

Comments
 (0)