-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Rework calls to git #30
Comments
Relevant: I had to work around an issue caused by the parsing of |
I also have a file which breaks
Here's the line it's trying to decode, the first line of a
The problematic bytes there (
However in
I started working on a fix for this by setting Then I started converting I'm intentionally doing things which put me in contact with esoteric formats and encodings as part of a file-identification library I'm working on, so this is probably not something many other users will encounter, but FYI regardless :) |
I believe your file should work when I finish my current refactoring effort: #33. |
Right now, git-crecord calls
git
to perform all Git operations; it uses code derived from Mercurial to do that. However, it’s quite verbose and sometimes usingsubprocess
directly was still necessary.pristine-lfs uses python-sh to talk to Git, but maybe using Dulwich or GitPython could work too?
In the past, Dulwich was used for some functionality, but other parts required Git anyway, so Dulwich was replaced by a hand-rolled alternative. Maybe this is the time to reconsider.
The text was updated successfully, but these errors were encountered: