Skip to content
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

Revert "Prefix the entire setup.py chroot. (#12359)" #12370

Merged
merged 1 commit into from
Jul 17, 2021

Conversation

Eric-Arellano
Copy link
Contributor

This was causing CI to fail with:

Exception: String("Cannot strip prefix chroot/dist from root directory (Digest with hash Fingerprint<0e3f69284c886c8b9b330122c22ed47e469e2f9020c9938fdf6d15c24471aeae>) - root directory didn't contain a directory named chroot but did contain directory named: dist")

[ci skip-rust]

[ci skip-build-wheels]

This reverts commit 36026df.

[ci skip-rust]

[ci skip-build-wheels]
@Eric-Arellano Eric-Arellano merged commit 9f5ff78 into pantsbuild:main Jul 17, 2021
@Eric-Arellano Eric-Arellano deleted the revert-setup-py branch July 17, 2021 00:23
benjyw added a commit to benjyw/pants that referenced this pull request Jul 17, 2021
@wisechengyi wisechengyi mentioned this pull request Jul 17, 2021
wisechengyi added a commit that referenced this pull request Jul 17, 2021
### Internal

* [internal] Manually fix Black lockfile to handle interpreter constraints ([#12366](#12366))
* Revert "Prefix the entire setup.py chroot. (#12359)" ([#12370](#12370))
* [internal] Cache native client binary in CI ([#12355](#12355))
* Prefix the entire setup.py chroot. ([#12359](#12359))
* [internal] Fix AWS CLI breaking due to Python 2 usage ([#12364](#12364))
* [Internal] Add `git_url()` helper to `docutil.py` ([#12352](#12352))
* [Internal] Refactor how `PythonToolBase` exposes requirements and interpreter constraints ([#12356](#12356))
* [internal] Add experimental per-tool lockfiles with Docformatter as an example ([#12346](#12346))
* Remove Pants's dpeendency on `requests` ([#12348](#12348))
* [internal] Remove `TwoStepPex` abstraction ([#12343](#12343))
* Add psycopg2-binary to default module mapping ([#12339](#12339))
* [internal] Upgrade toolchain pants plugin to 0.13.1 ([#12338](#12338))
* Add an API to coarsen/partition Targets by their cycles ([#12251](#12251))
* Prepare 2.5.1 ([#12329](#12329))
* Bootstrap fewer JVM versions in Coursier/javac tests to hopefully reduce CI flakiness ([#12325](#12325))
* Native client respects `--concurrent`. ([#12324](#12324))
* Add client lib tests. ([#12322](#12322))
* Special case enum option parse failures. ([#12281](#12281))
benjyw added a commit to benjyw/pants that referenced this pull request Jul 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants