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

Issue 249153003: NaCl: Update revision in DEPS, r13018 -> r13078. (Closed)

Created:
6 years, 8 months ago by David Yen
Modified:
6 years, 8 months ago
CC:
chromium-reviews, binji+watch_chromium.org, chromium-apps-reviews_chromium.org, Sam Clegg, extensions-reviews_chromium.org, JF, Derek Schuff
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r13018 -> r13078. Chromium NaCl scripts now uses package_version to extract. 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=365817 TEST= trybots Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266284

Patch Set 1 #

Patch Set 2 : rebase #

Patch Set 3 : Updated nacl rev to 13078 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+86 lines, -121 lines) Patch
M DEPS View 1 2 2 chunks +2 lines, -3 lines 0 comments Download
M build/download_nacl_toolchains.py View 2 chunks +12 lines, -16 lines 0 comments 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: 22 (0 generated)
David Yen
After some investigation, Mark is convinced if we bump the NaCl rev to a more ...
6 years, 8 months ago (2014-04-24 21:26:55 UTC) #1
Mark Seaborn
mac_nacl_sdk fails with: .../x86_64-nacl/bin/ld: cannot find crti.o: No such file or directory (from http://build.chromium.org/p/tryserver.chromium/builders/mac_nacl_sdk/builds/1570/steps/compile/logs/stdio)
6 years, 8 months ago (2014-04-24 21:35:52 UTC) #2
David Yen
On 2014/04/24 21:35:52, Mark Seaborn wrote: > mac_nacl_sdk fails with: > > .../x86_64-nacl/bin/ld: cannot find ...
6 years, 8 months ago (2014-04-24 21:38:47 UTC) #3
Mark Seaborn
On 2014/04/24 21:38:47, David Yen wrote: > On 2014/04/24 21:35:52, Mark Seaborn wrote: > > ...
6 years, 8 months ago (2014-04-24 21:48:55 UTC) #4
jvoung (off chromium)
lgtm
6 years, 8 months ago (2014-04-24 22:02:35 UTC) #5
David Yen
On 2014/04/24 22:02:35, jvoung (cr) wrote: > lgtm Now linux_chromium_chromeos_rel no longer works, does anyone ...
6 years, 8 months ago (2014-04-24 23:54:56 UTC) #6
jvoung (off chromium)
On 2014/04/24 23:54:56, David Yen wrote: > On 2014/04/24 22:02:35, jvoung (cr) wrote: > > ...
6 years, 8 months ago (2014-04-25 00:01:14 UTC) #7
Mark Seaborn
On 24 April 2014 16:54, <dyen@chromium.org> wrote: > On 2014/04/24 22:02:35, jvoung (cr) wrote: > ...
6 years, 8 months ago (2014-04-25 00:02:32 UTC) #8
Mark Seaborn
The CQ bit was checked by mseaborn@chromium.org
6 years, 8 months ago (2014-04-25 01:27:42 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dyen@chromium.org/249153003/20007
6 years, 8 months ago (2014-04-25 02:04:08 UTC) #10
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 8 months ago (2014-04-25 02:21:40 UTC) #11
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on chromium_presubmit
6 years, 8 months ago (2014-04-25 02:21:40 UTC) #12
David Yen
It looks like we still need a LGTM from either Brad or Noel, I added ...
6 years, 8 months ago (2014-04-25 16:50:11 UTC) #13
bradnelson
lgtm
6 years, 8 months ago (2014-04-25 18:34:04 UTC) #14
David Yen
The CQ bit was checked by dyen@chromium.org
6 years, 8 months ago (2014-04-25 18:36:10 UTC) #15
David Yen
The CQ bit was unchecked by dyen@chromium.org
6 years, 8 months ago (2014-04-25 20:21:37 UTC) #16
David Yen
The CQ bit was checked by dyen@chromium.org
6 years, 8 months ago (2014-04-25 20:21:47 UTC) #17
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dyen@chromium.org/249153003/20007
6 years, 8 months ago (2014-04-25 21:48:29 UTC) #18
David Yen
Does anyone know why the commit queue is not working? I tried selecting it twice ...
6 years, 8 months ago (2014-04-25 21:49:29 UTC) #19
David Yen
Oh nevermind, it started now. I wonder if it's because I published a comment.
6 years, 8 months ago (2014-04-25 21:50:12 UTC) #20
commit-bot: I haz the power
Change committed as 266284
6 years, 8 months ago (2014-04-25 23:30:28 UTC) #21
tonyg
6 years, 8 months ago (2014-04-26 01:53:46 UTC) #22
Message was sent while issue was closed.
A revert of this CL has been created in
https://codereview.chromium.org/250933006/ by tonyg@chromium.org.

The reason for reverting is: Breaks official build on chromium.perf waterfall.

FAILED: cd ../../native_client/src/shared/gio; export
BUILT_PRODUCTS_DIR=/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release;
export CONFIGURATION=Release; export PRODUCT_NAME=gio_lib; export
SDKROOT=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.6.sdk;
export
SRCROOT=/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release/../../native_client/src/shared/gio;
export SOURCE_ROOT="${SRCROOT}"; export
TARGET_BUILD_DIR=/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release;
export TEMP_DIR="${TMPDIR}";python ../../../../native_client/build/build_nexe.py
-t ../../../../out/Release/gen/sdk/ --arch x86-32 --build glibc_nso --root
../../../.. --name ../../../../out/Release/gen/tc_glibc/lib32/libgio.so --objdir
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib
"--include-dirs=../../../../out/Release/gen/tc_glibc/include ../../../..
../../../src/third_party ../../../.." "--compile_flags=-m32 -fPIC
-fomit-frame-pointer -fasynchronous-unwind-tables  -O2 -g -Wall
-fdiagnostics-show-option -Werror " --gomadir "" "--defines=__linux__
\"__STDC_LIMIT_MACROS=1\" \"__STDC_FORMAT_MACROS=1\" \"_GNU_SOURCE=1\"
\"_BSD_SOURCE=1\" \"_POSIX_C_SOURCE=199506\" \"_XOPEN_SOURCE=600\"
\"DYNAMIC_ANNOTATIONS_ENABLED=1\" \"DYNAMIC_ANNOTATIONS_PREFIX=NACL_\"
\"NACL_BUILD_ARCH=x86\" V8_DEPRECATION_WARNINGS
BLINK_SCALE_FILTERS_AT_RECORD_TIME \"NACL_LINUX=0\" \"NACL_ANDROID=0\"
\"NACL_OSX=1\" \"NACL_WINDOWS=0\" GOOGLE_CHROME_BUILD ENABLE_RLZ
\"USE_LIBJPEG_TURBO=1\" ENABLE_ONE_CLICK_SIGNIN \"ENABLE_REMOTING=1\"
\"ENABLE_WEBRTC=1\" USE_PROPRIETARY_CODECS ENABLE_PEPPER_CDMS
ENABLE_CONFIGURATION_POLICY ENABLE_INPUT_SPEECH ENABLE_NOTIFICATIONS
\"ENABLE_HIDPI=1\" DISCARDABLE_MEMORY_ALWAYS_SUPPORTED_NATIVELY
SYSTEM_NATIVELY_SIGNALS_MEMORY_PRESSURE \"ENABLE_EGLIMAGE=1\"
\"ENABLE_TASK_MANAGER=1\" \"ENABLE_EXTENSIONS=1\"
\"ENABLE_PLUGIN_INSTALLATION=1\" \"ENABLE_PLUGINS=1\"
\"ENABLE_SESSION_SERVICE=1\" \"ENABLE_THEMES=1\" \"ENABLE_AUTOFILL_DIALOG=1\"
\"ENABLE_BACKGROUND=1\" \"ENABLE_GOOGLE_NOW=1\" \"CLD_VERSION=2\"
\"ENABLE_FULL_PRINTING=1\" \"ENABLE_PRINTING=1\" \"ENABLE_SPELLCHECK=1\"
\"ENABLE_CAPTIVE_PORTAL_DETECTION=1\" \"ENABLE_APP_LIST=1\"
\"ENABLE_SETTINGS_APP=1\" \"ENABLE_MANAGED_USERS=1\"
\"ENABLE_SERVICE_DISCOVERY=1\" \"NACL_TARGET_SUBARCH=32\"
\"NACL_TARGET_ARCH=x86\" \"NACL_BUILD_SUBARCH=32\" \"NACL_BUILD_ARCH=x86\""
"--link_flags=-m32 -B../../../../out/Release/gen/tc_glibc/lib32  "
"--source-list=../../../../out/gypfiles/native_client/src/shared/gio/glibc-x86-32-so.gio_lib.source_list.gypcmd"
/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release/gen/sdk/mac_x86/nacl_x86_glibc/bin/../lib/gcc/x86_64-nacl/4.4.3/../../../../x86_64-nacl/bin/ld:
cannot find crti.o: No such file or directory
/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release/gen/sdk/mac_x86/nacl_x86_glibc/bin/../lib/gcc/x86_64-nacl/4.4.3/../../../../x86_64-nacl/bin/ld:
cannot find crtbeginS.o: No such file or directory
collect2: ld returned 1 exit status
../../../../out/Release/gen/sdk/mac_x86/nacl_x86_glibc/bin/x86_64-nacl-g++ -o
../../../../out/Release/gen/tc_glibc/lib32/libgio.so -Wl,--as-needed
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib/gio_mem.o
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib/gio.o
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib/gprintf.o
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib/gio_mem_snapshot.o
-m32 -B../../../../out/Release/gen/tc_glibc/lib32 -shared -Wl,-soname,libgio.so
FAILED: Command
'['../../../../out/Release/gen/sdk/mac_x86/nacl_x86_glibc/bin/x86_64-nacl-g++',
'-o', '../../../../out/Release/gen/tc_glibc/lib32/libgio.so', '-Wl,--as-needed',
'../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib/gio_mem.o',
'../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib/gio.o',
'../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib/gprintf.o',
'../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-32-so/gio_lib/gio_mem_snapshot.o',
'-m32', '-B../../../../out/Release/gen/tc_glibc/lib32', '-shared',
'-Wl,-soname,libgio.so']' returned non-zero exit status 1
FAILED: cd ../../native_client/src/shared/gio; export
BUILT_PRODUCTS_DIR=/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release;
export CONFIGURATION=Release; export PRODUCT_NAME=gio_lib; export
SDKROOT=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.6.sdk;
export
SRCROOT=/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release/../../native_client/src/shared/gio;
export SOURCE_ROOT="${SRCROOT}"; export
TARGET_BUILD_DIR=/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release;
export TEMP_DIR="${TMPDIR}";python ../../../../native_client/build/build_nexe.py
-t ../../../../out/Release/gen/sdk/ --arch x86-64 --build glibc_nso --root
../../../.. --name ../../../../out/Release/gen/tc_glibc/lib64/libgio.so --objdir
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib
"--include-dirs=../../../../out/Release/gen/tc_glibc/include ../../../..
../../../src/third_party ../../../.." "--compile_flags=-m64 -fPIC
-fomit-frame-pointer -fasynchronous-unwind-tables  -O2 -g -Wall
-fdiagnostics-show-option -Werror " --gomadir "" "--defines=__linux__
\"__STDC_LIMIT_MACROS=1\" \"__STDC_FORMAT_MACROS=1\" \"_GNU_SOURCE=1\"
\"_BSD_SOURCE=1\" \"_POSIX_C_SOURCE=199506\" \"_XOPEN_SOURCE=600\"
\"DYNAMIC_ANNOTATIONS_ENABLED=1\" \"DYNAMIC_ANNOTATIONS_PREFIX=NACL_\"
\"NACL_BUILD_ARCH=x86\" V8_DEPRECATION_WARNINGS
BLINK_SCALE_FILTERS_AT_RECORD_TIME \"NACL_LINUX=0\" \"NACL_ANDROID=0\"
\"NACL_OSX=1\" \"NACL_WINDOWS=0\" GOOGLE_CHROME_BUILD ENABLE_RLZ
\"USE_LIBJPEG_TURBO=1\" ENABLE_ONE_CLICK_SIGNIN \"ENABLE_REMOTING=1\"
\"ENABLE_WEBRTC=1\" USE_PROPRIETARY_CODECS ENABLE_PEPPER_CDMS
ENABLE_CONFIGURATION_POLICY ENABLE_INPUT_SPEECH ENABLE_NOTIFICATIONS
\"ENABLE_HIDPI=1\" DISCARDABLE_MEMORY_ALWAYS_SUPPORTED_NATIVELY
SYSTEM_NATIVELY_SIGNALS_MEMORY_PRESSURE \"ENABLE_EGLIMAGE=1\"
\"ENABLE_TASK_MANAGER=1\" \"ENABLE_EXTENSIONS=1\"
\"ENABLE_PLUGIN_INSTALLATION=1\" \"ENABLE_PLUGINS=1\"
\"ENABLE_SESSION_SERVICE=1\" \"ENABLE_THEMES=1\" \"ENABLE_AUTOFILL_DIALOG=1\"
\"ENABLE_BACKGROUND=1\" \"ENABLE_GOOGLE_NOW=1\" \"CLD_VERSION=2\"
\"ENABLE_FULL_PRINTING=1\" \"ENABLE_PRINTING=1\" \"ENABLE_SPELLCHECK=1\"
\"ENABLE_CAPTIVE_PORTAL_DETECTION=1\" \"ENABLE_APP_LIST=1\"
\"ENABLE_SETTINGS_APP=1\" \"ENABLE_MANAGED_USERS=1\"
\"ENABLE_SERVICE_DISCOVERY=1\" \"NACL_TARGET_SUBARCH=32\"
\"NACL_TARGET_ARCH=x86\" \"NACL_BUILD_SUBARCH=32\" \"NACL_BUILD_ARCH=x86\""
"--link_flags=-B../../../../out/Release/gen/tc_glibc/lib64  "
"--source-list=../../../../out/gypfiles/native_client/src/shared/gio/glibc-x86-64-so.gio_lib.source_list.gypcmd"
/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release/gen/sdk/mac_x86/nacl_x86_glibc/bin/../lib/gcc/x86_64-nacl/4.4.3/../../../../x86_64-nacl/bin/ld:
cannot find crti.o: No such file or directory
/Volumes/data/b/build/slave/chromium-rel-mac-builder/build/src/out/Release/gen/sdk/mac_x86/nacl_x86_glibc/bin/../lib/gcc/x86_64-nacl/4.4.3/../../../../x86_64-nacl/bin/ld:
cannot find crtbeginS.o: No such file or directory
collect2: ld returned 1 exit status
../../../../out/Release/gen/sdk/mac_x86/nacl_x86_glibc/bin/x86_64-nacl-g++ -o
../../../../out/Release/gen/tc_glibc/lib64/libgio.so -Wl,--as-needed
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib/gio_mem.o
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib/gio.o
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib/gprintf.o
../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib/gio_mem_snapshot.o
-B../../../../out/Release/gen/tc_glibc/lib64 -shared -Wl,-soname,libgio.so
FAILED: Command
'['../../../../out/Release/gen/sdk/mac_x86/nacl_x86_glibc/bin/x86_64-nacl-g++',
'-o', '../../../../out/Release/gen/tc_glibc/lib64/libgio.so', '-Wl,--as-needed',
'../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib/gio_mem.o',
'../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib/gio.o',
'../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib/gprintf.o',
'../../../../out/Release/obj/native_client/src/shared/gio/gio_lib.gen/glibc-x86-64-so/gio_lib/gio_mem_snapshot.o',
'-B../../../../out/Release/gen/tc_glibc/lib64', '-shared',
'-Wl,-soname,libgio.so']' returned non-zero exit status 1
ninja: build stopped: subcommand failed.

http://build.chromium.org/p/chromium.perf/builders/Mac%20Builder/builds/60425....

Powered by Google App Engine
This is Rietveld 408576698