Skip to content

[build] Inject GetEmptyStringAlreadyInited once for static proto#11045

Closed
orionr wants to merge 1 commit intopytorch:masterfrom
orionr:fix-proto-patch
Closed

[build] Inject GetEmptyStringAlreadyInited once for static proto#11045
orionr wants to merge 1 commit intopytorch:masterfrom
orionr:fix-proto-patch

Conversation

@orionr
Copy link
Contributor

@orionr orionr commented Aug 30, 2018

I've been seeing a lot of warnings about multiple declarations of this. Hopefully this fixes it.

cc @Yangqing @mingzhe09088 @ezyang

@orionr orionr changed the title Only inject GetEmptyStringAlreadyInited once for static proto [build] Only inject GetEmptyStringAlreadyInited once for static proto Aug 30, 2018
@orionr orionr changed the title [build] Only inject GetEmptyStringAlreadyInited once for static proto [build] Inject GetEmptyStringAlreadyInited once for static proto Aug 30, 2018
@orionr
Copy link
Contributor Author

orionr commented Aug 30, 2018

@pytorchbot retest this please

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 has imported 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 Aug 30, 2018
Summary:
I've been seeing a lot of warnings about multiple declarations of this. Hopefully this fixes it.

cc Yangqing mingzhe09088 ezyang
Pull Request resolved: pytorch/pytorch#11045

Reviewed By: mingzhe09088

Differential Revision: D9582756

Pulled By: orionr

fbshipit-source-id: 6171485609a2f2f357d6e1c44e26b4ecfcdb4ce6
PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
Summary:
I've been seeing a lot of warnings about multiple declarations of this. Hopefully this fixes it.

cc Yangqing mingzhe09088 ezyang
Pull Request resolved: pytorch#11045

Reviewed By: mingzhe09088

Differential Revision: D9582756

Pulled By: orionr

fbshipit-source-id: 6171485609a2f2f357d6e1c44e26b4ecfcdb4ce6
@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