Skip to content
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

Add headers required by XAudio2_7 to externals. #858

Merged
merged 1 commit into from Aug 23, 2014
Merged

Add headers required by XAudio2_7 to externals. #858

merged 1 commit into from Aug 23, 2014

Conversation

shuffle2
Copy link
Contributor

This makes all builds audio compatible with downlevel Windows and removes
the need for developers to install the June 2010 DirectX SDK.

@delroth
Copy link
Member

delroth commented Aug 23, 2014

@shuffle2: This comment grants you the permission to merge this pull request whenever you think it is ready. After addressing the remaining comments, click this link to merge.


Not a big fan from the licensing standpoint, but meh. Other projects seem to be doing the same thing, and these headers are part of a freely available SDK. I'm really not sure how licensing works here, usually I would try to stay on the safe side, but here both sides seem pretty safe.

LGTM, @dolphin-emu-bot allowmerge.

This makes all builds' audio compatible with downlevel Windows and removes
the need for developers to install the June 2010 DirectX SDK.
shuffle2 added a commit that referenced this pull request Aug 23, 2014
Add headers required by XAudio2_7 to externals.
@shuffle2 shuffle2 merged commit 8bb3fa5 into dolphin-emu:master Aug 23, 2014
@shuffle2 shuffle2 deleted the dx-old branch August 23, 2014 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants