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

windows builds fail to clone repository #6650

Closed
da1nerd opened this issue Jan 16, 2020 · 5 comments
Closed

windows builds fail to clone repository #6650

da1nerd opened this issue Jan 16, 2020 · 5 comments
Assignees
Milestone

Comments

@da1nerd
Copy link
Contributor

da1nerd commented Jan 16, 2020

I believe we are encountering the same issue as described in civetweb/civetweb#812

The clone fails with this error on the windows environment

##[error]fatal: filename in tree entry contains backslash: 'reports\report.html'

The problem seems to stem from changes in git 2.24.0. This is the version installed on the virtual environment.

See the following from the Git 2.25 Release Notes:

 * An earlier update to Git for Windows declared that a tree object is
   invalid if it has a path component with backslash in it, which was
   overly strict, which has been corrected.  The only protection the
   Windows users need is to prevent such path (or any path that their
   filesystem cannot check out) from entering the index.
   (merge 224c7d70fa js/mingw-loosen-overstrict-tree-entry-checks later to maint).
@da1nerd
Copy link
Contributor Author

da1nerd commented Jan 16, 2020

I've opened an issue to update the version of git installed on windows environments actions/runner-images#278

@da1nerd
Copy link
Contributor Author

da1nerd commented Jan 16, 2020

I've circumvented this problem by following the directions in actions/checkout#116 (comment).

This checks out the first commit which is faster and avoids the history that's breaking git.

@da1nerd
Copy link
Contributor Author

da1nerd commented Jan 16, 2020

Now we're getting the same error as in #6648

@da1nerd da1nerd self-assigned this Jan 16, 2020
@da1nerd
Copy link
Contributor Author

da1nerd commented Jan 21, 2020

This was fixed in #6649

@birchamp birchamp added this to the tC Sprint #89 milestone Jan 29, 2020
@PhotoNomad0
Copy link
Contributor

Looking good

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

No branches or pull requests

3 participants