Skip to content

[WIP] [Caffe2] A reasonable way to detect Python include dirs and library#9361

Closed
pjh5 wants to merge 2 commits intopytorch:masterfrom
pjh5:cmake_python
Closed

[WIP] [Caffe2] A reasonable way to detect Python include dirs and library#9361
pjh5 wants to merge 2 commits intopytorch:masterfrom
pjh5:cmake_python

Conversation

@pjh5
Copy link
Contributor

@pjh5 pjh5 commented Jul 11, 2018

No description provided.

@pjh5 pjh5 requested a review from ezyang July 11, 2018 21:49
@pjh5
Copy link
Contributor Author

pjh5 commented Jul 11, 2018

cc @ezyang who had comments on the last change I made to Python detection

@pjh5 pjh5 changed the title [Caffe2] A reasonable way to detect Python include dirs and library [WIP] [Caffe2] A reasonable way to detect Python include dirs and library Jul 12, 2018
endif()
endif(NOT DEFINED PYTHON_LIBRARY)

# These should fill in the rest of the variables, like versions, but resepct

This comment was marked as off-topic.

Copy link
Contributor

@ezyang ezyang left a comment

Choose a reason for hiding this comment

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

Yes 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.

@pjh5 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 Jul 13, 2018
Summary: Pull Request resolved: pytorch/pytorch#9361

Reviewed By: ml7

Differential Revision: D8837706

Pulled By: pjh5

fbshipit-source-id: 6979f9f37709c23e72b9169531787a60f3b37254
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 13, 2018
Summary: Pull Request resolved: pytorch/pytorch#9361

Reviewed By: ml7

Differential Revision: D8837706

Pulled By: pjh5

fbshipit-source-id: 6979f9f37709c23e72b9169531787a60f3b37254
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary: Pull Request resolved: pytorch#9361

Reviewed By: ml7

Differential Revision: D8837706

Pulled By: pjh5

fbshipit-source-id: 6979f9f37709c23e72b9169531787a60f3b37254
@pjh5 pjh5 deleted the cmake_python branch August 24, 2018 16:54
@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