All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denis@denix.org>
To: rs@ti.com
Cc: reatmon@ti.com, afd@ti.com, detheridge@ti.com,
	meta-arago@lists.yoctoproject.org
Subject: Re: [meta-arago][master/kirkstone][PATCH] kexec-tools: drop the bbappend
Date: Tue, 6 Jun 2023 18:39:43 -0400	[thread overview]
Message-ID: <20230606223943.GV9226@denix.org> (raw)
In-Reply-To: <20230606215108.2655321-1-rs@ti.com>

On Tue, Jun 06, 2023 at 04:51:08PM -0500, rs@ti.com wrote:
> From: Randolph Sapp <rs@ti.com>
> 
> This is a do-nothing bbappend. On top of that, it always expects the
> sysvinit script to be present, which is not supposed to be true in our
> default config. Drop the append entirely.
> 
> Signed-off-by: Randolph Sapp <rs@ti.com>
> ---
> 
> This is a prereq for the following patch apparently:
> https://lists.yoctoproject.org/g/meta-arago/message/14498
> 
> I tested that patch against core build targets not expecting us to do
> something this weird in meta-arago.

Heh, that was a weird one indeed - I was being asked to apply hacks and weird 
fixes w/o much explanation, so I couldn't even put anything in the commit log:
https://git.yoctoproject.org/meta-arago/log/meta-arago-distro/recipes-kernel/kexec/

Some dark history... That's why meta-arago-distro was often a dumping ground 
of hacks and kludges :(


>  .../recipes-kernel/kexec/kexec-tools_%.bbappend            | 7 -------
>  1 file changed, 7 deletions(-)
>  delete mode 100644 meta-arago-distro/recipes-kernel/kexec/kexec-tools_%.bbappend
> 
> diff --git a/meta-arago-distro/recipes-kernel/kexec/kexec-tools_%.bbappend b/meta-arago-distro/recipes-kernel/kexec/kexec-tools_%.bbappend
> deleted file mode 100644
> index 88786124..00000000
> --- a/meta-arago-distro/recipes-kernel/kexec/kexec-tools_%.bbappend
> +++ /dev/null
> @@ -1,7 +0,0 @@
> -FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:"
> -
> -PR:append = ".arago2"
> -
> -do_install:append() {
> -	chmod -x ${D}${sysconfdir}/init.d/kdump
> -}
> -- 
> 2.40.1


      reply	other threads:[~2023-06-06 22:40 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-06 21:51 [meta-arago][master/kirkstone][PATCH] kexec-tools: drop the bbappend rs
2023-06-06 22:39 ` Denys Dmytriyenko [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=20230606223943.GV9226@denix.org \
    --to=denis@denix.org \
    --cc=afd@ti.com \
    --cc=detheridge@ti.com \
    --cc=meta-arago@lists.yoctoproject.org \
    --cc=reatmon@ti.com \
    --cc=rs@ti.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 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.