From: Henning Schild <henning.schild@siemens.com>
To: "[ext] claudius.heine.ext@siemens.com" <claudius.heine.ext@siemens.com>
Cc: <isar-users@googlegroups.com>, Claudius Heine <ch@denx.de>
Subject: Re: [PATCH v2 0/3] One-shot configuration packages
Date: Wed, 23 May 2018 16:03:27 +0200 [thread overview]
Message-ID: <20180523160327.714453e7@md1pvb1c.ad001.siemens.net> (raw)
In-Reply-To: <20180523131248.31907-1-claudius.heine.ext@siemens.com>
Nice patchset. It follows the "everything that enters the rootfs should
come from a package" approach. And it introduced a nice hack to execute
scripts in the "last package", switching from the chrootable folder to
the image.
Things like removing qemu, resolv.conf etc could also be done in such a
"final" package postrm.
Henning
Am Wed, 23 May 2018 15:12:45 +0200
schrieb "[ext] claudius.heine.ext@siemens.com"
<claudius.heine.ext@siemens.com>:
> From: Claudius Heine <ch@denx.de>
>
> Hi,
>
> here is the rebased patchset that removes the issue with the global
> localepurge configuration as well.
>
> Claudius
>
> Changes from v1:
> - rebased to current next
> - removed locale.nopurge setup in isar-bootstrap
>
> Claudius Heine (3):
> isar-bootstrap|configscript: implement one shot config option
> meta/isar-cfg-localepurge: added locale purge recipe
> meta-isar/isar-image-base: switch use isar-cfg-localepurge
>
> .../images/files/debian-configscript.sh | 9 ------
> .../images/files/raspbian-configscript.sh | 8 -----
> .../recipes-core/images/isar-image-base.bb | 6 ++--
> meta/classes/isar-bootstrap-helper.bbclass | 23 +++++++++++---
> .../isar-bootstrap/files/locale.nopurge | 10 -------
> .../isar-bootstrap/isar-bootstrap.bb | 4 +--
> .../isar-cfg-localepurge/files/locale.debconf | 12 ++++++++
> .../isar-cfg-localepurge/files/locale.gen | 1 +
> .../isar-cfg-localepurge/files/postinst | 30
> +++++++++++++++++++ .../isar-cfg-localepurge.bb |
> 19 ++++++++++++ 10 files changed, 85 insertions(+), 37 deletions(-)
> delete mode 100644
> meta/recipes-core/isar-bootstrap/files/locale.nopurge create mode
> 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.debconf
> create mode 100644
> meta/recipes-support/isar-cfg-localepurge/files/locale.gen create
> mode 100644 meta/recipes-support/isar-cfg-localepurge/files/postinst
> create mode 100644
> meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb
>
next prev parent reply other threads:[~2018-05-23 14:03 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-23 13:12 claudius.heine.ext
2018-05-23 13:12 ` [PATCH v2 1/3] isar-bootstrap|configscript: implement one shot config option claudius.heine.ext
2018-05-23 14:06 ` Henning Schild
2018-05-24 7:06 ` Claudius Heine
2018-05-24 8:57 ` Henning Schild
2018-05-23 13:12 ` [PATCH v2 2/3] meta/isar-cfg-localepurge: added locale purge recipe claudius.heine.ext
2018-05-23 14:16 ` Henning Schild
2018-05-23 14:39 ` Claudius Heine
2018-05-23 13:12 ` [PATCH v2 3/3] meta-isar/isar-image-base: switch use isar-cfg-localepurge claudius.heine.ext
2018-05-23 14:03 ` Henning Schild [this message]
2018-05-24 8:38 ` [PATCH v2 0/3] One-shot configuration packages Claudius Heine
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180523160327.714453e7@md1pvb1c.ad001.siemens.net \
--to=henning.schild@siemens.com \
--cc=ch@denx.de \
--cc=claudius.heine.ext@siemens.com \
--cc=isar-users@googlegroups.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox