-
Notifications
You must be signed in to change notification settings - Fork 108
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
Add a custom "system level" gitconfig #508
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only one question… other than that looks good!
@@ -82,14 +82,14 @@ jobs: | |||
# Delete the command line tools to make sure they don't get our builds | |||
# messed up with macOS SDK 11 stuff. | |||
sudo rm -rf /Library/Developer/CommandLineTools | |||
- uses: actions/checkout@v2 | |||
- uses: actions/checkout@v3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any reason for not using v4 here too?
- uses: actions/checkout@v3 | |
- uses: actions/checkout@v4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, v4 used node 20 which in turn required glibc 2.28 which we don't have on Linux thanks to our use of the 18.04 docker container. We'll have to take that fight another day but I'm afraid that day might come sooner than we'd like.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
credential-helper
Git for Windows ships a whole environment including its own /etc/gitconfig which we can (and do) manipulate during build in order to insert our "sane defaults" into it. On macOS and Linux that's not the case so in this PR we're creating our own system level config that dugite can then instruct Git to use.