Skip to content

update: add native windows compilation - #6

Closed
ToddyTheNoobDud wants to merge 2 commits into
PerformanC:improve/native-sendingfrom
ToddyTheNoobDud:improve/native-sending
Closed

update: add native windows compilation#6
ToddyTheNoobDud wants to merge 2 commits into
PerformanC:improve/native-sendingfrom
ToddyTheNoobDud:improve/native-sending

Conversation

@ToddyTheNoobDud

Copy link
Copy Markdown
Member

Changes

Allows building from source in windows + steps

Also fixes headerSize being an constant (the value can change later, https://github.com/ToddyTheNoobDud/voice/blob/ccfd80386c84e747d6f6c9cc3ab4af125bbf8396/index.js#L1160)

Why

Discussed internally.

Checkmarks

  • The modified functions have been tested.
  • Used the same indentation as the rest of the project.
  • Updated documentation (changelog).
  • Is a documented feature by Discord.

Additional information

ToddyTheNoobDud and others added 2 commits July 19, 2026 10:23
Find libsodium is to find it from vcpkg or the MSCV install (which is
the one i use), and sets it to the env so it can compile. Installation
is documented in readme.

Yet, the audio sending itself does not work !! will hopefully fix in
future commits
@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown

All Contributors have signed the CLA. The PR is now allowed to be merged.
Posted by the CLA Assistant Lite bot.

@ToddyTheNoobDud

Copy link
Copy Markdown
Member Author

I have read the CLA Document and I hereby sign the CLA

performanc-bot added a commit to PerformanC/CLA-Signatures that referenced this pull request Aug 2, 2026
@ToddyTheNoobDud

Copy link
Copy Markdown
Member Author

Windows now compiles sucessfully from this commit: 5018fb1

@github-actions github-actions Bot locked and limited conversation to collaborators Aug 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant