Don't compare against 'sys.stdout'

In Python 3, 'click.File' returns an instance of 'io.BufferedWriter'
which is not comparable against 'sys.stdout'. Instead of doing this,
compare the 'fileno'.

Signed-off-by: Stephen Finucane <[email protected]>
Closes: #48

Author: stephenfin

git_pw/bundle.py

@@ -3,6 +3,7 @@
 """
 
 import logging
+import pty
 import sys
 
 import click
@@ -76,7 +77,7 @@ def download_cmd(bundle_id, output):
 
         output.write(content)
 
-        if output != sys.stdout:
+        if output.fileno() != pty.STDOUT_FILENO:
             path = output.name
     else:
         path = api.download(bundle['mbox'])

git_pw/patch.py

@@ -3,6 +3,7 @@
 """
 
 import logging
+import pty
 import sys
 
 import arrow
@@ -76,7 +77,7 @@ def download_cmd(patch_id, output, fmt):
 
         output.write(content)
 
-        if output != sys.stdout:
+        if output.fileno() != pty.STDOUT_FILENO:
             path = output.name
     else:
         if fmt == 'diff':

git_pw/series.py

@@ -3,7 +3,7 @@
 """
 
 import logging
-import sys
+import pty
 
 import arrow
 import click
@@ -56,7 +56,7 @@ def download_cmd(series_id, output):
 
         output.write(content)
 
-        if output != sys.stdout:
+        if output.fileno() != pty.STDOUT_FILENO:
             path = output.name
     else:
         path = api.download(series['mbox'])

releasenotes/notes/issue-48-694495f722119fed.yaml

@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    An info-level log is now correctly skipped when downloading patches,
+    bundles or series to ``STDOUT`` on Python 3.