Skip to content

Commit

Permalink
cygwinccompiler: Get the compilers from sysconfig
Browse files Browse the repository at this point in the history
On CLANG64 environment we use should use clang
instead of gcc. This patch gets the compiler from
sysconfig and which would be set when python
was built.

Without this patch, the build fails when it's
trying to check if the compiler is cygwin's one.
  • Loading branch information
naveen521kk committed Sep 9, 2024
1 parent bddd259 commit 16e94a8
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions distutils/cygwinccompiler.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
DistutilsPlatformError,
)
from .file_util import write_file
from .sysconfig import get_config_vars
from .unixccompiler import UnixCCompiler
from .version import LooseVersion, suppress_known_deprecation

Expand Down Expand Up @@ -61,8 +62,12 @@ def __init__(self, verbose=False, dry_run=False, force=False):
"Compiling may fail because of undefined preprocessor macros."
)

self.cc = os.environ.get('CC', 'gcc')
self.cxx = os.environ.get('CXX', 'g++')
self.cc, self.cxx = get_config_vars('CC', 'CXX')

# Override 'CC' and 'CXX' environment variables mainly for
# building using MINGW compiler for MSVC python.
self.cc = os.environ.get('CC', self.cc or 'gcc')
self.cxx = os.environ.get('CXX', self.cxx or 'g++')

self.linker_dll = self.cc
self.linker_dll_cxx = self.cxx
Expand Down

0 comments on commit 16e94a8

Please sign in to comment.