Skip to content

chromium: Conditionally apply the Rust 1.8x patch#959

Merged
MaxIhlenfeldt merged 1 commit intoOSSystems:masterfrom
zboszor:master
Mar 11, 2026
Merged

chromium: Conditionally apply the Rust 1.8x patch#959
MaxIhlenfeldt merged 1 commit intoOSSystems:masterfrom
zboszor:master

Conversation

@zboszor
Copy link
Contributor

@zboszor zboszor commented Mar 11, 2026

For Rust 1.94 in Yocto master, the patch causes a build error:

--> ../../third_party/crabbyavif/src/src/lib.rs:16:46
|
16 | #![cfg_attr(feature = "disable_cfi", feature(no_sanitize))]
| ^^^^^^^^^^^ feature has been removed
|
= note: removed in 1.91.0; see rust-lang/rust#142681 for more information
= note: renamed to sanitize(xyz = "on|off")

Applying the patch conditionally based on RUSTVERSION allows building both under Yocto 5.3 and current master.

For Rust 1.94 in Yocto master, the patch causes a build error:

 --> ../../third_party/crabbyavif/src/src/lib.rs:16:46
   |
16 | #![cfg_attr(feature = "disable_cfi", feature(no_sanitize))]
   |                                              ^^^^^^^^^^^ feature has been removed
   |
   = note: removed in 1.91.0; see <rust-lang/rust#142681> for more information
   = note: renamed to sanitize(xyz = "on|off")

Applying the patch conditionally based on RUSTVERSION allows
building both under Yocto 5.3 and current master.

Signed-off-by: Zoltán Böszörményi <zboszor@gmail.com>
Copy link
Collaborator

@MaxIhlenfeldt MaxIhlenfeldt left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@MaxIhlenfeldt MaxIhlenfeldt merged commit 5f14862 into OSSystems:master Mar 11, 2026
Grippy98 pushed a commit to TexasInstruments/meta-arago that referenced this pull request Mar 13, 2026
Remove build workarounds that are no longer needed due to fixes in
meta-browser:
  - Rust 1.8x patch removal
  - ThinLTO linker OOM
  - pkg-config detection setup

These issues have been resolved upstream in meta-browser, so the local
changes in the bbappend can be removed.

OSSystems/meta-browser#957
OSSystems/meta-browser#958
OSSystems/meta-browser#959

Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
Signed-off-by: Ryan Eatmon <reatmon@ti.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants