Skip to content

[Cherrypick 0.13] Fixing issue with setup_env.sh in docker (#6106) #6109

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 30, 2022
Merged

[Cherrypick 0.13] Fixing issue with setup_env.sh in docker (#6106) #6109

merged 1 commit into from
May 30, 2022

Conversation

YosuaMichael
Copy link
Contributor

Cherrypick #6106

This PR is a fix to the CI that trigger error:

fatal: unsafe repository ('/home/circleci/project' is owned by someone else)
To add an exception for this directory, call:

	git config --global --add safe.directory /home/circleci/project

@vfdev-5 @YosuaMichael
Fixing issue with setup_env.sh in docker (pytorch#6106)
001866a 
New version of git checks folder owner and errors with 'fatal: unsafe repository' if owner is different.
Thus, `root_dir="$(git rev-parse --show-toplevel)"` is failing.

To fix the issue, just added all floder as safe directories
git rev-parse --show-toplevel
@YosuaMichael YosuaMichael changed the title Fixing issue with setup_env.sh in docker (#6106) [Cherrypick 0.13] Fixing issue with setup_env.sh in docker (#6106) May 30, 2022
@YosuaMichael YosuaMichael self-assigned this May 30, 2022
NicolasHug
NicolasHug approved these changes May 30, 2022
View reviewed changes
Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Thanks!

@NicolasHug
Copy link
Member

Let's also add this t o #6081

@YosuaMichael YosuaMichael merged commit 8fe0e0b into pytorch:release/0.13 May 30, 2022
@YosuaMichael YosuaMichael mentioned this pull request May 30, 2022
Closed
@YosuaMichael
Copy link
Contributor Author

The failed CI about ONNX is not related to this PR, will merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NicolasHug NicolasHug NicolasHug approved these changes

Assignees

@YosuaMichael YosuaMichael

Labels
bug cla signed module: ci
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@YosuaMichael @NicolasHug @facebook-github-bot @vfdev-5