Skip to content

Conversation

@oleg-nenashev
Copy link
Collaborator

Initial work for or #2 . It will not enable publishing yet, but at least ther ewill be a stable experimental build

@oleg-nenashev oleg-nenashev added the enhancement New feature or request label Oct 20, 2025
@oleg-nenashev
Copy link
Collaborator Author

So I got stuck with the Port file pulling the pkg-config file from the root repo instead of testcontainers-c. I am not 100% sure how to make VCPKG to pull it from the right location, but maybe running the root build is a bogus approach and two CMake calls should be called for the bridge and the C Library

vscode@271be3dfe8dd:/workspaces/testcontainers-native$ cat /usr/local/vcpkg/buildtrees/testcontainers-c/install-arm64-linux-dbg-err.log
CMake Error at testcontainers-c/cmake_install.cmake:46 (file):
  file INSTALL cannot find
  "/usr/local/vcpkg/buildtrees/testcontainers-c/arm64-linux-dbg/testcontainers-c.pc":
  No such file or directory.
Call Stack (most recent call first):
  cmake_install.cmake:52 (include)

Would appreciate sanity check @seffradev

"license": "MIT",
"dependencies": [
{
"name": "testcontainers-bridge",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That part does not work in vcpkg runs yet, even if local installation is fine.
Likely, the port files are messed up somehow

@oleg-nenashev oleg-nenashev mentioned this pull request Nov 12, 2025
3 tasks
IMPORTED_LOCATION ${CMAKE_CURRENT_BINARY_DIR}/${SHIM_TARGET_LIB}
INTERFACE_INCLUDE_DIRECTORIES ${CMAKE_CURRENT_BINARY_DIR})

# Install library and header files
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I am still not sure about making testcontainers-bridge a separate deliverable, but it seems to be the easiest approach if we want to preserve the project structure. When we get to 1.0, the non-committment on the API should be explicitly documented

Copy link
Contributor

Choose a reason for hiding this comment

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

My vision was that testcontainers-bridge would be statically linked (with -fPIC) into testcontainers-c, which would in turn be two targets, a static library, and a dynamic library.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah. That's easy with CMake. I just need to figure our how to do vcpkg ports in such a case, without breaking the dependency graph. Idea of having 2 packages is very debatable, maysbe I will just do twqo modes in the terstcontainers-c CMake

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants