All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Konrad Weihmann" <kweihmann@outlook.com>
To: bitbake-devel@lists.openembedded.org
Subject: Re: [PATCH] cooker: Drop support for BB_DANGLINGAPPENDS_WARNONLY
Date: Tue, 19 Nov 2024 04:56:29 -0800	[thread overview]
Message-ID: <6663.1732020989399254787@lists.openembedded.org> (raw)
In-Reply-To: <20241119103446.1983984-1-richard.purdie@linuxfoundation.org>

[-- Attachment #1: Type: text/plain, Size: 1384 bytes --]

It might be poorly designed but it helps a lot in the following situation
We have poky or any other "lower" layer providing a recipe-x_1.2.3.
Then we have a layer on top doing a recipe-x_1.2.3.bbappend (to applying backported patches or configuration options or anything version specific).
Now there is an update to the lower to version 1.2.4.
With the removal we get a failing build, with the WARN only we get just a warning, version 1.2.4 is picked.
Final decision if the build is good can be done in testing and/or post inspection.
With the functionality removed I would need to add a patch to the upper layer with a BBMASK
which then in any CI system I know would build with the older lower layer, basically removing the
entire append (which can't be right).
Alternative would be to schedule two changes at the same time (updated lower layer and patched upper layer).
To be honest I've not seen an automated CI system that works like that, leaving it all with
manual assembly of the inputs - requiring a ton of extra work.

Another alternative would be to loosen the bbappend mask to cover all version of the base recipe, but
that makes it impossible (without huge efforts) to apply patches for just a single version.
All in all the functionality might have been poorly designed but very helpful and
I don't see BBMASK as an equivalent replacement.

my 2 cents

[-- Attachment #2: Type: text/html, Size: 1557 bytes --]

      reply	other threads:[~2024-11-19 12:56 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-19 10:34 [PATCH] cooker: Drop support for BB_DANGLINGAPPENDS_WARNONLY Richard Purdie
2024-11-19 12:56 ` Konrad Weihmann [this message]

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=6663.1732020989399254787@lists.openembedded.org \
    --to=kweihmann@outlook.com \
    --cc=bitbake-devel@lists.openembedded.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.