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

Issue 261773002: Chromium NaCl scripts now uses package_version to extract. (Closed)

Created:
6 years, 7 months ago by David Yen
Modified:
6 years, 7 months ago
CC:
chromium-reviews, binji+watch_chromium.org, chromium-apps-reviews_chromium.org, Sam Clegg, extensions-reviews_chromium.org, Mark Seaborn, Roland McGrath
Visibility:
Public.

Description

Chromium NaCl scripts now uses package_version to extract. Updates NaCl revision from 13106 -> 13115 This CL reverts revision 266284. Now that we have split up the toolchain into multiple tars, we should use the package versioning script to manage which tars to extract. Unfortunately, currently the SDK uses different directory names compared to what the NaCl toolchain directories normally use, so we also must map and move the toolchain directories to the SDK ones. BUG=https://code.google.com/p/nativeclient/issues/detail?id=3839 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=267711

Patch Set 1 #

Patch Set 2 : nacl_revision 13106->13115 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+86 lines, -121 lines) Patch
M DEPS View 1 2 chunks +2 lines, -3 lines 0 comments Download
M build/download_nacl_toolchains.py View 2 chunks +12 lines, -16 lines 1 comment Download
M chrome/test/nacl_test_injection/buildbot_chrome_nacl_stage.py View 1 chunk +6 lines, -3 lines 0 comments Download
M native_client_sdk/src/build_tools/build_sdk.py View 5 chunks +56 lines, -87 lines 0 comments Download
M ppapi/native_client/src/untrusted/pnacl_support_extension/pnacl_support_extension.gyp View 3 chunks +10 lines, -12 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
David Yen
Here are the chromium side changes once more to bring in the package_version changes. Once ...
6 years, 7 months ago (2014-05-01 15:20:30 UTC) #1
David Yen
The NaCl changes have landed (revision 13115), I have rolled the DEPS to that revision ...
6 years, 7 months ago (2014-05-01 17:36:09 UTC) #2
David Yen
All of the trybot results look okay, I think we are ready to land this ...
6 years, 7 months ago (2014-05-01 22:24:35 UTC) #3
bradn
lgtm
6 years, 7 months ago (2014-05-01 22:25:37 UTC) #4
bradnelson
lgtm
6 years, 7 months ago (2014-05-01 22:25:46 UTC) #5
David Yen
The CQ bit was checked by dyen@chromium.org
6 years, 7 months ago (2014-05-01 22:26:14 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dyen@chromium.org/261773002/20001
6 years, 7 months ago (2014-05-01 22:28:17 UTC) #7
commit-bot: I haz the power
Change committed as 267711
6 years, 7 months ago (2014-05-02 01:04:03 UTC) #8
jvoung - send to chromium...
6 years, 7 months ago (2014-05-02 15:49:15 UTC) #9
Message was sent while issue was closed.
https://codereview.chromium.org/261773002/diff/20001/build/download_nacl_tool...
File build/download_nacl_toolchains.py (right):

https://codereview.chromium.org/261773002/diff/20001/build/download_nacl_tool...
build/download_nacl_toolchains.py:54: package_version.main(args)
I don't know if someone's mentioned this already or not, but the main waterfall
IOS builders seem to be attempting to download:

http://build.chromium.org/p/chromium.mac/builders/iOS%20Device%20%28ninja%29/...

python ../../../scripts/slave/runhooks_wrapper.py
 in dir /b/build/slave/iOS_Device__ninja_/build (timeout 600 secs) (maxTime
28800 secs)
 watching logfiles {}
 argv: ['python', '../../../scripts/slave/runhooks_wrapper.py']
 environment:
  AWS_CREDENTIAL_FILE=/b/build/site_config/.boto
  BOTO_CONFIG=/b/build/site_config/.boto
  CHROME_HEADLESS=1
  CHROMIUM_GYP_SYNTAX_CHECK=1
  DEPOT_TOOLS_UPDATE=0
  GYP_CROSSCOMPILE=1
  GYP_DEFINES=component=static_library OS=ios target_subarch=both
  GYP_GENERATORS=ninja
  GYP_GENERATOR_FLAGS=
  HOME=/Users/chrome-bot
  LANDMINES_VERBOSE=1
  LOGNAME=chrome-bot
  PAGER=cat
 
PATH=/Users/chrome-bot/slavebin:/b/depot_tools:/usr/local/git/bin:/Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin
  PWD=/b/build/slave/iOS_Device__ninja_/build
 
PYTHONPATH=/b/build/site_config:/b/build/scripts:/b/build/scripts/release:/b/build/third_party:/b/build_internal/site_config:/b/build_internal/symsrc:/b/build/slave:/b/build/third_party/buildbot_slave_8_4:/b/build/third_party/twisted_10_2:
  SHELL=/bin/bash
  SSH_AUTH_SOCK=/tmp/launch-yVpumu/Listeners
  USER=chrome-bot
 using PTY: False

gclient runhooks

________ running
'/Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/python
src/build/download_nacl_toolchains.py --exclude arm_trusted' in
'/b/build/slave/iOS_Device__ninja_/build'
Error: Command
/Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/python
src/build/download_nacl_toolchains.py --exclude arm_trusted returned non-zero
exit status 1 in /b/build/slave/iOS_Device__ninja_/build
Traceback (most recent call last):
  File "src/build/download_nacl_toolchains.py", line 59, in <module>
    sys.exit(Main(sys.argv[1:]))
  File "src/build/download_nacl_toolchains.py", line 54, in Main
    package_version.main(args)
  File
"/b/build/slave/iOS_Device__ninja_/build/src/native_client/build/package_version/package_version.py",
line 892, in main
    return COMMANDS[arguments.command].do_cmd_func(arguments)
  File
"/b/build/slave/iOS_Device__ninja_/build/src/native_client/build/package_version/package_version.py",
line 634, in _DoSyncCmd
    revision_file)
  File
"/b/build/slave/iOS_Device__ninja_/build/src/native_client/build/package_version/revision_info.py",
line 43, in __init__
    self.LoadRevisionFile(revision_file)
  File
"/b/build/slave/iOS_Device__ninja_/build/src/native_client/build/package_version/revision_info.py",
line 93, in LoadRevisionFile
    with open(revision_file, 'rt') as f:
IOError: [Errno 2] No such file or directory:
u'/b/build/slave/iOS_Device__ninja_/build/src/native_client/toolchain_revisions/nacl_x86_glibc.json'
program finished with exit code 2
elapsedTime=1.991995


I thought IOS builds would have disable_nacl=1 and skip out early, but it looks
like that's not the case... Did it used to skip downloads via a different
mechanism or ?

Powered by Google App Engine
This is Rietveld 408576698