-
Notifications
You must be signed in to change notification settings - Fork 22
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
kas/common/base: update isar to latest revision #411
Conversation
Checking issue with rpi4 builds |
thank you @baprusty ! |
127d744
to
cd38616
Compare
@chombourger, Rpi-firmware and mtda-image of rpi4b build is working with the updated changes. Please review |
Great. I have restarted the CI/CD checks |
@@ -7,8 +7,8 @@ | |||
|
|||
inherit dpkg | |||
|
|||
SRC_URI = "apt://${PN} \ | |||
file://changelog.tmpl" | |||
SRC_APT = "${PN}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And, in fact, it is not the official interface in Isar.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh okay got it, was got confused by warning message:
WARNING: /build/../repo/meta-isar/recipes-crypto/gnutls/gnutls28.bb: Unable to get checksum for gnutls28-native SRC_URI entry changelog.tmplapt: file could not be found
So, the issue is SRC_URI entry on this is getting parsed twice as:
- changelog.tmplapt, which is not added. Workaround: adding a space after filename like
file://changelog.tmpl "
fixes this warning message. - changelog.tmpl, which is correct
Another issue is gnutls28 fails to build: bitbake gnutls28
because of trousers library not found.
Checking further.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added a space at the end for now to avoid unneeded warning.
gnutls28 build is working fine on mtda-qemu-amd64
environment.
The previous failure of gnutls28
was tested on mtda-nanopi-neo
env if we do bitbake gnutls28
.
But nanopi-image includes upstream gnutls28:
bp@debian:~/mtda-update-isar/mtda/build/tmp/deploy/images/nanopi-neo$ grep gnutls mtda-image-mtda-bookworm-nanopi-neo.manifest
curl|7.88.1-10+deb12u5|libcurl3-gnutls:armhf|7.88.1-10+deb12u5
gnutls28|3.7.9-2+deb12u2|libgnutls30:armhf|3.7.9-2+deb12u2
which is coming as a part of dependency package of some other package.
@chombourger Querry:
- Is it expected to include gnutls28 recipe package with
+mtda
postfix: 3.7.9-2+deb12u2+mtda to the mtda image or it is fine?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we need our custom build as long as upstream does not build gnutls without tpm support (appears to not be enabled in Ubuntu 22.04). We need tpm support when running mtda-kvm with swtpm
cd38616
to
5b0908d
Compare
- Update to latest ISAR revision - Rpi-firmware: Consists of .elf binary files Avoid striping it or strip dwarf debug information Fixes build failure Signed-off-by: Badrikesh Prusty <[email protected]>
5b0908d
to
3c2de96
Compare
Avoid striping it or strip dwarf debug information
Fixes build failure
Nano pi build and boot tested:
Nanopi neo booting fine. Boot logs:
mtda-latest-isar.txt