Re: [v8 12/14] rootfs: Add version and arch for downloaded package

["'Jan Kiszka' via isar-users" <isar-users@googlegroups.com>]

On 29.07.26 10:55, Aliaksei Karpovich wrote:
> 
> On 7/28/26 11:25, MOESSBAUER, Felix wrote:
>> On Mon, 2026-07-27 at 13:26 +0200, Aliaksei Karpovich wrote:
>>> It fixes following error:
>>>   | E: Internal Error, Pathname to install is not absolute 'gcc-14-
>>> base_14.2.0-19_arm64.deb'
>>>
>>> In case of cross compile the arch of package is lost. F.e. the
>>> gcc-14-base_14.2.0-19_arm64.deb will be transformed in
>>> gcc-14-base name and amd64 version will be downloaded instead of
>>> arm64. We should take into account arch and version.
>>> After this fix package name will be extracted in following way:
>>> gcc-14-base_14.2.0-19_arm64.deb -> gcc-14-base:arm64=14.2.0-19
>>>
>>> Signed-off-by: Aliaksei Karpovich <akarpovich@ilbers.de>
>>> ---
>>>   meta/classes-recipe/rootfs.bbclass | 4 +++-
>>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/meta/classes-recipe/rootfs.bbclass b/meta/classes-
>>> recipe/rootfs.bbclass
>>> index 2ffbbca7..f6913cff 100644
>>> --- a/meta/classes-recipe/rootfs.bbclass
>>> +++ b/meta/classes-recipe/rootfs.bbclass
>>> @@ -374,13 +374,15 @@ rootfs_install_pkgs_isar_download() {
>>>       # Command apt-get install do not cache packages from local repos
>>>       # We can obtain non cached package URIs by recalling install
>>> command here
>>>       # No need in export those files to dl_dir, so we can run it
>>> right after
>>> +    # Package name extraction from URI for download : libmagic-
>>> mgc_1%3a5.46-5_amd64.deb -> libmagic-mgc:amd64=1:5.46-5
>>>       rootfs_cmd --bind "${ROOTFSDIR}/var/cache/apt/archives" /var/
>>> cache/apt/archives \
>>>                  --bind "${WORKDIR}/dpkg/lock" /var/lib/dpkg/lock \
>>>                  --bind "${WORKDIR}/dpkg/lock-frontend" /var/lib/
>>> dpkg/lock-frontend \
>>>                  --chdir "/var/cache/apt/archives" \
>>>                  ${ROOTFSDIR} \
>>>                  -- /usr/bin/sh -c "apt-get ${ROOTFS_APT_ARGS} --
>>> print-uris ${ROOTFS_PACKAGES} |  \
>>> -                                  sed -n \"s|^[^ ]*
>>> \([^_]*\)_[^_]*_[^_]*\\.deb .*|\1|p\" | \
>>> +                                  sed -n 's|^[^ ]*
>>> \([^_]*\)_\(.*\)_\([^_]*\)\.deb .*|\1:\3=\2|p' | \
>>> +                                  sed 's/%3a/:/g' | \
>>>                                     xargs -r apt-get download"
>> Hi, as we use --print-uris, the result should be something that can
>> directly be downloaded with curl (and a specific user-agent). Is there
>> any particular reason to use apt for the download? If not, the whole
>> reformatting could be avoided.
>>
>> Anyways, to me it looks like this is fix independent of the series.
>>
>> Felix
> 
> Thank for idea.
> This fix will be redone with curl and moved into separate patch.
> 

This change is critical due to [1]. Please address this independently of
the series soon so that we can resolve the "regression" of disabling
local "downloads". It's blocking downstream updates to rootless builds,
thus further testing. Maybe even fix the url parsing first and then
refactor to curl/wget fetching on top, whatever is faster.

Jan

[1]
https://groups.google.com/d/msgid/isar-users/4f230241-bb11-42ab-873d-d011b87c7e92%40siemens.com

-- 
Siemens AG, Foundational Technologies
Linux Expert Center

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/be154791-16b1-4914-9fb1-c66d27e50ff9%40siemens.com.