Skip to content

Migrating to VCPKG to support#6

Merged
guzman109 merged 18 commits intoICICLE-ai:mainfrom
guzman109:main
Feb 27, 2026
Merged

Migrating to VCPKG to support#6
guzman109 merged 18 commits intoICICLE-ai:mainfrom
guzman109:main

Conversation

@guzman109
Copy link
Contributor

Build CI was failing due to Conan needing to install Xorg and other Desktop Linux packages as dependencies for speech-to-text that is bundled with AWS-SDK-CPP.

We're using Alma Linux containers (manylinux) to support packaging the python module and shared libraries on Linux and different architectures.

guzman109 and others added 18 commits February 26, 2026 15:27
Refactor GitHub Actions workflow for building and publishing Python package. Removed unnecessary steps and streamlined Python setup with 'uv'.
Added additional development libraries for building.
Replace Conan package manager with vcpkg for C++ dependency management
in the GitHub Actions build workflow.

Changes:
- Remove Conan install, profile detect, and toolchain discovery steps
- Remove 40+ X11/ALSA apt-get packages required by Conan's transitive
deps
- Add vcpkg bootstrap and binary cache steps
- Linux system deps reduced to: cmake, ninja-build (yum in manylinux)
- Retain manylinux_2_28 containers for both x86_64 and aarch64
- vcpkg cache keyed on lib/vcpkg.json per OS/arch

Motivation: Conan pulled transitive xorg/system and Perl dependencies
for a headless cloud storage library, causing build failures in
manylinux containers. vcpkg builds only declared dependencies with
native CMake toolchain integration.
Added curl, zip, unzip, and tar to the Linux dependencies.
Fix the command to discover h5py HDF5 location in the CI workflow.
Added a debug step to check the contents of the virtual environment's bin directory.
Removed debug step for checking virtual environment and updated h5py HDF5 discovery command.
…ions instead.

Issues with github actions trying to build openssl from bottom up.
@guzman109 guzman109 merged commit 2fd5089 into ICICLE-ai:main Feb 27, 2026
18 of 19 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.

1 participant