Skip to content

Commit

Permalink
Fix Windows detection
Browse files Browse the repository at this point in the history
The previous logic included "darwin" (i.e., macOS) and "cigwin", so it
wasn't really good at spotting what we wanted to.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
  • Loading branch information
rtobar committed Nov 30, 2023
1 parent 6a6586d commit 9cd01e2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/test_dump.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ def _do_test_dump(self, method, multiple_values):
env['IJSON_BACKEND'] = 'python'
# Ensure printing works on the subprocess in Windows
# by using utf-8 on its stdout
if 'win' in sys.platform:
if sys.platform == 'win32':
env = dict(os.environ)
env['PYTHONIOENCODING'] = 'utf-8'
cmd = [sys.executable, '-m', 'ijson.dump', '-m', method, '-p', '']
Expand Down

0 comments on commit 9cd01e2

Please sign in to comment.