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

Janitor: Update the image crate #4816

Merged
merged 2 commits into from
Apr 15, 2024
Merged

Conversation

ogoffart
Copy link
Member

@ogoffart ogoffart commented Mar 11, 2024

Note: this could be a breaking change if someone did

image = { version = "0.24", features = [...] }

to enable more features decoder of the image to support more file format in slint

@ogoffart ogoffart requested a review from tronical March 11, 2024 17:17
Copy link
Member

@tronical tronical left a comment

Choose a reason for hiding this comment

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

Ah, the breakage would be that those image formats aren’t enabled anymore?

@ogoffart
Copy link
Member Author

Looks like the link is failling on Windows with missing symbols relative to the image crate :-(
https://github.com/slint-ui/slint/actions/runs/8237024401/job/22524823230

@tronical
Copy link
Member

And in a way strange way. There are unresolved symbols relating to image loading, but also these:

libi_slint_compiler-ed7475cbe5efdf15.rlib(i_slint_compiler-ed7475cbe5efdf15.i_slint_compiler.28c6a588-cgu.13.rcgu.o) : error LNK2001: unresolved external symbol _ZN4core3fmt10ArgumentV19new_debug17h2589fb0f131777baE±
libi_slint_compiler-ed7475cbe5efdf15.rlib(i_slint_compiler-ed7475cbe5efdf15.i_slint_compiler.28c6a588-cgu.1.rcgu.o) : error LNK2001: unresolved external symbol _ZN72_$LT$alloc..vec..Vec$LT$T$C$A$GT$$u20$as$u20$core..ops..deref..Deref$GT$5deref17he36e3e2ca0db1250E

@ogoffart ogoffart force-pushed the olivier/dependencies-updates branch from 1098a89 to 66b3ec0 Compare March 26, 2024 15:30
Note: this could be a breaking change if someone did

```
image = { version = "0.24", features = [...] }
```

To enable more features decoder of the image to support more file format
in slint
@ogoffart ogoffart force-pushed the olivier/dependencies-updates branch from 31dffc4 to a88bc23 Compare March 27, 2024 08:05
To enable the feature from the image crate.
And enable it by default.
Because users of older version of slint may have enabled feature from
the image 0.24 crate. But since we upgrade to image 0.25, this these
format would not be supported by Slint. So enable them by default to
keep compatibility
@ogoffart
Copy link
Member Author

Added a commit to add a (enabled by default) feature to enable all image backends with the image 0.25 crate, in order not to break the backends that were enabled for the 0.24 by users

@ogoffart ogoffart merged commit a159562 into master Apr 15, 2024
36 checks passed
@ogoffart ogoffart deleted the olivier/dependencies-updates branch April 15, 2024 10:11
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