Skip to content

[build] Support environments with no libprotobuf#11161

Closed
orionr wants to merge 1 commit intopytorch:masterfrom
orionr:fix-no-libprotobuf
Closed

[build] Support environments with no libprotobuf#11161
orionr wants to merge 1 commit intopytorch:masterfrom
orionr:fix-no-libprotobuf

Conversation

@orionr
Copy link
Contributor

@orionr orionr commented Aug 31, 2018

Just pulling this out of #10611

Make sure we can support environments where we don't have libprotobuf installed when we link-local protobuf.

cc @goldsborough @Yangqing

@orionr
Copy link
Contributor Author

orionr commented Sep 4, 2018

And thanks to @goldsborough for all the hard work getting to this elegant fix! It's actually his code.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

orionr is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Sep 5, 2018
Summary:
Just pulling this out of pytorch/pytorch#10611

Make sure we can support environments where we don't have libprotobuf installed when we link-local protobuf.

cc goldsborough Yangqing
Pull Request resolved: pytorch/pytorch#11161

Differential Revision: D9650282

Pulled By: orionr

fbshipit-source-id: 447b5e54cd2639973b4b10f58590d1c693a988d4
PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
Summary:
Just pulling this out of pytorch#10611

Make sure we can support environments where we don't have libprotobuf installed when we link-local protobuf.

cc goldsborough Yangqing
Pull Request resolved: pytorch#11161

Differential Revision: D9650282

Pulled By: orionr

fbshipit-source-id: 447b5e54cd2639973b4b10f58590d1c693a988d4
@ezyang ezyang added the merged label Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants