diff mbox series

[PATCHv1,1/3] git-p4: do not fail in verbose mode for missing 'fileSize' key

Message ID 20181012052833.6945-2-luke@diamand.org (mailing list archive)
State New, archived
Headers show
Series git-p4: improved unshelving | expand

Commit Message

Luke Diamand Oct. 12, 2018, 5:28 a.m. UTC
If deleting or moving a file, sometimes P4 doesn't report the file size.

The code handles this just fine but some logging crashes. Stop this

There was some earlier discussion on the list about this:


Signed-off-by: Luke Diamand <luke@diamand.org>
 git-p4.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
diff mbox series


diff --git a/git-p4.py b/git-p4.py
index 7fab255584..5701bad06a 100755
--- a/git-p4.py
+++ b/git-p4.py
@@ -2775,7 +2775,10 @@  def streamOneP4File(self, file, contents):
         relPath = self.stripRepoPath(file['depotFile'], self.branchPrefixes)
         relPath = self.encodeWithUTF8(relPath)
         if verbose:
-            size = int(self.stream_file['fileSize'])
+            if 'fileSize' in self.stream_file:
+                size = int(self.stream_file['fileSize'])
+            else:
+                size = 0 # deleted files don't get a fileSize apparently
             sys.stdout.write('\r%s --> %s (%i MB)\n' % (file['depotFile'], relPath, size/1024/1024))