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

VideoCommon{Backends}: Remove unnecessary wxWidgets references. #1455

Merged
merged 1 commit into from Nov 2, 2014

Conversation

lioncash
Copy link
Member

Also EmuWindow doesn't even exist anymore.

Turns out InputCommon actually relied on wx linking in the correct OSX libraries, so it removes even more coupling in that regard.

Now building VideoCommon and the Software and OGL backends isn't blocked by the compilation of wx (at least on OSX. *nix just ignores building the static version if wx is already installed on the system).

@lioncash lioncash changed the title VideoCommon: Remove unnecessary wxWidgets check from CMakeLists VideoCommon{Backends}: Remove unnecessary wxWidgets checks from the CMakeLists Oct 31, 2014
@lioncash
Copy link
Member Author

All good to merge now

@phire
Copy link
Member

phire commented Nov 1, 2014

@dolphin-emu-bot rebuild

@lioncash lioncash changed the title VideoCommon{Backends}: Remove unnecessary wxWidgets checks from the CMakeLists VideoCommon{Backends}: Remove unnecessary wxWidgets references. Nov 1, 2014
@lioncash lioncash force-pushed the dammit-wxwidgeeeeeets branch 3 times, most recently from a4131fe to 23ca707 Compare November 1, 2014 23:16
EmuWindow doesn't even exist anymore. wxWidgets is also decoupled from the backends.
@Sonicadvance1
Copy link
Contributor

<3 u.
LGTM.

lioncash added a commit that referenced this pull request Nov 2, 2014
VideoCommon{Backends}: Remove unnecessary wxWidgets references.
@lioncash lioncash merged commit b6b7b82 into dolphin-emu:master Nov 2, 2014
@lioncash lioncash deleted the dammit-wxwidgeeeeeets branch November 2, 2014 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants