Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(425)

Unified Diff: ports/libtommath/build.sh

Issue 730193002: Fix more compile failures caused by recent ARM toolchain upgrade. (Closed) Base URL: https://chromium.googlesource.com/external/naclports.git@master
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ports/libgit2/build.sh ('k') | ports/libyuv/build.sh » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ports/libtommath/build.sh
diff --git a/ports/libtommath/build.sh b/ports/libtommath/build.sh
index 487cab2b6eeb52b4e8c0e49f217e10f9502a16b1..e1acc6650d027d82be5d6c707fcadb46d2d21f90 100644
--- a/ports/libtommath/build.sh
+++ b/ports/libtommath/build.sh
@@ -6,18 +6,26 @@ BUILD_DIR=${SRC_DIR}
MAKE_TARGETS=mtest
EXECUTABLES=mtest/mtest${NACL_EXEEXT}
+# Workaround for arm-gcc bug:
+# https://code.google.com/p/nativeclient/issues/detail?id=3205
+# TODO(sbc): remove this once the issue is fixed
+if [ "${NACL_ARCH}" = "arm" ]; then
+ NACLPORTS_CPPFLAGS+=" -mfpu=vfp"
+fi
+
BuildStep() {
SetupCrossEnvironment
DefaultBuildStep
}
-#TestStep() {
+TestStep() {
# To run tests, pipe mtest.nexe output into test.nexe input
# mtest/mtest.exe | test.nexe
# However, this test is setup to run forever, so we don't run
# it as part of the build.
#RunSelLdrCommand mtest/mtest.nexe | RunSelLdrCommand test.nexe
-#}
+ return
+}
InstallStep() {
# copy libs and headers manually
« no previous file with comments | « ports/libgit2/build.sh ('k') | ports/libyuv/build.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698