Navigation Menu

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

Move gio::compile_resources to its own crate #670

Merged
merged 1 commit into from May 7, 2022
Merged

Conversation

RealKC
Copy link
Contributor

@RealKC RealKC commented May 4, 2022

Closes #574


Some things to consider:

  • naming: Should we name this gio-build-helpers, glib-build-helpers or something else entirely? This is (to me) made confusing by the fact that the executable is named glib-compile-resources but the resources machinery lives in Gio
  • versioning: Should this start from 0.1.0 or be in sync with the other crates? I don't think this would need version bumps at the same time as the other crates do, so I imagine it would end up desyncing sooner than later anyway

NOTE: If this PR is alright, I'll send PRs to gtk4-rs and gtk3-rs to move the examples to this crate

@RealKC
Copy link
Contributor Author

RealKC commented May 4, 2022

I think I need to update the CI workflows too? Not sure what I need to do exactly though

@RealKC RealKC closed this May 4, 2022
@RealKC RealKC reopened this May 4, 2022
@sdroege
Copy link
Member

sdroege commented May 4, 2022

This will either have to be moved to a separate crate or be removed completely, but keeping it inside the gio crate does not make much sense.

@sdroege
Copy link
Member

sdroege commented May 5, 2022

@Hofer-Julian mentioned that the book is using this before introducing meson, so it shouldn't go away at least

gio-build-helpers/LICENSE Outdated Show resolved Hide resolved
@RealKC RealKC force-pushed the fix574 branch 3 times, most recently from 1fef9e9 to db556bc Compare May 6, 2022 13:36
@RealKC RealKC force-pushed the fix574 branch 3 times, most recently from 28fa242 to 57424fb Compare May 6, 2022 15:29
@sdroege sdroege merged commit 4b53020 into gtk-rs:master May 7, 2022
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.

[FEATURE REQUEST] Move gio::compile_resources to a seperate crate
3 participants