public inbox for docs@lists.yoctoproject.org
 help / color / mirror / Atom feed
From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
To: Alexander Kanavin <alex.kanavin@gmail.com>
Cc: "docs@lists.yoctoproject.org" <docs@lists.yoctoproject.org>,
	"Alexander Kanavin" <alex@linutronix.de>
Subject: RE: [docs] [PATCH 5/5] common-tasks.rst: rewrite the section about configuration templates
Date: Tue, 13 Sep 2022 23:59:13 +0000	[thread overview]
Message-ID: <0c6e4dbbf24d4221ba58543cfc48a498@axis.com> (raw)
In-Reply-To: <CANNYZj_mTijA18UCxpTVa9V0k7pPakqvAGBLWBS39UDBCHS8RA@mail.gmail.com>

> -----Original Message-----
> From: Alexander Kanavin <alex.kanavin@gmail.com>
> Sent: den 13 september 2022 19:27
> To: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> Cc: docs@lists.yoctoproject.org; Alexander Kanavin <alex@linutronix.de>
> Subject: Re: [docs] [PATCH 5/5] common-tasks.rst: rewrite the section
> about configuration templates
> 
> On Tue, 13 Sept 2022 at 19:38, Peter Kjellerstedt
> <peter.kjellerstedt@axis.com> wrote:
> > I do not agree with this sentiment. If you create and distribute a setup
> > similar to Poky (which we do), why would we not change .templateconf to
> > point at our templates? This has been the documented method and the one
> > at least we use for our distribution. Where have you gotten the notion
> > that this is only to be used by OE-Core & Poky? It does not make sense.
> 
> First we need to find out, where the documentation section that sets
> the idea is coming from. It was written by Scott R. in 2014:
> https://git.yoctoproject.org/yocto-docs/commit/?id=c07fb7082fe08387bbc546b2a23620dedc7127b8
> from
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=5895
> 
> I believe Scott misunderstood how the whole thing is supposed to work,
> and never road tested what he had written.

And I think he documented it exactly as it was intended to be used. See 
Paul Eggleton's comment:

  One way to do it is to set TEMPLATECONF (an environment variable, not a 
  bitbake variable) to point to a directory where the custom conf-notes.txt 
  can be found. In a repository this can be set within a .templateconf file - 
  hopefully we document that somewhere as well."

> You are free to continue to
> do the .templateconf rewrite, but I do not want to keep it in the
> docs.

Well, I would prefer to keep it in the docs since it is a documented 
feature and (at least was) the recommended way to do things. Maybe this 
has changed over the years, but if it is removed from the docs there is 
nothing stopping someone from removing it. And just because it does not 
match how you use Yocto, doesn't mean it isn't useful to others.

> It just isn't actually supported, unless you *also* provide a
> description and tools for 'creating a distribution setup similar to
> poky' and for doing the actual rewrite, and find someone else except
> you who's using it (I believe you do this with custom unpublished
> scripts?).

Nowadays we do, since we gave up on the limitations of static 
configurations in the layers. Our repo manifests decide which layers 
are fetched and those are the layers that will be added to the 
bblayers.conf.sample file.

> Besides, the only feature it gets you is not having to
> specify TEMPLATECONF when you have a default choice for it, which is
> useful, but should be made possible with the setup everyone *actually*
> uses, which is cloning poky, then adding layers next to it.

Well, that's what we do too using our repo manifests. It's just that repo 
has the support to copy files into the directories it sets up, which we 
use to copy in a .templateconf that overrides the one from poky. Anyone 
can do that if they have a defined set of layers they use.

> 
> Alex

//Peter


  reply	other threads:[~2022-09-13 23:59 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-11 17:26 [PATCH 1/5] ref-manual: correct the location of default configuration template Alexander Kanavin
2022-09-11 17:26 ` [PATCH 2/5] common-tasks.rst: drop the output of 'bitbake-layers --help' Alexander Kanavin
2022-09-20 13:42   ` [docs] " Michael Opdenacker
2022-09-11 17:26 ` [PATCH 3/5] common-tasks.rst: describe the newly added layer setup and template config commands Alexander Kanavin
2022-09-19 15:23   ` [docs] " Quentin Schulz
     [not found]     ` <6b495f9b-fbc3-7f91-19b3-0e1bb433e54a@bootlin.com>
2022-09-20 21:22       ` Alexander Kanavin
2022-09-11 17:26 ` [PATCH 4/5] common-tasks.rst: describe the layer setup tooling Alexander Kanavin
2022-09-19 15:34   ` [docs] " Quentin Schulz
2022-09-19 15:37   ` Quentin Schulz
2022-09-11 17:26 ` [PATCH 5/5] common-tasks.rst: rewrite the section about configuration templates Alexander Kanavin
2022-09-13 17:38   ` [docs] " Peter Kjellerstedt
2022-09-13 18:27     ` Alexander Kanavin
2022-09-13 23:59       ` Peter Kjellerstedt [this message]
2022-09-14  4:50         ` Alexander Kanavin
2022-09-20 20:46           ` Michael Opdenacker
2022-09-20 20:53             ` Alexander Kanavin
     [not found]             ` <1716ACEC6B725C1D.5502@lists.yoctoproject.org>
2022-09-20 21:04               ` Alexander Kanavin
2022-09-19 15:43   ` Quentin Schulz
2022-09-21  9:00     ` [PATCH v2] dev-manual: " michael.opdenacker
2022-09-20 13:36 ` [docs] [PATCH 1/5] ref-manual: correct the location of default configuration template Michael Opdenacker

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=0c6e4dbbf24d4221ba58543cfc48a498@axis.com \
    --to=peter.kjellerstedt@axis.com \
    --cc=alex.kanavin@gmail.com \
    --cc=alex@linutronix.de \
    --cc=docs@lists.yoctoproject.org \
    /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