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

Issue 1486003004: Revert of [NaCl SDK] Remove support for bionic toolchain (Closed)

Created:
5 years ago by gab
Modified:
5 years ago
Reviewers:
binji, Sam Clegg
CC:
binji+watch_chromium.org, chromium-reviews, native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of [NaCl SDK] Remove support for bionic toolchain (patchset #3 id:40001 of https://codereview.chromium.org/1269623004/ ) Reason for revert: Breaks Linux x64 compile: https://build.chromium.org/p/chromium/builders/Linux%20x64 Log: ninja: error: '../../native_client_sdk/src/libraries/nacl_io/kernel_wrap_bionic.cc', needed by 'newlib_pnacl/obj/native_client_sdk/src/libraries/nacl_io/nacl_io/kernel_wrap_bionic.o', missing and no known rule to make it Original issue's description: > [NaCl SDK] Remove support for bionic toolchain > > CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_nacl_sdk;tryserver.chromium.mac:mac_nacl_sdk;tryserver.chromium.win:win_nacl_sdk > > Committed: https://crrev.com/2bd1f02649d6fe8b0a5616932e40b67478c58174 > Cr-Commit-Position: refs/heads/master@{#362454} TBR=binji@chromium.org,sbc@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=564214

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+847 lines, -28 lines) Patch
M native_client_sdk/src/BUILDING.rst View 1 chunk +2 lines, -2 lines 0 comments Download
M native_client_sdk/src/build_tools/build_projects.py View 4 chunks +9 lines, -0 lines 0 comments Download
M native_client_sdk/src/build_tools/build_sdk.py View 13 chunks +69 lines, -6 lines 0 comments Download
M native_client_sdk/src/build_tools/buildbot_common.py View 1 chunk +2 lines, -2 lines 0 comments Download
M native_client_sdk/src/build_tools/buildbot_run.py View 2 chunks +7 lines, -0 lines 0 comments Download
M native_client_sdk/src/build_tools/json/naclsdk_manifest2.json View 1 chunk +10 lines, -0 lines 0 comments Download
M native_client_sdk/src/build_tools/parse_dsc.py View 1 chunk +1 line, -0 lines 0 comments Download
M native_client_sdk/src/build_tools/tests/update_nacl_manifest_test.py View 5 chunks +25 lines, -1 line 0 comments Download
M native_client_sdk/src/build_tools/update_nacl_manifest.py View 6 chunks +22 lines, -4 lines 0 comments Download
M native_client_sdk/src/doc/sdk/download.rst View 2 chunks +2 lines, -0 lines 0 comments Download
M native_client_sdk/src/examples/demo/nacl_io_demo/example.dsc View 1 chunk +1 line, -1 line 0 comments Download
M native_client_sdk/src/libraries/nacl_io/jsfs/js_fs_node.cc View 1 chunk +2 lines, -1 line 0 comments Download
A native_client_sdk/src/libraries/nacl_io/kernel_wrap_bionic.cc View 1 chunk +502 lines, -0 lines 0 comments Download
M native_client_sdk/src/libraries/nacl_io/library.dsc View 2 chunks +2 lines, -1 line 0 comments Download
M native_client_sdk/src/libraries/ppapi_simple/library.dsc View 1 chunk +1 line, -1 line 0 comments Download
M native_client_sdk/src/libraries/ppapi_simple_cpp/library.dsc View 1 chunk +1 line, -1 line 0 comments Download
A native_client_sdk/src/libraries/ppapi_stub/library.dsc View 1 chunk +19 lines, -0 lines 0 comments Download
A native_client_sdk/src/libraries/ppapi_stub/main.c View 1 chunk +17 lines, -0 lines 0 comments Download
A native_client_sdk/src/libraries/ppapi_stub/ppapi_main.c View 1 chunk +110 lines, -0 lines 0 comments Download
M native_client_sdk/src/resources/Makefile.library.template View 1 chunk +1 line, -1 line 0 comments Download
M native_client_sdk/src/tests/nacl_io_test/example.dsc View 1 chunk +1 line, -1 line 0 comments Download
M native_client_sdk/src/tools/common.mk View 2 chunks +5 lines, -1 line 0 comments Download
M native_client_sdk/src/tools/create_nmf.py View 1 chunk +7 lines, -0 lines 0 comments Download
M native_client_sdk/src/tools/lib/get_shared_deps.py View 1 chunk +10 lines, -3 lines 0 comments Download
M native_client_sdk/src/tools/nacl_gcc.mk View 5 chunks +19 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (4 generated)
gab
Created Revert of [NaCl SDK] Remove support for bionic toolchain
5 years ago (2015-12-01 20:01:31 UTC) #1
commit-bot: I haz the power
5 years ago (2015-12-01 20:04:05 UTC) #5

Powered by Google App Engine
This is Rietveld 408576698