-
-
Notifications
You must be signed in to change notification settings - Fork 132
Automated PR: Changes from updating gir/gir-files #1821
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
base: main
Are you sure you want to change the base?
Conversation
c501c72 to
1770de7
Compare
abc9b30 to
9b240a2
Compare
9b240a2 to
17decdb
Compare
c9868a6 to
7e62964
Compare
7e62964 to
aa23d6a
Compare
|
Sorry for the big commit but i don't have the time to cleanly do it. Other than that, we still miss README.md in those new crates, a license symlink like the others and probably a review of what I did. |
6a5f921 to
f41f3c5
Compare
80d9c57 to
dffad11
Compare
dffad11 to
f41f3c5
Compare
We require v2.84 for it nowadays anyways
Requires fixing gir
That is part of glib-unix
ee1d1ce to
bd5573f
Compare
Copying what we had in gio where it makes sense
We need a gsvbuild bump for that, can be done later
74981ff to
c2aacc5
Compare
c2aacc5 to
f9883ab
Compare
f9883ab to
46e5fcc
Compare
46e5fcc to
41ef29d
Compare
41ef29d to
038b0a7
Compare
038b0a7 to
ecca590
Compare
|
We need #1853 to fix the windows builds |
| pub fn g_unix_pipe_close( | ||
| self_: *mut glib::GUnixPipe, | ||
| end: glib::GUnixPipeEnd, | ||
| self_: *mut GUnixPipe, |
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.
The commit glib/unix: HACK: Manually fix GIR file probably needs to be dropped and solved differently or otherwise the CI will find changes when running the generator
| env: | ||
| # git revision of gvsbuild we use for to build GLib and the other dependencies | ||
| gvsbuildref: 230e1074047b9654b76217c5d48b10c726fbe0f6 | ||
| gvsbuildref: 352e6772c61763af1178b1bb0c2cb0adb8f81d4b |
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.
Needs rebasing for your other PR that was merged :)
| #[cfg(windows)] | ||
| mod win32_input_stream; | ||
| #[cfg(windows)] | ||
| pub use self::win32_input_stream::Win32InputStream; |
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.
For backwards compatibility reasons to backport this, we probably need some kind of pub use gio_unix::* (and the same for win32 and glib) here. I wouldn't do it for main though.
Is that somehow possible or should we just stop backporting new API until next summer?
| ( | ||
| Socket::from_fd(OwnedFd::from_raw_fd(fds[0])).unwrap(), | ||
| Socket::from_fd(OwnedFd::from_raw_fd(fds[1])).unwrap(), | ||
| /* |
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.
Some commented out code is leftover here
| pub use ffi; | ||
|
|
||
| #[allow(unused_imports)] | ||
| #[allow(clippy::derived_hash_with_manual_eq)] |
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.
Which type is that for? That's potentially a bug
|
|
||
| #[allow(clippy::too_many_arguments)] | ||
| #[allow(unused_imports)] | ||
| #[allow(clippy::empty_line_after_outer_attr)] |
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.
Is fixed now :)
| - { name: "gdk-pixbuf", test: true, args: "--features v2_42" } | ||
| - { name: "gio", test: true, args: "--features v2_80" } | ||
| - { name: "gio-win32", test: true, args: "--features v2_84" } | ||
| - { name: "gio-win32", test: true, args: "--features v2_66" } |
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.
Happened
|
This generally looks good to me apart from the above. I guess my main question left here is how to handle backports, but we don't have to answer that as part of this PR. |
This PR contains auto-generated changes after a merge in the external gir-files repository.