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

x11: clipboard; wayland: unmarshaling, protocol #27

Merged
merged 12 commits into from
Oct 27, 2021

Conversation

levovix0
Copy link
Contributor

No description provided.

@guzba
Copy link
Collaborator

guzba commented Oct 27, 2021

Hello, great to see more work here!

I believe Windows is at a good enough point for me get Mac caught up again and finally focus on Linux.

I order to make the Linux implementation learning and review more manageable for me, and to ensure quick review and turn-around, I am going to set up a fresh empty x11 platform file with just the API stubs. From there, I'd like to work with you to discuss and bring over the implementations in parts. This will be slower but I really care that I understand and could maintain the implementation going forward.

It should go pretty quick once we are rolling, but I do ask for your patience at first while we work through the first few commits.

Merging this so we have it!

@guzba guzba merged commit bf6a1d2 into treeform:master Oct 27, 2021
@guzba
Copy link
Collaborator

guzba commented Oct 27, 2021

Going to talk through the stub idea with treeform #28
Assuming we merge that, I would like to start with just opening a basic window with an opengl context. You left some notes in the current x11 impl I want to start by figuring out.

@guzba
Copy link
Collaborator

guzba commented Oct 27, 2021

(Obviously you are not obligated to do any of this to be clear, but your contributions are appreciated and it would be great to work with you to really understand the linux code.)

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