Platform independent manifest verification

bin/verify-manifest

@@ -1,12 +0,0 @@
-#!/bin/bash
-OUTPUT=`check-manifest`
-EXPECTED_OUTPUT="lists of files in version control and sdist match"
-if [[ "$OUTPUT" != "$EXPECTED_OUTPUT" ]]; then
-    echo "Check-manifest didn't match."
-    echo "OUTPUT: $OUTPUT"
-    echo "EXPECTED_OUTPUT: $EXPECTED_OUTPUT"
-    exit 1 # Error
-else
-    echo "Manifest verified."
-    exit 0
-fi

bin/verify-manifest.py

@@ -0,0 +1,17 @@
+import os, subprocess
+import errno, sys
+
+EXPECTED_OUTPUT = "lists of files in version control and sdist match"
+
+
+def check_manifest():
+	os.chdir('..')
+	output = subprocess.check_output(["check-manifest"]).decode()
+	print(output)
+	if EXPECTED_OUTPUT == output.strip():
+		return 1
+	else:
+		sys.exit(2)
+
+if __name__ == '__main__':
+	check_manifest()

minecraft/networking/types.py

@@ -104,14 +104,14 @@ class VarInt(Type):
 
     @staticmethod
     def send(value, socket):
-        o = ""
+        out = ""
         while True:
-            b = value & 0x7F
+            byte = value & 0x7F
             value >>= 7
-            o += struct.pack("B", b | (0x80 if value > 0 else 0))
+            out += struct.pack("B", byte | (0x80 if value > 0 else 0))
             if value == 0:
                 break
-        socket.send(o)
+        socket.send(out)
 
     @staticmethod
     def size(value):

tox.ini

@@ -62,6 +62,7 @@ commands =
 basepython = python3.4
 deps =
     check-manifest
+changedir = bin
 commands =
-    {toxinidir}/bin/verify-manifest
+    python verify-manifest.py