Skip to content

build: replace shared library with standalone cuevalidator binary #1096

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

behnazh-w
Copy link
Member

Summary

This PR replaces the Go shared library previously used via C-bindings in Python with a standalone binary for the cuevalidator component. The binary can now be invoked as a subprocess, simplifying integration and improving portability.

Description of changes

  • cuevalidator is now compiled as a standalone Go binary instead of a shared library.
  • Removed dependency on ctypes/C-bindings in Python code. Python now calls the cuevalidator binary using subprocess.
  • This change removes the need for platform-specific shared library builds, reducing complexity in packaging and deployment.
  • Updated the Makefile and instructions to build and distribute the binary as part of the project tooling.

@behnazh-w behnazh-w requested a review from tromai as a code owner June 3, 2025 06:10
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jun 3, 2025
@behnazh-w behnazh-w force-pushed the behnazh/cue-execution branch from 48daab3 to 4693061 Compare June 5, 2025 06:38
@behnazh-w behnazh-w force-pushed the behnazh/cue-execution branch from 4693061 to c754024 Compare June 5, 2025 08:35
Copy link
Member

@tromai tromai left a comment

Choose a reason for hiding this comment

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

I have left some minor questions about the Pull Request.

@behnazh-w behnazh-w merged commit 06fd636 into main Jun 6, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants