Skip to content

[CircleCI] Use canary images until VC++ 14.27 issue is resolved#43220

Closed
malfet wants to merge 1 commit intopytorch:masterfrom
malfet:malfet/use-windows-canary-images
Closed

[CircleCI] Use canary images until VC++ 14.27 issue is resolved#43220
malfet wants to merge 1 commit intopytorch:masterfrom
malfet:malfet/use-windows-canary-images

Conversation

@malfet
Copy link
Copy Markdown
Contributor

@malfet malfet commented Aug 18, 2020

Should fix binary build issue on Windows, and promptly error out if images are updated to a different version of VC++

@malfet malfet requested review from albanD and peterjc123 August 18, 2020 19:42
@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented Aug 18, 2020

💊 CI failures summary and remediations

As of commit da48872 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 3 times.

Copy link
Copy Markdown
Collaborator

@albanD albanD left a comment

Choose a reason for hiding this comment

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

How will it actually error out?
Is it a property of the canary build?

Comment thread .circleci/verbatim-sources/job-specs/pytorch-job-specs.yml Outdated
@malfet malfet force-pushed the malfet/use-windows-canary-images branch from 568b483 to da48872 Compare August 18, 2020 21:05
@malfet
Copy link
Copy Markdown
Contributor Author

malfet commented Aug 18, 2020

How will it actually error out?
Because config.yml now have VC++ version it should error out at this point:

call "C:\Program Files (x86)\Microsoft Visual Studio\%VC_YEAR%\%VC_PRODUCT%\VC\Auxiliary\Build\vcvarsall.bat" x64 -vcvars_ver=%VC_VERSION%

@malfet malfet requested a review from albanD August 18, 2020 21:07
Copy link
Copy Markdown
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.

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@malfet merged this pull request in 1e248ca.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
…torch#43220)

Summary:
Should fix binary build issue on Windows, and promptly error out if images are updated to a different version of VC++

Pull Request resolved: pytorch#43220

Reviewed By: ezyang

Differential Revision: D23198530

Pulled By: malfet

fbshipit-source-id: 0c80361ad7dcfb7aaffccc306b7d741671bedc11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants