mirror of https://github.com/python/cpython
Catch up with terminology change in UnixCCompiler: 'includes' -> 'include_dirs'.
This commit is contained in:
parent
1d0495e05c
commit
0bdd90a7e7
|
@ -262,7 +262,7 @@ class CCompiler:
|
||||||
sources,
|
sources,
|
||||||
output_dir=None,
|
output_dir=None,
|
||||||
macros=None,
|
macros=None,
|
||||||
includes=None,
|
include_dirs=None,
|
||||||
extra_preargs=None,
|
extra_preargs=None,
|
||||||
extra_postargs=None):
|
extra_postargs=None):
|
||||||
"""Compile one or more C/C++ source files. 'sources' must be
|
"""Compile one or more C/C++ source files. 'sources' must be
|
||||||
|
@ -277,7 +277,7 @@ class CCompiler:
|
||||||
undefines a macro. Later definitions/redefinitions/
|
undefines a macro. Later definitions/redefinitions/
|
||||||
undefinitions take precedence.
|
undefinitions take precedence.
|
||||||
|
|
||||||
'includes', if given, must be a list of strings, the directories
|
'include_dirs', if given, must be a list of strings, the directories
|
||||||
to add to the default include file search path for this
|
to add to the default include file search path for this
|
||||||
compilation only.
|
compilation only.
|
||||||
|
|
||||||
|
@ -489,12 +489,12 @@ def new_compiler (plat=None,
|
||||||
return klass (verbose, dry_run, force)
|
return klass (verbose, dry_run, force)
|
||||||
|
|
||||||
|
|
||||||
def gen_preprocess_options (macros, includes):
|
def gen_preprocess_options (macros, include_dirs):
|
||||||
"""Generate C pre-processor options (-D, -U, -I) as used by at
|
"""Generate C pre-processor options (-D, -U, -I) as used by at
|
||||||
least two types of compilers: the typical Unix compiler and Visual
|
least two types of compilers: the typical Unix compiler and Visual
|
||||||
C++. 'macros' is the usual thing, a list of 1- or 2-tuples, where
|
C++. 'macros' is the usual thing, a list of 1- or 2-tuples, where
|
||||||
(name,) means undefine (-U) macro 'name', and (name,value) means
|
(name,) means undefine (-U) macro 'name', and (name,value) means
|
||||||
define (-D) macro 'name' to 'value'. 'includes' is just a list of
|
define (-D) macro 'name' to 'value'. 'include_dirs' is just a list of
|
||||||
directory names to be added to the header file search path (-I).
|
directory names to be added to the header file search path (-I).
|
||||||
Returns a list of command-line options suitable for either
|
Returns a list of command-line options suitable for either
|
||||||
Unix compilers or Visual C++."""
|
Unix compilers or Visual C++."""
|
||||||
|
@ -506,7 +506,7 @@ def gen_preprocess_options (macros, includes):
|
||||||
# line). I don't think it's essential, though, since most (all?)
|
# line). I don't think it's essential, though, since most (all?)
|
||||||
# Unix C compilers only pay attention to the latest -D or -U
|
# Unix C compilers only pay attention to the latest -D or -U
|
||||||
# mention of a macro on their command line. Similar situation for
|
# mention of a macro on their command line. Similar situation for
|
||||||
# 'includes'. I'm punting on both for now. Anyways, weeding out
|
# 'include_dirs'. I'm punting on both for now. Anyways, weeding out
|
||||||
# redundancies like this should probably be the province of
|
# redundancies like this should probably be the province of
|
||||||
# CCompiler, since the data structures used are inherited from it
|
# CCompiler, since the data structures used are inherited from it
|
||||||
# and therefore common to all CCompiler classes.
|
# and therefore common to all CCompiler classes.
|
||||||
|
@ -532,7 +532,7 @@ def gen_preprocess_options (macros, includes):
|
||||||
# shell at all costs when we spawn the command!
|
# shell at all costs when we spawn the command!
|
||||||
pp_opts.append ("-D%s=%s" % macro)
|
pp_opts.append ("-D%s=%s" % macro)
|
||||||
|
|
||||||
for dir in includes:
|
for dir in include_dirs:
|
||||||
pp_opts.append ("-I%s" % dir)
|
pp_opts.append ("-I%s" % dir)
|
||||||
|
|
||||||
return pp_opts
|
return pp_opts
|
||||||
|
|
|
@ -63,19 +63,20 @@ class MSVCCompiler (CCompiler) :
|
||||||
sources,
|
sources,
|
||||||
output_dir=None,
|
output_dir=None,
|
||||||
macros=None,
|
macros=None,
|
||||||
includes=None,
|
include_dirs=None,
|
||||||
extra_preargs=None,
|
extra_preargs=None,
|
||||||
extra_postargs=None):
|
extra_postargs=None):
|
||||||
|
|
||||||
if macros is None:
|
if macros is None:
|
||||||
macros = []
|
macros = []
|
||||||
if includes is None:
|
if include_dirs is None:
|
||||||
includes = []
|
include_dirs = []
|
||||||
|
|
||||||
objectFiles = []
|
objectFiles = []
|
||||||
|
|
||||||
base_pp_opts = gen_preprocess_options (self.macros + macros,
|
base_pp_opts = \
|
||||||
self.include_dirs + includes)
|
gen_preprocess_options (self.macros + macros,
|
||||||
|
self.include_dirs + include_dirs)
|
||||||
|
|
||||||
base_pp_opts.append('/c')
|
base_pp_opts.append('/c')
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue