Re: [PATCH 1/6] meta-isar-bin: Enable caching of deb packages

[Claudius Heine <claudius.heine.ext@siemens.com>]

Hi,

On 09/06/2017 04:21 PM, [ext] Henning Schild wrote:
>> --- a/meta/classes/image.bbclass
>> +++ b/meta/classes/image.bbclass
>> @@ -11,18 +11,34 @@ inherit ${IMAGE_TYPE}
>>   
>>   do_populate[stamp-extra-info] = "${MACHINE}-${DISTRO}"
>>   
>> -# Install Debian packages, that were built from sources
>> +# Install Debian packages from the cache
>>   do_populate() {
>> +    readonly DIR_CACHE="${DEBCACHEDIR}/${DISTRO}"
>> +
>>       if [ -n "${IMAGE_INSTALL}" ]; then
>> -        sudo mkdir -p ${S}/deb
>> +        if [ "${DEBCACHE_ENABLED}" != "0" ]; then
>> +            sudo mkdir -p "${S}/${DEBCACHEMNT}"
>> +            sudo mount -o bind "${DIR_CACHE}" "${S}/${DEBCACHEMNT}"
>> +
>> +            sudo chroot "${S}" apt-get update -y
>> +            for package in ${IMAGE_INSTALL}; do
>> +                sudo chroot "${S}" apt-get install -t
>> "${DEBDISTRONAME}" -y \
>> +                    --allow-unauthenticated "${package}"
> 
> With do_populate using apt-get we now how to deal with package-name
> collisions. We could have the case where our repo contains a package
> that already exists in upstream debian.
> 
> We will have to decide how to deal with them if they occur. My
> suggestions/first idea would be:
>   - a recipe that creates a .deb with a taken package name fails by
>     default -EBUSY
>   - a recipe can set a variable to force itself in "OVERRIDE_DEBIAN =
>     True"
>   - a recipe that overrides the upstream package has to "win", we need
>     a proper apt Pin-Priority

If all packages that are created by isar have for instance 'isar' in the 
version, then it should be possible to specify this version explicitly 
when installing them from the repository. I think pining is only 
necessary if upgrading packages on the target is a required feature.

One option for this would be that another meta package (maybe created in 
the image recipe) has explicit version dependencies on all packages 
created by isar. This way apt would not upgrade those packages because 
it would break the version dependency.

Cheers,
Claudius


-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de