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

[Aliaksei Karpovich <akarpovich@ilbers.de>]

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.

-- 
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/1c3606e4-ec08-4701-8487-688cb8e01d67%40ilbers.de.