File protocol doesn't support LFS files
Nobody has claimed this yet.
- Dominant language
- F#
- Stars
- 2.1k
- Forks
- 528
- Avg merge
- 1d 12m
- Merged PRs (30d)
- 54
Description
Description
Using the file protocol to cache git repos does not support LFS. In our case a prebuilt binary is checked into git lfs, that needs to be pulled with the repo.
Since Paket caches a bare repo, and then use to file protocol to copy it locally, this file was missing when we went to build.
Repro steps
reference a repo that contains LFS files.
Expected behavior
That the LFS files would be copied
Actual behavior
They weren't
Known workarounds
this is actually referenced in a closed issue from 2017, he just kinda tacked it on at the end (I'm surprised a new issue wasn't created): https://github.com/fsprojects/Paket/pull/2226#issuecomment-302044193
I haven't had a chance to try this workaround, we had to move forward with submodules for the time being.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue identifies Paket's file-protocol path for copying a cached bare Git repository; start by tracing that path and reproducing the case with a repository containing Git LFS files. Compare the result with the workaround referenced in closed issue 2017 and verify that the LFS files are present after copying and available for the build.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fsharp, git
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100