Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update double-conversion to 3.1.7, and use vendor-provided Bazel BUILD file instead. #53618

Merged

Conversation

yongtang
Copy link
Member

@yongtang yongtang commented Jan 3, 2022

The double-conversion library used in tensorflow has not been updated for quite some time (previous version used was in 2018) and the Bazel BUILD file was maintained by tensorflow.

Recently double-conversion has provided Bazel BUILD file.

This PR updates double-conversion to the latest 3.1.7, and use the vendor-provided BUILD file instead so that tensorflow does not need to maintain the Bazel BUILD file separately.

Signed-off-by: Yong Tang yong.tang.github@outlook.com

…D file instead.

The double-conversion library used in tensorflow has not been updated
for quite some time (previous version used was in 2018) and the Bazel
BUILD file was maintained by tensorflow.

Recently double-conversion has provided Bazel BUILD file.

This PR updates double-conversion to the latest 3.1.7, and use the vendor-provided
BUILD file instead so that tensorflow does not need to maintain the Bazel BUILD file
separately.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@google-ml-butler google-ml-butler bot added the size:M CL Change Size: Medium label Jan 3, 2022
@google-ml-butler google-ml-butler bot added the awaiting review Pull request awaiting review label Jan 3, 2022
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Jan 4, 2022
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Jan 5, 2022
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jan 5, 2022
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jan 5, 2022
@copybara-service copybara-service bot merged commit b0b3344 into tensorflow:master Jan 6, 2022
@google-ml-butler google-ml-butler bot removed awaiting review Pull request awaiting review ready to pull PR ready for merge process labels Jan 6, 2022
@yongtang yongtang deleted the double-conversion-3.1.7 branch January 6, 2022 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M CL Change Size: Medium
Projects
PR Queue
  
Approved by Reviewer
Development

Successfully merging this pull request may close these issues.

None yet

4 participants