-
-
Notifications
You must be signed in to change notification settings - Fork 189
Add gdk4-win32 #1103
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 gdk4-win32 #1103
Conversation
I test the build with the pending Fedora package:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would also look at updating the windows job to build gdk-win so we are sure it works. I will look into reserving the name on crates.io ;)
a87455c
to
f971c46
Compare
Would you mind changing the windows CI to re-build everytime the gdk-win32 is changed, see https://github.com/gtk-rs/gtk4-rs/blob/master/.github/workflows/windows-msvc.yml#L6-L18, as well as adding a step here https://github.com/gtk-rs/gtk4-rs/blob/master/.github/workflows/windows-msvc.yml#L91-L95 for build, test & clippy gdk-win32 |
I forgot that we should do the same for the docs job, especially the part in https://github.com/gtk-rs/gtk4-rs/blob/master/.github/workflows/docs.yml#L70 otherwise we won't get any gdk-win32 docs. |
1f225f2
to
24db6c1
Compare
One last CI change is needed: https://github.com/gtk-rs/gtk4-rs/blob/master/.github/workflows/CI.yml#L5-L23 & https://github.com/gtk-rs/gtk4-rs/blob/master/.github/workflows/CI.yml#L145-L151 and it is ready to go |
Sorry for always bringing yet another change to do, but the main readme at https://github.com/gtk-rs/gtk4-rs#gtk4-rs- mentions the various supported gdk backends, would it be possible to add a line for gdk win32? thanks |
f8ec350
to
12b92d9
Compare
Signed-off-by: Marc-André Lureau <[email protected]>
Signed-off-by: Marc-André Lureau <[email protected]>
Signed-off-by: Marc-André Lureau <[email protected]>
Signed-off-by: Marc-André Lureau <[email protected]>
Signed-off-by: Marc-André Lureau <[email protected]>
I could use some help, it currently fails with:
and