Skip to content

Update to 0.27.0#9

Open
dcaliste wants to merge 46 commits into
sailfishos:masterfrom
dcaliste:update
Open

Update to 0.27.0#9
dcaliste wants to merge 46 commits into
sailfishos:masterfrom
dcaliste:update

Conversation

@dcaliste
Copy link
Copy Markdown

@dcaliste dcaliste commented Jan 13, 2026

This is an attempt to update to 0.27.0. Some remarks:

  • I didn't find any other solution than patching the Cargo.toml files of subcrates when they are using data from a workspace.
  • I'm dropping Windows support, that saves the windows-rs crate. Debian is doing the same, as far as I know.
  • I've singled out every update and addition of new crates. I can squash all if preferable.

I don't know if I properly formatted the patches, especially in terms of paths. Currently, I'm unable to build it with rpmbuild -bb --build-in-place inside cbindgen/ subdir. So, I'm applying the patches by hand, from the root of the repository, then applying by hand the various other steps in prep.

@mlehtima
Copy link
Copy Markdown
Contributor

mlehtima commented Jan 13, 2026

Sorry but you probably should use the existing 0.26 update PR as example, we are moving away from submodule use for rust crates which makes things much simpler. Need to see if that 0.27 works with current rust we have or should we first integrate that 0.26 I can also see if I should just update that existing 0.26 to newer version.

@dcaliste
Copy link
Copy Markdown
Author

Ah, sorry, my bad. I didn't notice. I went to the branch page to see what was going on, and even didn't visit the "all" tab. And the "active" tab was only showing the master branch. I forgot to give a look to the PR.

Indeed dropping the submodules will help a lot ! I let you continue with the 0.26 PR and I'll update this one with a simpler upgrade to 0.27 when ready. Thanks.

@mlehtima
Copy link
Copy Markdown
Contributor

The 0.26.0 PR was merged so this can now be rebased and made much simpler.

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.

2 participants