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

Bump gtk #1845

Merged
merged 6 commits into from
Aug 21, 2021
Merged

Bump gtk #1845

merged 6 commits into from
Aug 21, 2021

Conversation

JAicewizard
Copy link
Contributor

@JAicewizard JAicewizard commented Jul 1, 2021

Build time hasn't changed:

before:
Finished dev [unoptimized + debuginfo] target(s) in 1m 02s
after:
Finished dev [unoptimized + debuginfo] target(s) in 1m 06s

There are a couple less dependencies, but those didn't seem to have any effect on total build times. This also removed direct dependencies from druid-shell. This will reduce maintenance effort a bit.

@@ -15,7 +15,7 @@ default-target = "x86_64-pc-windows-msvc"

[features]
default = ["gtk"]
gtk = ["gio", "gdk", "gdk-sys", "glib", "glib-sys", "gtk-sys", "gtk-rs", "gdk-pixbuf"]
gtk = ["gdk-sys", "glib-sys", "gtk-sys", "gtk-rs"]

Choose a reason for hiding this comment

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

You don't need -sys neither, they are re-exported as ffi.

Copy link
Collaborator

@PoignardAzur PoignardAzur left a comment

Choose a reason for hiding this comment

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

LGTM.

Rebase and merge?

@JAicewizard
Copy link
Contributor Author

Blocked on piet version bump.

@maan2003 maan2003 added the S-blocked waits for another PR or dependency label Aug 4, 2021
@cmyr
Copy link
Member

cmyr commented Aug 20, 2021

@JAicewizard I've released piet 0.5.0-pre1, which you can use to test this out. Let me know if you have any issues?

@JAicewizard
Copy link
Contributor Author

rebased, CI seems to have issues but it compiles fine locally.

druid-shell/Cargo.toml Outdated Show resolved Hide resolved
@PoignardAzur PoignardAzur merged commit 31f4577 into linebender:master Aug 21, 2021
@xStrom xStrom removed the S-blocked waits for another PR or dependency label Jan 20, 2023
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.

6 participants