[needs-packaging] arm-trusted-firmware-s32

Bug #2034642 reported by Laider Lai
16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu
In Progress
Wishlist
NemOS Team

Bug Description

[needs-packaging] arm-trusted-firmware-s32 (2.5-bsp37.0-8+nemos~202310131713~ubuntu23.10.1)
For Erlangen project, we have to provide the NXP S32G arm-trusted-firmware to the customer via Ubuntu archive (universe).

URL: https://launchpad.net/~nemos-team/+archive/ubuntu/ppa/+packages
Source code: https://code.launchpad.net/~nemos-team/nemos/+git/arm-trusted-firmware-s32/+ref/ubuntu/devel
License: https://git.launchpad.net/~nemos-team/nemos/+git/arm-trusted-firmware-s32/tree/debian/copyright
Notes: The NXP S32G arm-trusted-firmware (ATF) [Mantic]

Laider Lai (laiderlai)
information type: Public → Private
Changed in ubuntu:
assignee: nobody → Laider Lai (laiderlai)
status: New → In Progress
Laider Lai (laiderlai)
information type: Private → Public
Revision history for this message
Brian Murray (brian-murray) wrote :

*** This is an automated message ***

This bug is tagged needs-packaging which identifies it as a request for a new package in Ubuntu. As a part of the managing needs-packaging bug reports specification, https://wiki.ubuntu.com/QATeam/Specs/NeedsPackagingBugs, all needs-packaging bug reports have Wishlist importance. Subsequently, I'm setting this bug's status to Wishlist.

Changed in ubuntu:
importance: Undecided → Wishlist
Revision history for this message
Dave Jones (waveform) wrote :

Where is the upstream repo for this package? d/changelog claims that the base version is from git tag "bsp37.0-2.5" but the repo pointed to in d/control (https://source.codeaurora.org/external/autobsps32/arm-trusted-firmware) doesn't exist (in fact the whole source.codeaurora.org site appears to have disappeared according to https://bye.codeaurora.org/), and I can't find that tag in any of the arm-trusted-firmware repos pointed to from that page (the NXP GitHub or QUIC GitLab). Nor does it appear in the repo pointed to by d/watch (which also implies that's incorrect too).

Revision history for this message
Isaac True (itrue) wrote :

Hi Dave,

Sorry those repos were decommissioned and taken offline by NXP a few months ago. I'll update the packages with the new GitHub repos ASAP.

Revision history for this message
Dave Jones (waveform) wrote :

Ah, is the upstream repo https://github.com/nxp-auto-linux/arm-trusted-firmware by any chance? Just discovered that while looking at the associated u-boot package in LP: #2034650.

Revision history for this message
Isaac True (itrue) wrote :

Yes that's the one. Most NXP S32 BSP content is hosted under the nxp-auto-linux GH account

Revision history for this message
Isaac True (itrue) wrote :

I've updated the URLs in the git repo

Revision history for this message
Isaac True (itrue) wrote :

I've updated the copyright information in the repository. I've also made a few changes to the build process in order to enable UEFI secure boot on the target platform.

Laider Lai (laiderlai)
Changed in ubuntu:
assignee: Laider Lai (laiderlai) → NemOS Team (nemos-team)
Revision history for this message
Laider Lai (laiderlai) wrote :

Please hold on for this request.
Because the same platform's kernel resources may have some limitations.

We have to confirm the find public method about the kernel resources. Tks.

Changed in ubuntu:
status: In Progress → Incomplete
Revision history for this message
Laider Lai (laiderlai) wrote :

After checking with the HWE team, the kernel resources are okay to be published to the Ubuntu archive.
Therefore, the release target (Ubuntu archive) review process can continue.

Changed in ubuntu:
status: Incomplete → In Progress
Laider Lai (laiderlai)
description: updated
Laider Lai (laiderlai)
description: updated
Laider Lai (laiderlai)
description: updated
Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :
Download full text (6.2 KiB)

Hi,

I took a look at the current state of the package in the git repository and I have some notes to share:

- Source packag name: why did you decide to use arm-trusted-firmware-s32 when
  upstream names it as arm-trusted-firmware? I was not able to find the reason
  behind that decision.

- d/watch: it is not working properly. There is also a typo in the
  filenamemangle option:

diff --git a/debian/watch b/debian/watch
index 73f7a4b86..8af606be8 100644
--- a/debian/watch
+++ b/debian/watch
@@ -1,6 +1,6 @@
 version=4
 opts="\
-filenamemangle=s/(bsp\d+\.\d+)-(\d+\.\d+)\.tar\.gz/arm-trusted-firware-s32-$2-$1\.tar\.gz/,\
+filenamemangle=s/(bsp\d+\.\d+)-(\d+\.\d+)\.tar\.gz/arm-trusted-firmware-s32_$2-$1\.tar\.gz/,\
 uversionmangle=s/(bsp\d+\.\d+).(\d+\.\d+)/$2-$1/,\
 " \
 https://github.com/nxp-auto-linux/arm-trusted-firmware/tags .*/(bsp\d+\.\d+)-(\d+\.\d+)\.tar\.gz

  When trying to run uscan to download the version your are packaging it fails:

$ uscan --verbose --download-version 2.5-bsp37.0
uscan info: uscan (version 2.20.2ubuntu2) See uscan(1) for help
uscan info: Scan watch files in .
uscan info: Check debian/watch and debian/changelog in .
uscan info: package="arm-trusted-firmware-s32" version="2.5-bsp37.0-8" (as seen in debian/changelog)
uscan info: package="arm-trusted-firmware-s32" version="2.5-bsp37.0" (no epoch/revision)
uscan info: ./debian/changelog sets package="arm-trusted-firmware-s32" version="2.5-bsp37.0"
uscan info: Process watch file at: debian/watch
    package = arm-trusted-firmware-s32
    version = 2.5-bsp37.0
    pkg_dir = .
uscan info: opts: filenamemangle=s/(bsp\d+\.\d+)-(\d+\.\d+)\.tar\.gz/arm-trusted-firmware-s32_$2-$1\.tar\.gz/,uversionmangle=s/(bsp\d+\.\d+).(\d+\.\d+)/$2-$1/,
uscan info: line: https://github.com/nxp-auto-linux/arm-trusted-firmware/tags .*/(bsp\d+\.\d+)-(\d+\.\d+)\.tar\.gz
uscan info: Parsing filenamemangle=s/(bsp\d+\.\d+)-(\d+\.\d+)\.tar\.gz/arm-trusted-firmware-s32_$2-$1\.tar\.gz/
uscan info: Parsing uversionmangle=s/(bsp\d+\.\d+).(\d+\.\d+)/$2-$1/
uscan info: line: https://github.com/nxp-auto-linux/arm-trusted-firmware/tags .*/(bsp\d+\.\d+)-(\d+\.\d+)\.tar\.gz
uscan info: Last orig.tar.* tarball version (from debian/changelog): 2.5-bsp37.0
uscan info: Download the --download-version specified version: 2.5-bsp37.0
uscan info: Requesting URL:
   https://github.com/nxp-auto-linux/arm-trusted-firmware/tags
uscan info: Matching pattern:
   (?:(?:https://github.com)?\/nxp\-auto\-linux\/arm\-trusted\-firmware\/tags)?.*/(bsp\d+\.\d+)-(\d+\.\d+)\.tar\.gz
uscan info: Found the following matching hrefs on the web page (newest first):
   /nxp-auto-linux/arm-trusted-firmware/archive/refs/tags/bsp38.0-2.5.tar.gz (2.5-bsp38.0) index=2.5-bsp38.0-1
   /nxp-auto-linux/arm-trusted-firmware/archive/refs/tags/bsp37.0-2.5.tar.gz (2.5-bsp37.0) index=2.5-bsp37.0-1 matched with the download version
   /nxp-auto-linux/arm-trusted-firmware/archive/refs/tags/bsp36.0-2.5.tar.gz (2.5-bsp36.0) index=2.5-bsp36.0-1
   /nxp-auto-linux/arm-trusted-firmware/archive/refs/tags/bsp35.0-2.5.tar.gz (2.5-bsp35.0) index=2.5-bsp35.0-1
   /nxp-auto-linux/arm-trusted-firmware/archive/refs/tags/bsp34.2-2.5.tar.gz (2.5-bsp34.2) index=2.5...

Read more...

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.