From b5d52d737ffc4e2e6a60d9c5dc4d434e9c797c38 Mon Sep 17 00:00:00 2001 From: jheaff1 Date: Wed, 5 Jan 2022 12:47:19 +0000 Subject: [PATCH] fix compile_pip_requirements on windows PR #529 updated `click` to a version which depends on `colorama` on Windows. This broke `compile_pip_requirements` on windows. This commit resolves the issue by adding colorama as a dependency. --- python/pip_install/repositories.bzl | 5 +++++ python/pip_install/requirements.bzl | 1 + 2 files changed, 6 insertions(+) diff --git a/python/pip_install/repositories.bzl b/python/pip_install/repositories.bzl index a2ed84d9e8..942711d0c2 100644 --- a/python/pip_install/repositories.bzl +++ b/python/pip_install/repositories.bzl @@ -12,6 +12,11 @@ _RULE_DEPS = [ "https://files.pythonhosted.org/packages/76/0a/b6c5f311e32aeb3b406e03c079ade51e905ea630fc19d1262a46249c1c86/click-8.0.1-py3-none-any.whl", "fba402a4a47334742d782209a7c79bc448911afe1149d07bdabdf480b3e2f4b6", ), + ( + "pypi__colorama", + "https://files.pythonhosted.org/packages/44/98/5b86278fbbf250d239ae0ecb724f8572af1c91f4a11edf4d36a206189440/colorama-0.4.4-py2.py3-none-any.whl", + "9f47eda37229f68eee03b24b9748937c7dc3868f906e8ba69fbcbdd3bc5dc3e2", + ), ( "pypi__pip", "https://files.pythonhosted.org/packages/47/ca/f0d790b6e18b3a6f3bd5e80c2ee4edbb5807286c21cdd0862ca933f751dd/pip-21.1.3-py3-none-any.whl", diff --git a/python/pip_install/requirements.bzl b/python/pip_install/requirements.bzl index b040ebd1a0..0bc382e2b0 100644 --- a/python/pip_install/requirements.bzl +++ b/python/pip_install/requirements.bzl @@ -59,6 +59,7 @@ def compile_pip_requirements( deps = [ requirement("click"), + requirement("colorama"), requirement("pip"), requirement("pip_tools"), requirement("setuptools"),