这是indexloc提供的服务,不要输入任何密码
Skip to content

Send proper paths to git cmd for file hashing #1216

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

Merged
merged 1 commit into from
May 11, 2022
Merged

Conversation

gsoltis
Copy link
Contributor

@gsoltis gsoltis commented May 10, 2022

We were setting the wrong (relative) path for git ls-files. Use fs.AbsolutePath to ensure that we have the right thing, and plumb it through from Config.

Fixes #1085 #1198

@vercel
Copy link

vercel bot commented May 10, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
turbo-site ✅ Ready (Inspect) Visit Preview May 10, 2022 at 11:50PM (UTC)

@gsoltis gsoltis marked this pull request as ready for review May 10, 2022 23:52
@jaredpalmer
Copy link
Contributor

Sanity check: hashes are still the same across machines, right?

@gsoltis
Copy link
Contributor Author

gsoltis commented May 11, 2022

@jaredpalmer yup, this is changing the directory that we set for where git is run. The input patterns are unchanged, and at any rate git is reporting the hash of the file's contents

@kodiakhq kodiakhq bot merged commit 5949497 into main May 11, 2022
@kodiakhq kodiakhq bot deleted the gsoltis/fix_git_paths branch May 11, 2022 14:57
@nathanhammond
Copy link
Contributor

One thing that I noticed about this is that we use shell globbing as an input togit here, and doublestar elsewhere—which is a fast path to having inconsistent behavior for inputs. I'm tempted to enumerate prior to passing to git.

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.

Cache miss even when inputs pattern is specified to run in particular folder
3 participants