Skip to content

Emit the C++ version when compiling pytorch from source.#32819

Closed
v0dro wants to merge 1 commit intopytorch:masterfrom
v0dro:emit-cpp-version
Closed

Emit the C++ version when compiling pytorch from source.#32819
v0dro wants to merge 1 commit intopytorch:masterfrom
v0dro:emit-cpp-version

Conversation

@v0dro
Copy link
Contributor

@v0dro v0dro commented Jan 30, 2020

The need for this is felt because sometimes we change a build script and change the std=c++XX flag, which does not get caught until the compilation has progressed for a while.

#31757

@v0dro v0dro changed the title Emit the C++ version when pytorch from source. Emit the C++ version when compiling pytorch from source. Jan 30, 2020
@kostmo
Copy link
Member

kostmo commented Jan 30, 2020

💊 CircleCI build failures summary and remediations

As of commit 73e31dd:

None of the build failures appear to be your fault.

  • 2/2 broken upstream at merge base 821b6aa since Jan 30

    Please rebase on the viable/strict branch (expand for instructions)

    If your commit is newer than viable/strict, you can try basing on an older, stable commit:

    git fetch origin viable/strict
    git rebase --onto viable/strict $(git merge-base origin/master HEAD)
    

    If your commit is older than viable/strict:

    git fetch origin viable/strict
    git rebase viable/strict
    

    Check out the recency history of this "viable master" tracking branch.

Detailed failure analysis

One may explore the probable reasons each build failed interactively on the Dr. CI website.

🚧 2 upstream failures recognized by patterns:

These builds matched patterns, but were probably caused by upstream breakages:


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.

This comment has been revised 1 time.

@vincentqb vincentqb requested a review from ezyang January 30, 2020 16:16
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.

..suuuure.

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
Copy link
Contributor

@ezyang merged this pull request in b34e0dd.

BowenBao pushed a commit to BowenBao/pytorch that referenced this pull request Feb 12, 2020
Summary:
The need for this is felt because sometimes we change a build script and change the `std=c++XX` flag, which does not get caught until the compilation has progressed for a while.

pytorch#31757
Pull Request resolved: pytorch#32819

Differential Revision: D19697205

Pulled By: ezyang

fbshipit-source-id: b045a1d15e24c4c6007b5d1464756051d32bf911
ttumiel pushed a commit to ttumiel/pytorch that referenced this pull request Mar 4, 2020
Summary:
The need for this is felt because sometimes we change a build script and change the `std=c++XX` flag, which does not get caught until the compilation has progressed for a while.

pytorch#31757
Pull Request resolved: pytorch#32819

Differential Revision: D19697205

Pulled By: ezyang

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants