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

Implement glutin_interface on this crate. #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

goddessfreya
Copy link

The formatting changes are all contained in the fmt commit, where I only ran cargo fmt --all due to being too lazy to manually format the prior commits. I can revert that commit if requested and just manually format stuff.

Signed-off-by: Freya Gentz <[email protected]>
Signed-off-by: Freya Gentz <[email protected]>
Signed-off-by: Freya Gentz <[email protected]>
Signed-off-by: Freya Gentz <[email protected]>
Signed-off-by: Freya Gentz <[email protected]>
Signed-off-by: Freya Gentz <[email protected]>
Signed-off-by: Freya Gentz <[email protected]>
Signed-off-by: Freya Gentz <[email protected]>
@goddessfreya goddessfreya changed the title [Draft] Implement glutin_interface on this crate. Implement glutin_interface on this crate. Feb 17, 2020
bitflags = "1.2.1"
wayland-client = { version = "0.23", optional = true }
drm = { version = "0.3.4", optional = true }
glutin_interface = { version = "0.1.0", path = "../glutin_interface", optional = true }
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Drakulix These two lines will be fixed when the crates are published.

@Drakulix
Copy link
Member

Looks good.

Could you give me a quick rundown about what glutin_interface is and why it exists? Seems that you are doing quite the refactor in winit.

What about I format the master branch and you rebase your commits on top? Would that work for you?

Otherwise I would like to see the commits updating the dependencies and everything related to glutin_interface squashed into two separate commits (with some more descriptive contents), when this PR is ready. (This is will likely have to wait until glutin_interface is published.)

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.

2 participants