Skip to content

Fix IDEEP CMakefile#9217

Closed
yinghai wants to merge 1 commit intopytorch:masterfrom
yinghai:ideep_cmake
Closed

Fix IDEEP CMakefile#9217
yinghai wants to merge 1 commit intopytorch:masterfrom
yinghai:ideep_cmake

Conversation

@yinghai
Copy link
Contributor

@yinghai yinghai commented Jul 6, 2018

The reason is that we are referencing __ideep_looked_for here:

find_package_handle_standard_args(IDEEP DEFAULT_MSG ${__ideep_looked_for})

This was first flushed out in #8105 and probably can help with #9024

@yinghai yinghai requested review from bddppq and pjh5 July 6, 2018 20:35
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.

@yinghai has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@yinghai
Copy link
Contributor Author

yinghai commented Jul 6, 2018

@pytorchbot retest this please

zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 6, 2018
Summary:
The reason is that we are referencing `__ideep_looked_for` here: https://github.com/pytorch/pytorch/blob/77484d91db052dfcfa22a38408349853b6246f8a/cmake/Modules/FindMKL.cmake#L350

This was first flushed out in pytorch/pytorch#8105 and probably can help with pytorch/pytorch#9024
Pull Request resolved: pytorch/pytorch#9217

Reviewed By: houseroad

Differential Revision: D8754491

Pulled By: yinghai

fbshipit-source-id: 70aecc2d60684b9ea522403dc98a0a1a2c3db7e6
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 13, 2018
Summary:
The reason is that we are referencing `__ideep_looked_for` here: https://github.com/pytorch/pytorch/blob/77484d91db052dfcfa22a38408349853b6246f8a/cmake/Modules/FindMKL.cmake#L350

This was first flushed out in pytorch/pytorch#8105 and probably can help with pytorch/pytorch#9024
Pull Request resolved: pytorch/pytorch#9217

Reviewed By: houseroad

Differential Revision: D8754491

Pulled By: yinghai

fbshipit-source-id: 70aecc2d60684b9ea522403dc98a0a1a2c3db7e6
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
The reason is that we are referencing `__ideep_looked_for` here: https://github.com/pytorch/pytorch/blob/77484d91db052dfcfa22a38408349853b6246f8a/cmake/Modules/FindMKL.cmake#L350

This was first flushed out in pytorch#8105 and probably can help with pytorch#9024
Pull Request resolved: pytorch#9217

Reviewed By: houseroad

Differential Revision: D8754491

Pulled By: yinghai

fbshipit-source-id: 70aecc2d60684b9ea522403dc98a0a1a2c3db7e6
@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