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

[Re-landing] Fix caffe2 windows CI for new Windows AMI #21452

Closed
wants to merge 1 commit into from

Conversation

peterjc123
Copy link
Collaborator

The alternative of #21410.

@pytorchbot pytorchbot added caffe2 module: build Build system issues module: cpp Related to C++ API labels Jun 6, 2019
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Jun 6, 2019
@peterjc123 peterjc123 requested review from ezyang and removed request for ebetica, yf225 and goldsborough June 6, 2019 06:43
@pytorchbot pytorchbot added the module: ci Related to continuous integration label Jun 6, 2019
@peterjc123 peterjc123 force-pushed the caffe_ci_fix_2 branch 2 times, most recently from 870956e to 8a47868 Compare June 6, 2019 12:52
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.

Good to go assuming tests pass

(cherry picked from commit ce4a9c4)

Fix caffe2 ci job for new Windows AMI

(cherry picked from commit 67d1093582699788b3098c719f7b23c9bd30f9f5)
(cherry picked from commit 4c52769)

Install jit test binaries

Install test_api binaries

Path fixes

More path changes

More path fixes

Apply INSTALL_TEST check

Fix dir names

Fix path

More path fixes

Try path

debug

Fix paths

Path try 1
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.

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

facebook-github-bot pushed a commit that referenced this pull request Jun 6, 2019
Summary:
Fixes #21026.
1. Improve build docs for Windows
2. Change `BUILD_SHARED_LIBS=ON` for Caffe2 local builds
3. Change to out-source builds for LibTorch and Caffe2 (transferred to #21452)
Pull Request resolved: #21190

Differential Revision: D15695223

Pulled By: ezyang

fbshipit-source-id: 0ad69d7553a40fe627582c8e0dcf655f6f63bfdf
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jun 6, 2019
Summary:
The alternative of #21410.
Pull Request resolved: pytorch/pytorch#21452

Differential Revision: D15701767

Pulled By: ezyang

fbshipit-source-id: e65c1d6bfcc98e88460f4a57e5b99c2f395c0ceb
@peterjc123 peterjc123 deleted the caffe_ci_fix_2 branch June 7, 2019 01:38
@facebook-github-bot
Copy link
Contributor

@ezyang merged this pull request in bb78863.

yf225 pushed a commit to yf225/pytorch that referenced this pull request Jun 11, 2019
yf225 pushed a commit to yf225/pytorch that referenced this pull request Jun 12, 2019
@yf225
Copy link
Contributor

yf225 commented Jun 12, 2019

Reverting this PR because it breaks libtorch binary build (sample error: https://circleci.com/gh/pytorch/pytorch/1939687).

peterjc123 added a commit to peterjc123/pytorch that referenced this pull request Jun 14, 2019
Summary:
The alternative of pytorch#21410.
Pull Request resolved: pytorch#21452

Differential Revision: D15701767

Pulled By: ezyang

fbshipit-source-id: e65c1d6bfcc98e88460f4a57e5b99c2f395c0ceb
(cherry picked from commit bb78863)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
caffe2 Merged module: build Build system issues module: ci Related to continuous integration module: cpp Related to C++ API oncall: jit Add this issue/PR to JIT oncall triage queue open source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants