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

GFX Support Features #172

Merged
merged 3 commits into from
Apr 12, 2020
Merged

GFX Support Features #172

merged 3 commits into from
Apr 12, 2020

Conversation

zicklag
Copy link
Contributor

@zicklag zicklag commented Apr 9, 2020

This PR contains additions to surfman needed for supporting the gfx-backend-gl effort that is being worked on in gfx-rs/gfx#3151. Still a work in progress at the moment, but it is almost finished I think.

This supercedes: pcwalton#61.

The PR adds new functions for creating a native widget from a raw window handle and will also contain fixes to allow compiling on MacOS without having to disable unsupported features.

These are both features needed by gfx-backend-gl.

@zicklag zicklag force-pushed the gfx-support branch 8 times, most recently from 74b3075 to 1041594 Compare April 10, 2020 03:28
@zicklag zicklag marked this pull request as ready for review April 10, 2020 03:30
@kvark kvark requested a review from pcwalton April 10, 2020 15:08
@zicklag zicklag force-pushed the gfx-support branch 2 times, most recently from 82b962e to e951740 Compare April 10, 2020 15:51
zicklag and others added 3 commits April 10, 2020 20:20
Adds the ability to create a native widget from a raw window handle.
- Add cfg aliases to simplify feature checks and make sure that
  platform specific features will not break when enabled on
  unsupported platforms.
- Disable X11 on MacOS. It appears it was meant to be supported, but
  never actually was. Disabling it so that MacOS builds can run with
  the X11 feature ( when desired for Linux ) without breaking.
- Make the platform::unix module Linux specific ( for now? ) as Android
  and MacOS have their own modules and all unix backends aren't
  supported for Android or MacOS yet.
@zicklag
Copy link
Contributor Author

zicklag commented Apr 11, 2020

This PR now successfully works with the gfx-backend-gl create with this PR: gfx-rs/gfx#3151.

Other than needing a review, it is ready as far as I know.

@pcwalton
Copy link
Contributor

@bors-servo: r+

@bors-servo
Copy link
Contributor

📌 Commit 2925168 has been approved by pcwalton

@bors-servo
Copy link
Contributor

⌛ Testing commit 2925168 with merge 41ac1ee...

@bors-servo
Copy link
Contributor

☀️ Test successful - checks-travis
Approved by: pcwalton
Pushing 41ac1ee to master...

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.

4 participants