[Bug 2034648] Re: [needs-packaging] optee-os-s32

[Laider Lai]

Hey Loic,

Thanks for your review.
A newer version (optee-os-s32 - 3.18-bsp37.0-0ubuntu1~ppa5) is uploaded to PPA.
The changes are listed below.
(commit: https://git.launchpad.net/~nemos-team/nemos/+git/optee-os-s32/commit/?id=69b90f29d401ef8f323b17413967cf437e2d83bc)

=> I think this should be "s32g2" instead of "export-ta_arm64" since optee is built with PLATFORM=s32 PLATFORM_FLAVOR=s32g2
[Laider] Align with optee-os_4.2.0-3.dsc from Debian
         The tee-*_v2.bin is installed to /usr/lib/optee/s32g2/
         The export-ta_arm64 content is installed to /usr/lib/optee-os-dev/s32

=> I think we should align the build command between optee-os and optee-os-s32 (config flags)
[Laider] We can't do it due to the flags got building fail with Launchpad PPA
Ex.
-O2 flag is not supported

2)5)7)...
7) the reason for debian/patches/drivers-hse-Remove-access-to-HSE-RNG-without-proper-init.patch should be
documented in the patch
[Laider] The README.Debian is added with why we have this package, the patch, and explain how to build HSE version.
         It also includes how to build optee-test with the content of optee-* dev package.
         And 

3) optee-os-s32 build-depends on nemos-dev-cert and nemos-dev-key...
[Laider] Indeed, we can use default key for optee-os and optee-test (sign TAs)
         The key is changed back to use the default one.
  
4) IMO the build-dep on binutils can be dropped; binutils is pulled through gcc through build-essential, so pretty much no package will list it explicitly
[Laider] Agree with it, binutils is removed.

6) keep changelog history
[Laider] Thanks, I keep the changelog history at the latest version

** Changed in: ubuntu
       Status: In Progress => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/2034648

Title:
  [needs-packaging] optee-os-s32

Status in Ubuntu:
  Incomplete

Bug description:
  [needs-packaging] optee-os-s32 (3.18-bsp37.0-0ubuntu1)
  For Erlangen project, we have to provide the NXP S32G OP-TEE OS to the customer via Ubuntu archive (universe).

  URL: https://launchpad.net/~nemos-team/+archive/ubuntu/archive-target/+packages
  Source code: https://code.launchpad.net/~nemos-team/nemos/+git/optee-os-s32/+ref/ubuntu/archive
  License: https://git.launchpad.net/~nemos-team/nemos/+git/optee-os-s32/tree/debian/copyright
  Notes: The NXP S32G OP-TEE OS [Noble]

  This package has a dependency for (LP: #2043448) and should wait for
  it to be approved into the target archive.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/2034648/+subscriptions