Skip to content
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

Fix gitlab build (#6025 -> v2) #6027

Merged
merged 1 commit into from
Sep 12, 2024
Merged

Conversation

andrewlock
Copy link
Member

Summary of changes

Enables longpath support on the gitlab Windows jobs

Reason for change

We moved the location of the repo in GitLab, which increased the path length, and we hit long path issues

Implementation details

Make the runner set git config --system core.longpaths true - it's a crazy olde time restriction, so realistically everyone should have this anyway

Test coverage

The build works, so it's fixed!

Other details

We could shorten the paths to work around it, but it's just another accident waiting to happen. This is not the first time 🙈

Backport of #6025

## Summary of changes

Enables longpath support on the gitlab Windows jobs

## Reason for change

We moved the location of the repo in GitLab, which increased the path
length, and we hit long path issues

## Implementation details

Make the runner set `git config --system core.longpaths true` - it's [a
crazy olde time
restriction](https://learn.microsoft.com/en-us/windows/win32/fileio/maximum-file-path-limitation?tabs=powershell),
so realistically everyone should have this anyway

## Test coverage

The build works, so it's fixed!

## Other details
We could shorten the paths to work around it, but it's just another
accident waiting to happen. This is not the first time 🙈

<!-- ⚠️ Note: where possible, please obtain 2 approvals prior to
merging. Unless CODEOWNERS specifies otherwise, for external teams it is
typically best to have one review from a team member, and one review
from apm-dotnet. Trivial changes do not require 2 reviews. -->
@andrewlock andrewlock added the area:builds project files, build scripts, pipelines, versioning, releases, packages label Sep 12, 2024
@andrewlock andrewlock requested a review from a team as a code owner September 12, 2024 11:23
Copy link
Contributor

@daniel-romano-DD daniel-romano-DD left a comment

Choose a reason for hiding this comment

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

TY

@andrewlock andrewlock merged commit e4a222a into release/2.x Sep 12, 2024
87 of 89 checks passed
@andrewlock andrewlock deleted the andrew/fix-gitlab-build-v2 branch September 12, 2024 12:08
@github-actions github-actions bot added this to the vNext-v2 milestone Sep 12, 2024
@andrewlock andrewlock added the type:backport Feature backport to old version label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:builds project files, build scripts, pipelines, versioning, releases, packages type:backport Feature backport to old version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants