Skip to content

Commit 7272fc8

Browse files
committed
Add explanatory comment for MS_WIN64
1 parent e646b64 commit 7272fc8

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

setup.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,9 @@ def _extract_tarball(self, tarball_path, lib_dir):
9898
jq_extension = Extension(
9999
"jq",
100100
sources=["jq.pyx"],
101-
define_macros=[("MS_WIN64" , 1)] if os.name == "nt" and sys.maxsize > 2**32 else None, # https://github.com/cython/cython/issues/2670
101+
# MS_WIN64 has to be set to successfully build when using MinGW for 64-bit
102+
# Windows. See: https://github.com/cython/cython/issues/2670
103+
define_macros=[("MS_WIN64" , 1)] if os.name == "nt" and sys.maxsize > 2**32 else None,
102104
include_dirs=[os.path.join(jq_lib_dir, "src")],
103105
extra_link_args=["-lm"] + (["-Wl,-Bstatic", "-lpthread", "-lshlwapi", "-static-libgcc"] if os.name == 'nt' else []) + link_args_deps,
104106
extra_objects=extra_objects,

0 commit comments

Comments
 (0)