Skip to content

Conversation

@nikola-sh
Copy link
Contributor

Git inteprets any files with .pack extensions as packs.
For example, look at prepare_pack function in https://github.com/git/git/blob/master/packfile.c
We should use same rules for Git LFS to prevent problems with arbitrary pack names.

Git inteprets any files with .pack extensions as packs.
For example, look at prepare_pack function in https://github.com/git/git/blob/master/packfile.c
We should use same rules for Git LFS to prevent problems with arbitrary pack names.
Copy link
Member

@bk2204 bk2204 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fine. Thanks for the contribution!

Out of curiosity, is this just for tidiness, or did this cause you a problem somewhere?

@bk2204 bk2204 merged commit 1d0a28b into git-lfs:master May 15, 2019
@nikola-sh
Copy link
Contributor Author

Thank you for approval!

Did this cause you a problem somewhere?

Yes, it caused problem with repository created by VFSForGit ( https://github.com/microsoft/VFSForGit ).
I am tring to add LFS support to this project.

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.

2 participants