Skip to content

Add cgo script support - byo#146

Open
alfredosa wants to merge 5 commits into
mainfrom
cgo-script-support
Open

Add cgo script support - byo#146
alfredosa wants to merge 5 commits into
mainfrom
cgo-script-support

Conversation

@alfredosa

@alfredosa alfredosa commented Dec 16, 2025

Copy link
Copy Markdown

this is the pr that adds support for:

  • Support for a cgo dependencies installation
  • Support to specify a specific language instead of autobuild

There are some interesting caveats. Both are required for cgo because for example: Enri and Autobuild detect the project to be a cpp project. which was causing issues when trying to run code scanning for go.

Dependencies for cgo are quite specific to the project.

@alfredosa alfredosa requested a review from a team as a code owner December 16, 2025 09:53
type: string
required: false
cgo-setup-script:
description: 'Path to CGO dependencies setup script (relative to repo root)'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is there not a way to kinda unify this script? The way it is setup now is that it can inject anything which is a bit of a security risk. Some form of serialization at least would be good (or an issue to fix this)

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.

2 participants