diff options
| author |  Jacques Pienaar <jpienaar@google.com> | 2018-03-15 12:58:08 -0700 |
|---|---|---|
| committer |  TensorFlower Gardener <gardener@tensorflow.org> | 2018-03-15 13:03:19 -0700 |
| commit | ccd8079e579604547f4b4d8a6b061cfdc6ce49bf (patch) | |
| tree | 0d498e84ca32a101afcada0993a30a5e3b0452a2 /third_party/py | |
| parent | 61032e9ca7bf9849cb65db9b646381d124080856 (diff) | |
Merge changes from github.
PiperOrigin-RevId: 189231636
Diffstat (limited to 'third_party/py')
| -rw-r--r-- | third_party/py/BUILD.tpl | 22 |
1 files changed, 14 insertions, 8 deletions
diff --git a/third_party/py/BUILD.tpl b/third_party/py/BUILD.tpl index de06ad5f27..1dd8ab433a 100644 --- a/third_party/py/BUILD.tpl +++ b/third_party/py/BUILD.tpl @@ -2,20 +2,26 @@ licenses(["restricted"]) package(default_visibility = ["//visibility:public"]) +# To build Python C/C++ extension on Windows, we need to link to python import library pythonXY.lib +# See https://docs.python.org/3/extending/windows.html +cc_import( + name = "python_lib", + interface_library = select({ + ":windows": ":python_import_lib", + # A placeholder for Unix platforms which makes --no_build happy. + "//conditions:default": "not-existing.lib", + }), + system_provided = 1, +) + cc_library( name = "python_headers", hdrs = [":python_include"], - data = select({ - ":windows": [":python_import_lib"], + deps = select({ + ":windows": [":python_lib"], "//conditions:default": [], }), includes = ["python_include"], - linkopts = select({ - # TODO(pcloudy): Ideally, this should just go into deps after resolving - # https://github.com/bazelbuild/bazel/issues/3237, - ":windows": ["$(locations :python_import_lib)"], - "//conditions:default": [], - }), ) cc_library( |