Skip to content

Update emscripten config settings to conform to the official emscripten toolchain.#1330

Merged
copybara-service[bot] merged 1 commit intomasterfrom
test_360784362
Mar 4, 2021
Merged

Update emscripten config settings to conform to the official emscripten toolchain.#1330
copybara-service[bot] merged 1 commit intomasterfrom
test_360784362

Conversation

@copybara-service
Copy link
Contributor

Update emscripten config settings to conform to the official emscripten toolchain.

Current Bazel config settings for Emscripten are specific to TensorFlow.js's custom Emscripten toolchain, but TFJS is switching to the official toolchain in #4769. The official emscripten toolchain expects crosstool_top to be //emscripten_toolchain:everything.

@google-cla google-cla bot added the cla: yes label Mar 4, 2021
@mattsoulanille
Copy link

Replaces #1329.

…en toolchain.

Current Bazel config settings for Emscripten are specific to TensorFlow.js's custom Emscripten toolchain, but TFJS is switching to the official toolchain in [#4769](tensorflow/tfjs#4769). The official emscripten toolchain expects `crosstool_top` to be `//emscripten_toolchain:everything`.

PiperOrigin-RevId: 360787196
@copybara-service copybara-service bot merged commit f69ac5d into master Mar 4, 2021
@copybara-service copybara-service bot deleted the test_360784362 branch March 4, 2021 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants