Skip to content

Commit 7a72572

Browse files
committed
Remove --no-build-isolation from pip install command
1 parent 7c5e7cf commit 7a72572

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,10 @@ A brief description of the categories of changes:
3333
([2310](https://github.com/bazelbuild/rules_python/issues/2310)).
3434
* (publish) The dependencies have been updated to the latest available versions
3535
for the `twine` publishing rule.
36+
* (whl_library) Remove `--no-build-isolation` to allow non-hermetic sdist builds
37+
by default. Users wishing to keep this argument and to enforce more hermetic
38+
builds can do so by passing the argument in
39+
[`pip.parse#extra_pip_args`](https://rules-python.readthedocs.io/en/latest/api/rules_python/python/extensions/pip.html#pip.parse.extra_pip_args)
3640

3741
{#v0-0-0-fixed}
3842
### Fixed

python/private/pypi/whl_library.bzl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,7 @@ def _whl_library_impl(rctx):
230230
# and, allow getting build dependencies from PYTHONPATH, which we
231231
# setup in this repository rule, but still download any necessary
232232
# build deps from PyPI (e.g. `flit_core`) if they are missing.
233-
extra_pip_args.extend(["--no-build-isolation", "--find-links", "."])
233+
extra_pip_args.extend(["--find-links", "."])
234234

235235
args = _parse_optional_attrs(rctx, args, extra_pip_args)
236236

0 commit comments

Comments
 (0)