Skip to content

Added windows support for libwhisper bindings#92

Merged
djthorpe merged 1 commit intomainfrom
machayka/0125/windows
Jan 26, 2026
Merged

Added windows support for libwhisper bindings#92
djthorpe merged 1 commit intomainfrom
machayka/0125/windows

Conversation

@djthorpe
Copy link
Copy Markdown
Member

This pull request adds Windows support to the whisper package by introducing Windows-specific build and pkg-config directives in sys/whisper/generate.go.

Platform support:

  • Added Windows-specific #cgo directive for linking against libwhisper-windows in generate.go.
  • Added a go:generate command for generating Windows-specific pkg-config files and linking instructions in generate.go.

Thanks to @machayka

@djthorpe djthorpe self-assigned this Jan 25, 2026
@djthorpe djthorpe added the enhancement New feature or request label Jan 25, 2026
Copilot AI review requested due to automatic review settings January 25, 2026 11:57
@djthorpe djthorpe mentioned this pull request Jan 25, 2026
@djthorpe djthorpe linked an issue Jan 25, 2026 that may be closed by this pull request
Copy link
Copy Markdown

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

Adds Windows support for the sys/whisper CGO bindings by introducing Windows-specific pkg-config integration and generation of a corresponding .pc file.

Changes:

  • Added a Windows-specific #cgo windows pkg-config: libwhisper-windows directive.
  • Added a go:generate command to produce libwhisper-windows.pc with Windows linker flags.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@djthorpe djthorpe merged commit a48fb72 into main Jan 26, 2026
6 checks passed
@djthorpe djthorpe deleted the machayka/0125/windows branch January 26, 2026 12:01
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.

Develop in Windows OS

3 participants