Hoernchen has submitted this change. (
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/41908?usp=email )
Change subject: contrib/jenkins.sh: clean up CFLAGS
......................................................................
contrib/jenkins.sh: clean up CFLAGS
Change-Id: I4e78d6eb3c6c7b40312072fcf70f4b6d80042959
---
M contrib/jenkins.sh
1 file changed, 7 insertions(+), 2 deletions(-)
Approvals:
Hoernchen: Looks good to me, approved
Jenkins Builder: Verified
osmith: Looks good to me, but someone else must approve
Objections:
laforge: I would prefer this is not merged as is
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index b948198..9462cc3 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -39,6 +39,11 @@
osmo-deps.sh libosmocore master
cd libosmocore
+CFLAGS="-Os -g3 -ffunction-sections -fdata-sections -nostartfiles
-nodefaultlibs"
+CFLAGS="$CFLAGS -mcpu=cortex-m4 -mfloat-abi=softfp -mfpu=fpv4-sp-d16 -mthumb
-mlong-calls"
+CFLAGS="$CFLAGS -Werror -Wno-error=deprecated
-Wno-error=deprecated-declarations -Wno-error=cpp -Wno-error=format"
+CFLAGS="$CFLAGS -I$TOPDIR/sysmoOCTSIM" # this is needed for talloc.h
(--disable-pseudotalloc)
+
mkdir -p "$inst/stow"
autoreconf --install --force
./configure \
@@ -49,8 +54,8 @@
--enable-embedded \
--disable-pseudotalloc \
--disable-doxygen \
- CFLAGS="-Os -ffunction-sections -fdata-sections -nostartfiles
-nodefaultlibs -Werror -Wno-error=deprecated -Wno-error=deprecated-declarations
-Wno-error=cpp -mthumb -Os -mlong-calls -g3 -mcpu=cortex-m4 -mfloat-abi=softfp
-mfpu=fpv4-sp-d16 -I$TOPDIR/sysmoOCTSIM -Wno-error=format" \
- CPPFLAGS="-D__thread=''"
+ CFLAGS="$CFLAGS" \
+ CPPFLAGS="-D__thread=''"
make $PARALLEL_MAKE install
make clean
STOW_DIR="$inst/stow" stow --restow libosmocore
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/41908?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I4e78d6eb3c6c7b40312072fcf70f4b6d80042959
Gerrit-Change-Number: 41908
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Hoernchen <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
