Skip to content

.sync/codeql.yml: Use Git long paths on Windows#136

Merged
makubacki merged 1 commit into
microsoft:mainfrom
makubacki:use_git_longpaths_in_codeql_workflow
Mar 1, 2023
Merged

.sync/codeql.yml: Use Git long paths on Windows#136
makubacki merged 1 commit into
microsoft:mainfrom
makubacki:use_git_longpaths_in_codeql_workflow

Conversation

@makubacki
Copy link
Copy Markdown
Member

It is a well known limitation that GitHub uses the pattern
<install_dir>/_work/<repo_name>/<repo_name>/ as the working
directory on the GitHub workflow agents.

It is redundant and particularly impactful on Windows as discussed
below:

https://github.com/msysgit/msysgit/wiki/Git-cannot-create-a-file-or-directory-with-a-long-path

This especially causes an issue in a Mu repository at the moment
(mu_common_intel_min_platform) due to its longer name which is
repeated twice in the path (as <repo-name>).

This change is limited to the GitHub CodeQL workflow and was found
to resolve the issue encountered and not raise any additional issues
in this workflow in testing.

Signed-off-by: Michael Kubacki michael.kubacki@microsoft.com

It is a well known limitation that GitHub uses the pattern
`<install_dir>/_work/<repo_name>/<repo_name>/` as the working
directory on the GitHub workflow agents.

It is redundant and particularly impactful on Windows as discussed
below:

https://github.com/msysgit/msysgit/wiki/Git-cannot-create-a-file-or-directory-with-a-long-path

This especially causes an issue in a Mu repository at the moment
(`mu_common_intel_min_platform`) due to its longer name which is
repeated twice in the path (as `<repo-name>`).

This change is limited to the GitHub CodeQL workflow and was found
to resolve the issue encountered and not raise any additional issues
in this workflow in testing.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@makubacki makubacki requested review from apop5, cfernald, kuqin12 and os-d March 1, 2023 19:31
@makubacki makubacki merged commit 8aac623 into microsoft:main Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants