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

Unified Diff: buildbot/buildbot_pnacl.sh

Issue 9824001: Change built_elsewhere flag to work with pexe mode (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: add AS, line len Created 8 years, 9 months 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
Index: buildbot/buildbot_pnacl.sh
diff --git a/buildbot/buildbot_pnacl.sh b/buildbot/buildbot_pnacl.sh
index 26b746cc86fa9c2d4d1d2ef54f21c709758ebf28..a41d9d11fe2d7e40ee7514d32f5a81d19fbf251c 100755
--- a/buildbot/buildbot_pnacl.sh
+++ b/buildbot/buildbot_pnacl.sh
@@ -83,16 +83,19 @@ archive-for-hw-bots() {
echo "@@@BUILD_STEP tar_generated_binaries@@@"
# clean out a bunch of files that are not needed
find scons-out/ \
- \( -name '*.[so]' -o -name '*.bc' -o -name '*.pexe' -o -name '*.ll' \) \
+ \( -name '*.o' -o -name '*.bc' \) \
-print0 | xargs -0 rm -f
- tar cvfz arm-tcb.tgz scons-out/
+ # delete nexes from pexe mode directories to force translation
+ # TODO(dschuff) enable this once we can translate on the hw bots
+ #find scons-out/*pexe*/ -name '*.nexe' -print0 | xargs -0 rm -f
+ tar cvfz arm-scons.tgz scons-out/*arm*
echo "@@@BUILD_STEP archive_binaries@@@"
if [[ ${try} == "try" ]] ; then
- ${UP_DOWN_LOAD} UploadArmBinariesForHWBotsTry ${name} arm-tcb.tgz
+ ${UP_DOWN_LOAD} UploadArmBinariesForHWBotsTry ${name} arm-scons.tgz
else
- ${UP_DOWN_LOAD} UploadArmBinariesForHWBots ${name} arm-tcb.tgz
+ ${UP_DOWN_LOAD} UploadArmBinariesForHWBots ${name} arm-scons.tgz
fi
}
@@ -103,14 +106,14 @@ unarchive-for-hw-bots() {
echo "@@@BUILD_STEP fetch_binaries@@@"
if [[ ${try} == "try" ]] ; then
- ${UP_DOWN_LOAD} DownloadArmBinariesForHWBotsTry ${name} arm-tcb.tgz
+ ${UP_DOWN_LOAD} DownloadArmBinariesForHWBotsTry ${name} arm-scons.tgz
else
- ${UP_DOWN_LOAD} DownloadArmBinariesForHWBots ${name} arm-tcb.tgz
+ ${UP_DOWN_LOAD} DownloadArmBinariesForHWBots ${name} arm-scons.tgz
fi
echo "@@@BUILD_STEP untar_binaries@@@"
rm -rf scons-out/
- tar xvfz arm-tcb.tgz --no-same-owner
+ tar xvfz arm-scons.tgz --no-same-owner
}
# Build with gyp - this only exercises the trusted TC and hence this only
@@ -362,6 +365,11 @@ mode-buildbot-arm() {
# Run all 3 test suites in a single scons invocation to minimize
# processing time of *.scons files
scons-tests "arm" "${mode} -k" "small_tests medium_tests large_tests"
+
+ # Run tests in pexe mode
+ scons-tests-no-translator "arm" "${mode} -k pnacl_generate_pexe=1" \
+ "toolchain_tests"
+
# Full test suite of translator for ARM is too flaky on QEMU
# http://code.google.com/p/nativeclient/issues/detail?id=2581
# Running a subset here (and skipping in scons-test() itself).
@@ -390,6 +398,8 @@ mode-buildbot-arm-hw() {
local flags="naclsdk_validate=0 built_elsewhere=1 $1"
scons-tests-no-translator "arm" "${flags} -k" \
"small_tests medium_tests large_tests"
+ scons-tests-no-translator "arm" "${flags} -k pnacl_generate_pexe=1" \
+ "toolchain_tests"
browser-tests "arm" "${flags}"
}

Powered by Google App Engine
This is Rietveld 408576698