From: Jan Kiszka <jan.kiszka@siemens.com>
To: "Maxim Yu. Osipov" <mosipov@ilbers.de>,
Claudius Heine <claudius.heine.ext@siemens.com>,
isar-users@googlegroups.com
Cc: Claudius Heine <ch@denx.de>
Subject: Re: [PATCH v6 1/9] isar-bootstrap-host: disable DISTRO_BOOTSTRAP_KEYS usage
Date: Wed, 22 May 2019 09:02:27 +0200 [thread overview]
Message-ID: <80d56fc5-00fa-5508-7fb1-976b4b5c61db@siemens.com> (raw)
In-Reply-To: <f63ae39c-f5f5-3f08-41fd-4149593366e0@ilbers.de>
On 22.05.19 08:55, Maxim Yu. Osipov wrote:
> On 5/22/19 8:37 AM, Claudius Heine wrote:
>> Hi Jan,
>>
>> On 21/05/2019 18.56, Jan Kiszka wrote:
>>> On 15.05.19 12:11, [ext] claudius.heine.ext@siemens.com wrote:
>>>> From: Claudius Heine <ch@denx.de>
>>>>
>>>> isar-bootstrap-host only supports bootstrapping Debian root file
>>>> systems. Therefore deactivate any DISTRO_BOOTSTRAP_KEYS from other
>>>> distributions.
>>>
>>> Actually not totally true, as I just realized: What about bootstrapping the
>>> buildchroot from a custom debian repo that was differently signed (e.g.
>>> because it is a condensed version of upstream)? Seems we do need
>>> HOST_DISTRO_BOOTSTRAP_KEYS, right?
>>
>> You are right, I haven't considered that case. Well, the next task on the todo
>> list should probably be to refactor and streamline the isar-bootstrap, and
>> especially the host bootstrap process and fix those kind of issues while doing
>> that.
>>
>> There are a lot of possible customization options gained if the current giant
>> bootstrap function would be split up, similar to how the rootfs system works
>> in the pre-processing patchset.
>>
>> Maybe it makes sense to also start renaming "host" and "target" to be
>> compatible with the gcc nomenclature [1] in that patchset. I am a bit
>> reluctant to do so, because of the breakage involved. But the further we wait,
>> to more stuff will break downstream.
>>
>> The plan would be to rename all occurrences of "host" to "build" and "target"
>> to "host". That would lead to the following recipe changes:
>
>
>> "buildchroot-host" -> "buildchroot-build"
>> "buildchroot-target" -> "buildchroot-host"
>> "isar-bootstrap-host" -> "isar-bootstrap-build"
>> "isar-bootstrap-target" -> "isar-bootstrap-host"
>
>
>> I am on the fence of that change. Correctness vs. no-breakage
>>
>> Any comments about that?
>
> I would prefer to avoid such a renaming taking into account needed efforts and
> possible confusion for current Isar users.
>
Is that build/host scheme then also in line with Debian naming? Then there will
be eventually no way around it anyway. But we really need to do this thoroughly,
specifically /wrt to user-visible interfaces, so that it will be one cut only.
Jan
--
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux
next prev parent reply other threads:[~2019-05-22 7:02 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-15 10:11 [PATCH v6 0/9] Cleanup rootfs creation claudius.heine.ext
2019-05-15 10:11 ` [PATCH v6 1/9] isar-bootstrap-host: disable DISTRO_BOOTSTRAP_KEYS usage claudius.heine.ext
2019-05-21 16:56 ` Jan Kiszka
2019-05-22 6:37 ` Claudius Heine
2019-05-22 6:55 ` Maxim Yu. Osipov
2019-05-22 7:02 ` Jan Kiszka [this message]
2019-05-22 7:31 ` Claudius Heine
2019-05-22 11:35 ` Jan Kiszka
2019-05-22 11:39 ` Claudius Heine
2019-05-22 12:15 ` Jan Kiszka
2019-05-22 12:50 ` Jan Kiszka
2019-05-22 13:37 ` Claudius Heine
2019-05-22 19:10 ` Baurzhan Ismagulov
2019-05-23 9:22 ` Claudius Heine
2019-05-15 10:11 ` [PATCH v6 2/9] isar-bootstrap-helper: move 'HOST_ARCH' and 'HOST_DISTRO' to base.bbclass claudius.heine.ext
2019-05-15 10:11 ` [PATCH v6 3/9] move 'HOST_DISTRO_APT_SOURCES' from bootstrap-helper to isar-bootstrap claudius.heine.ext
2019-05-15 10:11 ` [PATCH v6 4/9] buildchroot.bbclass: only cross build if HOST_ARCH != DISTRO_ARCH claudius.heine.ext
2019-05-15 10:11 ` [PATCH v6 5/9] isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR claudius.heine.ext
2019-05-15 10:11 ` [PATCH v6 6/9] bitbake.conf: remove unneeded and differently used variables claudius.heine.ext
2019-05-15 10:11 ` [PATCH v6 7/9] image.bbclass: make IMAGE_ROOTFS overwritable claudius.heine.ext
2019-05-15 10:11 ` [PATCH v6 8/9] bitbake.conf: set default QEMU_ARCH variables claudius.heine.ext
2019-05-15 10:11 ` [PATCH v6 9/9] buildchroot/configscript: make creation of builder uid/gid idempotent claudius.heine.ext
2019-05-15 16:01 ` [PATCH v6 0/9] Cleanup rootfs creation Maxim Yu. Osipov
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=80d56fc5-00fa-5508-7fb1-976b4b5c61db@siemens.com \
--to=jan.kiszka@siemens.com \
--cc=ch@denx.de \
--cc=claudius.heine.ext@siemens.com \
--cc=isar-users@googlegroups.com \
--cc=mosipov@ilbers.de \
/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