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

Avoid user defined git template in resolver #528

Merged
merged 3 commits into from
Mar 20, 2022

Conversation

lzap
Copy link
Contributor

@lzap lzap commented Oct 12, 2021

I have couple of hooks in my global git template (skeleton) directory, this breaks shards completely and it was such a pain to figure it out. Symptoms are:

Resolving dependencies
Fetching https://github.com/will/crystal-pg.git
Installing pg (0.24.0)
Failed git --work-tree=/home/lzap/work/invidious/lib/pg checkout
 refs/tags/v0.24.0 -- . ().
 Maybe a commit, branch or file doesn't exist?

The patch overrides the user-defined configuration so git checkouts are clean.

I have couple of hooks in my global git template (skeleton)
directory, this breaks shards completely and it was such a pain
to figure it out. Symptoms are:

```
Resolving dependencies
Fetching https://github.com/will/crystal-pg.git
Installing pg (0.24.0)
Failed git --work-tree=/home/lzap/work/invidious/lib/pg checkout
 refs/tags/v0.24.0 -- . ().
 Maybe a commit, branch or file doesn't exist?
```
@lzap
Copy link
Contributor Author

lzap commented Oct 13, 2021

Hmmm are failures really relevant? I am trying to navigate through it, this is weird.

@bcardiff bcardiff added this to the v0.17.0 milestone Mar 17, 2022
@straight-shoota straight-shoota merged commit 888f84c into crystal-lang:master Mar 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants