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

Added bindings for Gio.DBusObjectManager, Gio.DBusObjectManagerClientFlags #1156

Merged
merged 2 commits into from
Aug 7, 2023

Conversation

pentamassiv
Copy link
Contributor

I noticed that the above mentioned objects are not generated. I am currently working on updated bindings for Gnome Online Accounts and some of the code can't be generated because they are missing from gio.

For some reason I get a ton of formatting changes after regenerating the code, so I first regenerated before doing changes. Then I added them to the Gir.toml file and regenerated the bindings again. There were only the expected lines to uncommented after that. I ran cargo build and cargo test without any issues.

Because I don't know why I had all these formatting changes, I did not include any of the regeneration commits.

@bilelmoussaoui
Copy link
Member

You have to re-run the generator as there are no bindings generated as you can tell.

@pentamassiv
Copy link
Contributor Author

Unfortunately I can't. I tried to explain that for some reason I get a ton of formatting changes when I regenerate the code and it becomes difficult to see the changes of this PR.
I added another commit for you to see what I mean. I switched to stable Rust and tried reinstalling gir from this repo. It still causes big changes :S

@pentamassiv
Copy link
Contributor Author

pentamassiv commented Aug 6, 2023

Sorry, nevermind. It works. I previously ran gir directly and did not use the python script. It works when using the python script

@bilelmoussaoui bilelmoussaoui added the needs-backport PR needs backporting to the current stable branch label Aug 6, 2023
@pentamassiv pentamassiv force-pushed the dbusObjectMgr branch 2 times, most recently from 6b9f8dd to e745005 Compare August 6, 2023 19:14
@sdroege sdroege merged commit 59e891f into gtk-rs:master Aug 7, 2023
47 checks passed
@bilelmoussaoui bilelmoussaoui added backported PR was backported to the current stable branch and removed needs-backport PR needs backporting to the current stable branch labels Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported PR was backported to the current stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants