Hi Junio, On Mon, 26 Aug 2019, Junio C Hamano wrote: > Funny that the patch is line-wrapped, which I do not recall ever > seeing in GGG-generated e-mails. Dscho, do you know if anything > funny is going on? Yes, this was me trying to re-send the patch via GMail's web UI because the first time GitGitGadget sent it, it did not get through (only the cover letter did). So I tried to fix the screw-up by sending manually, and screwed it up even more. Sorry about that. Dscho > > Git Gadget <gitgitgadget@xxxxxxxxx> writes: > > > From: "Philip.McGraw" <Philip.McGraw@xxxxxxxxxxx> > > ... > > diff --git a/git-p4.py b/git-p4.py > > index c71a6832e2..33bdb14fd1 100755 > > --- a/git-p4.py > > +++ b/git-p4.py > > @@ -1160,13 +1160,11 @@ def exceedsLargeFileThreshold(self, relPath, contents): > > if contentsSize <= > > gitConfigInt('git-p4.largeFileCompressedThreshold'): > > return False > > contentTempFile = self.generateTempFile(contents) > > - compressedContentFile = > > tempfile.NamedTemporaryFile(prefix='git-p4-large-file', delete=False) > > - zf = zipfile.ZipFile(compressedContentFile.name, mode='w') > > - zf.write(contentTempFile, compress_type=zipfile.ZIP_DEFLATED) > > ... >