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

Require scie-pants 0.9.2 or newer, for new distribution model (Cherry-pick of #19654) #19694

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

WorkerPants
Copy link
Member

This updates Pants to require and check for the scie-pants launcher binary version: we need version 0.9 or later, to have support for the new distribution model. This syncs with pantsbuild/scie-pants#246 (which will become scie-pants 0.9.2 / 0.9.3), which sets the SCIE_PANTS_VERSION environment variable when running pants.

This will be cherry-picked back as far as we support, to help users upgrade their launcher earlier.

Fixes #19600

This updates Pants to require and check for the scie-pants launcher
binary version: we need version 0.9 or earlier, to have support for the
new distribution model. This syncs with
pantsbuild/scie-pants#246 (which will become
scie-pants 0.9.2 / 0.9.3), which sets the `SCIE_PANTS_VERSION`
environment variable when running pants.

This will be cherry-picked back as far as we support, to help users
upgrade their launcher earlier.

Fixes #19600
@huonw huonw merged commit eb9f305 into 2.18.x Aug 29, 2023
24 checks passed
@huonw huonw deleted the cherry-pick-19654-to-2.18.x branch August 29, 2023 22:39
huonw added a commit that referenced this pull request Jan 15, 2024
This replaces the over-complicated address URI scheme-based look-up from
#19827 for deciding on which provider to use with just a normal option
`[GLOBAL].remote_provider`. This defaults to `reapi`, to preserve the
current (stable) behaviour.

That is, now, one can set:

```toml
[GLOBAL]
remote_provider = "experimental-file"
remote_store_address = "file:///..."
```

Or similar.

Another item in #19694 

The commits are somewhat usefully broken up for review.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants