Skip to content

Switch to more specific (=smaller) bindings for GCP#1314

Merged
xhochy merged 10 commits intoconda-forge:mainfrom
h-vetinari:gstorage
Feb 21, 2024
Merged

Switch to more specific (=smaller) bindings for GCP#1314
xhochy merged 10 commits intoconda-forge:mainfrom
h-vetinari:gstorage

Conversation

@h-vetinari
Copy link
Member

@h-vetinari h-vetinari commented Feb 20, 2024

This needed a bunch of work on the google-cloud-cpp side (see here), but now we should hopefully be able to reap the benefits!

Closes #1305

Currently based on #1312, draft until that goes in.

@h-vetinari h-vetinari changed the title Switch to more specific (=smalle) bindings for GCP Switch to more specific (=smaller) bindings for GCP Feb 20, 2024
@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@h-vetinari
Copy link
Member Author

@conda-forge/arrow-cpp this should be ready, PTAL

CC @coryan

@h-vetinari
Copy link
Member Author

(Note: I added a couple clean-ups that aren't strictly related to the GCP bindings)

@h-vetinari h-vetinari mentioned this pull request Feb 20, 2024
5 tasks
raulcd pushed a commit to apache/arrow that referenced this pull request Feb 20, 2024
…rge script errors (#37819)" (#40150)

This reverts commit 79e49db.

#37931 should have been fixed as of pypa/setuptools-scm@056584b in setuptools_scm 8.0.4; I tested that this works in conda-forge/arrow-cpp-feedstock#1314.

CC @ AlenkaF @ raulcd 
* Closes: #37931

Authored-by: H. Vetinari <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
@xhochy xhochy merged commit 217ca69 into conda-forge:main Feb 21, 2024
@h-vetinari h-vetinari deleted the gstorage branch February 21, 2024 08:25
@h-vetinari
Copy link
Member Author

Please don't squash-merge, these commits were intentionally well-separated.

@xhochy
Copy link
Member

xhochy commented Feb 21, 2024

Please don't squash-merge, these commits were intentionally well-separated.

This wasn't intentional. I just clicked on the default merge button.

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.

Consider switching to libgoogle-cloud-storage from google-cloud-cpp.

2 participants