Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Conversation

driazati
Copy link
Contributor

This gates the debug build behavior behind the BUILD_DEBUG_INFO env variable, which is set in binary_linux_build.sh in pytorch/pytorch#58897

This gates the debug build behavior behind the `BUILD_DEBUG_INFO` env variable, which is set in `binary_linux_build.sh` in pytorch/pytorch#58897
@seemethere seemethere merged commit 2fb7768 into master May 25, 2021
@seemethere seemethere deleted the esotq branch May 25, 2021 22:09
facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request May 26, 2021
Summary:
Pull Request resolved: #58897

We don't need to be building debug info on PRs since it's just filling up S3/CircleCI storage with useless 800 MB zips, this flips it so it's only run on master + release branches. See #58898 for CI signal

Also see pytorch/builder counterpart (unlike the last debuginfo PR there is no hard dependency between these two so there won't be any churn on un-rebased PRs): pytorch/builder#778

Test Plan: Imported from OSS

Reviewed By: seemethere, samestep

Differential Revision: D28689413

Pulled By: driazati

fbshipit-source-id: 77a37e84afe492215008d5e023ceab0c24adb33c
deniskokarev pushed a commit to deniskokarev/pytorch that referenced this pull request Jun 9, 2021
Summary:
Pull Request resolved: pytorch#58897

We don't need to be building debug info on PRs since it's just filling up S3/CircleCI storage with useless 800 MB zips, this flips it so it's only run on master + release branches. See pytorch#58898 for CI signal

Also see pytorch/builder counterpart (unlike the last debuginfo PR there is no hard dependency between these two so there won't be any churn on un-rebased PRs): pytorch/builder#778

Test Plan: Imported from OSS

Reviewed By: seemethere, samestep

Differential Revision: D28689413

Pulled By: driazati

fbshipit-source-id: 77a37e84afe492215008d5e023ceab0c24adb33c
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants