Uh oh!
There was an error while loading. Please reload this page.
ARROW-14171: [C++][Python][Packaging] Upgrade VCPKG version and add google-cloud-cpp dependency - #11569
ARROW-14171: [C++][Python][Packaging] Upgrade VCPKG version and add google-cloud-cpp dependency#11569coryan wants to merge 33 commits into
Conversation
kou
commented
Oct 28, 2021
@github-actions crossbow submit -g nightly |
Revision: 7f69cf410b42194a7fffb4687835a08c4bf3c3f0 Submitted crossbow builds: ursacomputing/crossbow @ actions-1059 |
coryan
commented
Nov 2, 2021
Ping? Is there something else I need to do here? |
kou
commented
Nov 4, 2021
It seems that this is a problem. https://github.com/ursacomputing/crossbow/runs/4039103923?check_suite_focus=true#step:6:1189 |
I probably do not understand how these are used, probably something for later.
coryan
commented
Nov 4, 2021
I see. The code checks out version Line 75 in faf8b82 That version does not include the How do you feel about pinning vcpkg to a specific SHA, but not to a tag? |
kou
commented
Nov 4, 2021
kszucs
commented
Nov 4, 2021
We used to pin specific SHAs beforehand, so it's fine. |
coryan
commented
Nov 4, 2021
Thanks, I have updated this PR accordingly. I also sent ursacomputing/crossbow#6 for review, I am not sure which one should be merged first, please let me know if I should help with that. |
Thanks! Since it affects the build cache we should wait for the 6.0.1 patch release with both of the PRs. (technically we could already merge the crossbow PR because the vcpkg version is included in the cache key, but the github actions cache eviction policy may remove the previous artifacts) |
kszucs
commented
Nov 22, 2021
Merged the crossbow PR. I'm going to rebuild and push the windows docker images for the windows wheels then we can submit the crossbow jobs here. |
kszucs
commented
Nov 23, 2021
After fixing the It has been already resolved upstream, so I pinned the HEAD commit. |
kszucs
commented
Nov 24, 2021
@github-actions crossbow submit -g wheel |
Revision: 47e8935 Submitted crossbow builds: ursacomputing/crossbow @ actions-1170 |
New changes:
Still, We have several problems here:
|
kszucs
commented
Nov 26, 2021
@github-actions crossbow submit wheel-macos-big-sur-*-universal2 |
Revision: c9bfce9 Submitted crossbow builds: ursacomputing/crossbow @ actions-1201
|
kszucs
commented
Nov 26, 2021
@github-actions crossbow submit wheel-macos-big-sur-cp39-universal2 |
kszucs
commented
Nov 26, 2021
@github-actions crossbow submit -g wheel |
Revision: f8c7af4 Submitted crossbow builds: ursacomputing/crossbow @ actions-1202
|
Revision: f8c7af4 Submitted crossbow builds: ursacomputing/crossbow @ actions-1203 |
kszucs
commented
Nov 26, 2021
@github-actions crossbow submit wheel-macos-big-sur-cp39-universal2 |
Revision: 0662793 Submitted crossbow builds: ursacomputing/crossbow @ actions-1205
|
kszucs
commented
Nov 26, 2021
@github-actions crossbow submit wheel-macos-big-sur-cp39-universal2 |
Revision: 283d7a8 Submitted crossbow builds: ursacomputing/crossbow @ actions-1206
|
kszucs
commented
Nov 26, 2021
@github-actions crossbow submit -g wheel |
Revision: 283d7a8 Submitted crossbow builds: ursacomputing/crossbow @ actions-1207 |
coryan
commented
Nov 26, 2021
If you mean in Arrow, then no, they do not exist yet. I had no plans to create them either. |
pitrou
commented
Nov 26, 2021
That's something that other contributors can do :-) |
kszucs
commented
Nov 29, 2021
Added a note about enabling the GCS feature in the wheels for https://issues.apache.org/jira/browse/ARROW-14892 @pitrou this should be ready for review then |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
| vcpkg install --clean-after-build \ | ||
| abseil \ | ||
| aws-sdk-cpp[config,cognito-identity,core,identity-management,s3,sts,transfer] \ | ||
| boost-filesystem \ |
There was a problem hiding this comment.
Hmm, do we still need boost-filesystem here? AFAIR the dependency is only for unit tests.
There was a problem hiding this comment.
Going to create a follow-up to remove it.
There was a problem hiding this comment.
Benchmark runs are scheduled for baseline = 736a69e and contender = 2233ac5. 2233ac5 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
No description provided.