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

kas/common/base: update isar to latest revision #411

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

baprusty
Copy link
Contributor

@baprusty baprusty commented Mar 8, 2024

  • Update to latest ISAR revision
  • Use SRC_APT for apt packages (vs SRC_URI). Here, for gnutls package.
  • Rpi-firmware: Consists of .elf binary files
    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

@baprusty
Copy link
Contributor Author

baprusty commented Mar 8, 2024

Checking issue with rpi4 builds

@chombourger
Copy link
Collaborator

Checking issue with rpi4 builds

thank you @baprusty !

@baprusty
Copy link
Contributor Author

baprusty commented Mar 8, 2024

@chombourger, Rpi-firmware and mtda-image of rpi4b build is working with the updated changes. Please review

@chombourger chombourger added check and removed check labels Mar 8, 2024
@chombourger
Copy link
Collaborator

@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}"
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not needed.

Copy link
Contributor

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.

Copy link
Contributor Author

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.

Copy link
Contributor Author

@baprusty baprusty Mar 9, 2024

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?

Copy link
Collaborator

@chombourger chombourger Mar 11, 2024

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

- 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]>
@chombourger chombourger merged commit 0382342 into siemens:master Mar 11, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants