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

Update meta-balena-rust to Scarthgap #3448

Draft
wants to merge 35 commits into
base: master
Choose a base branch
from
Draft

Conversation

alexgg
Copy link
Contributor

@alexgg alexgg commented Jun 24, 2024


Contributor checklist

Reviewer Guidelines

  • When submitting a review, please pick:
    • 'Approve' if this change would be acceptable in the codebase (even if there are minor or cosmetic tweaks that could be improved).
    • 'Request Changes' if this change would not be acceptable in our codebase (e.g. bugs, changes that will make development harder in future, security/performance issues, etc).
    • 'Comment' if you don't feel you have enough information to decide either way (e.g. if you have major questions, or you don't understand the context of the change sufficiently to fully review yourself, but want to make a comment)

jakogut and others added 30 commits March 29, 2024 13:20
This is a 1:1 copy of meta-balena-kirkstone.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
As of Yocto Mickledore (4.2), checksums are mandatory for Rust crates.

https://docs.yoctoproject.org/next/migration-guides/migration-4.2.html#rust-mandatory-checksums-for-crates

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
As of Yocto Mickledore (4.2), checksums are mandatory for Rust crates.

https://docs.yoctoproject.org/next/migration-guides/migration-4.2.html#rust-mandatory-checksums-for-crates

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
As of Yocto Mickledore (4.2), checksums are mandatory for Rust crates.

https://docs.yoctoproject.org/next/migration-guides/migration-4.2.html#rust-mandatory-checksums-for-crates

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
As of Yocto Mickledore (4.2), checksums are mandatory for Rust crates.

https://docs.yoctoproject.org/next/migration-guides/migration-4.2.html#rust-mandatory-checksums-for-crates

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Drop unneeded patch, as it raises a QA issue, and isn't needed anymore
after enabling usrmerge.

Fixes "QA Issue: Malformed Upstream-Status in patch"

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
bmaptool was renamed and incorporated into the Yocto project in
Scarthgap, so adjust the package name.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Fixes build with kernel v6.6.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Bump to latest revision, which pulls in Makefile fixes for merged usr
directory.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Co-authored-by: Alexandru Costache <[email protected]>
Yocto scarthgap includes a version of systemd that requires the usrmerge
distro feature. Enable it.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
This fixes an issue with the package not obeying the usrmerge feature.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
The datastore ('d') variable is no longer accessible in this event. Copy
upstream behavior and get the datastore from the 'e' variable available
to the event handler.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
We have a Rust layer in order to use the same toolchain across all
supported compatibility layers.

Yocto Langdale splits classes by usage context [0], which makes our recipe
inherit from the upstream class instead of the one from the same layer.

Add the balena- prefix to the classes for this layer to ensure we
inherit from the appropriate versions.

[0] https://docs.yoctoproject.org/next/migration-guides/migration-4.1.html#classes-now-split-by-usage-context

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Replace references to deprecated distutils module in Rust toolchain.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
The networkmanager-daemon package specifies an initscript that a poky
postinstall script fails to find for installation, as we didn't copy it
to meta-balena, and use systemd instead.

Clear the initscript variables in the bbappend.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Nanbield introduced a minor change [0] that includes IMAGE_NAME_SUFFIX
in IMAGE_NAME and IMAGE_LINK_NAME.

Adapt image output paths to support this change, and backport the new
behavior to images built with older Yocto releases.

[0] https://docs.yoctoproject.org/dev/migration-guides/migration-4.3.html#output-file-naming-changes

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
systemd-tmpfiles reports that /var/run is a legacy path, and updates it
to just /run

Update this configuration file to use the now preferred path to silence
this warning.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
When building plymouth 24.004.60 with GCC 13.2 and the `-Dudev=disabled`
configuration, the build fails with several undefined symbols.

    | /work/build/tmp/work/core2-64-poky-linux/plymouth/24.004.60/recipe-sysroot-native/usr/bin/x86_64-poky-linux/../../libexec/x86_64-poky-linux/gcc/x86_64-poky-linux/13.2.0/ld: src/libply-splash-core/libply-splash-core.so.5.0.0.p/ply-device-manager.c.o: in function `create_devices_for_terminal_and_renderer_type':
    | ply-device-manager.c:(.text+0xf1e): undefined reference to `add_input_devices_to_renderer'
    | collect2: error: ld returned 1 exit status

It's unclear what's causing this build error, but it appears to be a bug
in plymouth. Building with clang results in a similar undefined symbol
error.

Rather than clearing the entire PACKAGECONFIG variable, explicitly
remove unneeded configs, leaving udev enabled to fix the build.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
@alexgg alexgg marked this pull request as draft June 24, 2024 10:08
@alexgg alexgg force-pushed the alexgg/scarthgap-osconfig branch 6 times, most recently from 0e5b255 to c77ff73 Compare June 24, 2024 10:41
This corresponds to Poky Scarthgap
abcd5a88a3712e07589073212d95adff4b4ce959

Change-type: minor
Signed-off-by: Alex Gonzalez <[email protected]>
@alexgg alexgg force-pushed the alexgg/scarthgap-osconfig branch from c77ff73 to 63fe879 Compare June 24, 2024 11:43
jaomaloy added 2 commits July 10, 2024 14:17
Use cmake 3.20.1 for dunfell devices to build
with rust 1.73.0

Change-type: patch
Use cmake 3.20.1 to build rust v1.73.0 successfully.

Change-type: patch
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