Skip to content

Conversation

@khustup2
Copy link
Contributor

@khustup2 khustup2 commented Aug 9, 2025

No description provided.

Copilot AI review requested due to automatic review settings August 9, 2025 23:11

This comment was marked as outdated.

@khustup2 khustup2 force-pushed the wheel-build-fix branch 3 times, most recently from cd2c7f0 to 1a8d36d Compare August 10, 2025 10:24
@khustup2 khustup2 requested a review from Copilot August 10, 2025 11:16

This comment was marked as outdated.

khustup2 and others added 4 commits August 10, 2025 11:19
@khustup2 khustup2 force-pushed the wheel-build-fix branch 2 times, most recently from 8f67c3e to def3ced Compare August 10, 2025 11:21
@khustup2 khustup2 requested a review from Copilot August 10, 2025 11:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes wheel builds by implementing proper architecture-specific handling for cross-compilation and updating build configurations. The changes enable successful wheel builds on different platforms and architectures by correctly setting up vcpkg triplets, FFmpeg installations, and build environments.

  • Adds vcpkg triplet configuration and cross-compilation setup for CMake
  • Implements architecture-specific FFmpeg installations to avoid conflicts during cross-compilation
  • Updates GitHub Actions workflow to support different runner types and properly configure platform-specific builds

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
cpp/CMakeLists.txt Adds vcpkg triplet setup and cross-compilation verification
cmake/modules/findFFmpeg.cmake Implements architecture-specific FFmpeg build directories and configuration
.github/workflows/wheel-build.yaml Updates workflow to support flexible runners and platform-specific wheel builds

@khustup2 khustup2 merged commit d43bbfc into main Aug 14, 2025
17 of 22 checks passed
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.

3 participants