Re: [PATCH v4 0/4] Basic binary cache implementation

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

Hi,

On 10/0	'2/2017 05:45 PM, Alexander Smirnov wrote:
> Hello everybody,
> 
> this series introduces basic binary caching for Isar.
> 
> There is a new layer: meta-isar-bin which is intended to be the binary
> cache. All the packages that are built in Isar are stored in this
> cache using reprepro.
> 
> Having the separate layer makes possible to manage this cache separately
> from the Isar build tree. So once you have built your packages, you could
> re-use this cache for further builds.

We still have to talk about your requirement for the cache to be 
sharable and your resulting implementation of it as a meta layer.

Since I still don't see the benefit of a sharable cache and think that 
putting binary packages into some kind of strange meta layer is not the 
right solution and could potentially create more problems that is solves 
because its very much in conflict with the fundamental idea of a meta layer.

Please give me a reason for this requirement.

And to be precise: Yes, a cache is useful and I fully support it, but 
having it sharable over multiple hosts is not and is potentially dangerous.

Claudius

> 
> Also this cache repo is now used to generate image filesystem via
> multistrap.
> 
> Changes since v3:
>   - Check return code from reprepro, continue loop only if EEXIST code is
>     returned. Othrwise build is aborted.
> 
> Changes since v2:
>   - Droped anonymous functions
>   - Added number of attempts to access apt cache before fail
> 
> Changes since v1:
>   - Total refactoring
>   - Split huge patches into small pieces
>   - Avoid races
> 
> With best regards,
> Alex
> 
> Alexander Smirnov (4):
>    meta-isar-bin: Add reprepro configs
>    meta-isar-bin: Generate cache repos
>    meta-isar-bin: Populate cache
>    meta-isar-bin: Install packages via multistrap
> 
>   meta-isar-bin/conf/layer.conf                      | 14 +++++
>   meta-isar-bin/files/distributions.in               |  3 ++
>   meta-isar/conf/bblayers.conf.sample                |  1 +
>   meta-isar/conf/local.conf.sample                   |  4 ++
>   .../recipes-core/images/files/multistrap.conf.in   |  9 +++-
>   meta-isar/recipes-core/images/isar-image-base.bb   |  5 +-
>   meta/classes/ext4-img.bbclass                      |  2 +-
>   meta/classes/image.bbclass                         | 62 +++++++++++++++++++---
>   8 files changed, 89 insertions(+), 11 deletions(-)
>   create mode 100644 meta-isar-bin/conf/layer.conf
>   create mode 100644 meta-isar-bin/files/distributions.in
> 

-- 
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