Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit f99ea2a

Browse filesBrowse files
jheaff1alexeagle
andauthored
fix compile_pip_requirements on windows (bazel-contrib#595)
PR bazel-contrib#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. Co-authored-by: Alex Eagle <eagle@post.harvard.edu>
1 parent 28cfb11 commit f99ea2a
Copy full SHA for f99ea2a

File tree

2 files changed

+6
-0
lines changed
Filter options

2 files changed

+6
-0
lines changed

‎python/pip_install/repositories.bzl

Copy file name to clipboardExpand all lines: python/pip_install/repositories.bzl
+5Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,11 @@ _RULE_DEPS = [
1212
"https://files.pythonhosted.org/packages/76/0a/b6c5f311e32aeb3b406e03c079ade51e905ea630fc19d1262a46249c1c86/click-8.0.1-py3-none-any.whl",
1313
"fba402a4a47334742d782209a7c79bc448911afe1149d07bdabdf480b3e2f4b6",
1414
),
15+
(
16+
"pypi__colorama",
17+
"https://files.pythonhosted.org/packages/44/98/5b86278fbbf250d239ae0ecb724f8572af1c91f4a11edf4d36a206189440/colorama-0.4.4-py2.py3-none-any.whl",
18+
"9f47eda37229f68eee03b24b9748937c7dc3868f906e8ba69fbcbdd3bc5dc3e2",
19+
),
1520
(
1621
"pypi__pip",
1722
"https://files.pythonhosted.org/packages/47/ca/f0d790b6e18b3a6f3bd5e80c2ee4edbb5807286c21cdd0862ca933f751dd/pip-21.1.3-py3-none-any.whl",

‎python/pip_install/requirements.bzl

Copy file name to clipboardExpand all lines: python/pip_install/requirements.bzl
+1Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ def compile_pip_requirements(
5959

6060
deps = [
6161
requirement("click"),
62+
requirement("colorama"),
6263
requirement("pip"),
6364
requirement("pip_tools"),
6465
requirement("setuptools"),

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.