Sebastian Thiel comments

Results 1797 comments of


                                            Sebastian Thiel

IOError: [Errno 24] Too many open files

I just spent some time to find something along the lines of calling `release()` on the `odb` instance of the repository, but only came to the conclusion that such functionality...

`git.repo.base.repo.tree()` blocks deletion of repo directory on Windows (fine in Linux)

@altendky Thanks for posting your progress here, as I do hope that others will find it useful! It's a known limitation of GitPython and its API, and best I can...

Error when handling non-ascii filename

The handling of filepaths is completely broken in `GitPython` as it tries to handle it like it's strings even though these imply a known encoding. This could definitely use an...

Error when handling non-ascii filename

Probably most of it is the way it is due to a lack of understanding of how to handle paths in general and in git specifically. Most problems occour merely...

PushInfo is documented to return a local reference, but a pushed commit need not be a reference

Thanks for looking into this and offering a fix, it's much appreciated! I suggest to keep `local_ref` as is and add another field like `local_hash`, which is set to the...

PushInfo is documented to return a local reference, but a pushed commit need not be a reference

Thanks for the analysis! `local_ref` is already declared as `Union[SymbolicReference, None]`, so setting it to `None` while having a second field named `local_hash` seems to be the way to go...

Plaintext credential is exposed in CommandError if Authorization is passed in http.extraHeader

I acknowledge this issue tentatively as the issue is related to security, but wish there would be steps that help to reproduce it as well. A potential fix could use...

configparser.NoOptionError if submodule's URL key name is `Url` rather than `url` in .gitmodules

Thanks for reporting. The submodules implementation is definitely not according to specification and it's more a matter of luck if it works. Maybe issues like these that seem to be...

total_in(&self) / total_out(&self) implementation for GzDecoder / GzEncoder / MultiGzDecoder

Thanks for bringing this up! Even though I don't have an answer, I wonder if that could be an oversight that could be remedied by adding these methods and wiring...

Using MultiGzDecoder for file with garbage after gzip data

Thanks for bringing this up! Could you provide a testfile along with a test-program that shows the current behaviour? My assumption is that it's not actually possible to decode the...