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

Rename //rules_python package to //packaging #228

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 14, 2019

Conversation

brandjon
Copy link
Contributor

This avoids a name clash between the package name and the repo name, which
prevented whl_test from running in the Bazel federation's CI.

Regenerated the par files.

Tested that this still works in environments where the system Python has a
library named "packaging" installed, by creating a virtualenv, pip installing
"packaging", and running bazel test //examples/....

Fixes #227.

@brandjon brandjon requested a review from fweikert August 14, 2019 16:44
@brandjon brandjon requested a review from lberki as a code owner August 14, 2019 16:44
@brandjon brandjon self-assigned this Aug 14, 2019
This avoids a name clash between the package name and the repo name, which
prevented whl_test from running in the Bazel federation's CI.

Regenerated the par files.

Tested that this still works in environments where the system Python has a
library named "packaging" installed, by creating a virtualenv, pip installing
"packaging", and running `bazel test //examples/...`.

Fixes bazel-contrib#227.
@fweikert
Copy link
Contributor

Nice, thank you very much!

@brandjon brandjon merged commit 9d68f24 into bazel-contrib:master Aug 14, 2019
@brandjon brandjon deleted the mv-test branch August 14, 2019 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

whl_test fails when invoked from outside main repo
3 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.