https://pantsbuild.org/ logo
Join Slack
Powered by
Another question: Can I use a pants version from a...
# general
f
Another question: Can I use a pants version from a certain commit in a branch by providing the sha256 sum of it? I know how to run it from sources, but then everyone in the team needs to clone the repo, checkout the right commit and use the 
pants_from_sources.sh
script.
c
this used to be by 
PANTS_SHA
but that is deprecated....
is there any reason to not use one of the released versions? which commit do you want to run from..?
f
I'd like to use the features of my PR already https://github.com/pantsbuild/pants/pull/20374
but it's still in discussion and I need to find the time to finish it
c
if you fix the lint issue, I think it will build wheels, and you could then use the 
PANTS_SHA
for now...
f
I rebased the PR and fixed the lint issues, can you approve the workflow?
✅ 1
May I ask why PANTS_SHA is deprecated? Too many resources publishing all the artifacts? For me it would also be an option if pants would build itself locally from the given SHA. Then the load would be on the user's machine.
b
PANTS_SHA required the Pantsbuild project to maintain an archive of all built artifacts forever, which is hard for a low-budget community non-profit (or have optimisations to manage the cost, which is hard for a volunteer-driven project: who wants to do that? 😀 ) Thus, it isn’t supported for recent releases, and I’m not sure it was ever supported for every PR (I don’t recall for sure, though). https://github.com/pantsbuild/scie-pants/issues/234 is a feature request that would help with this.
(The assumption is that we can now do releases easily enough that new code landing in main will be released soon after merging, but this obviously doesn’t help with pending PRs)
f
Thanks for the info. I think an SHA configuration would be good that auto-builds pants locally and uses that artifact.
CI does not build wheels for my PR https://github.com/pantsbuild/pants/pull/20374/checks, the steps are skipped based on the Classify changes output.
😢 1
b
Yeah, that would be great! Do you think you would be interested in implementing it? If so, let us know here or on the issue above and I’m sure someone can provide guidance etc
2 Views
Open in Slack
PreviousNext
Powered by