All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-arago][master/kirkstone][PATCH] kexec-tools: drop the bbappend
@ 2023-06-06 21:51 rs
  2023-06-06 22:39 ` Denys Dmytriyenko
  0 siblings, 1 reply; 2+ messages in thread
From: rs @ 2023-06-06 21:51 UTC (permalink / raw)
  To: reatmon, denis, afd, detheridge; +Cc: meta-arago, Randolph Sapp

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.

 .../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



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [meta-arago][master/kirkstone][PATCH] kexec-tools: drop the bbappend
  2023-06-06 21:51 [meta-arago][master/kirkstone][PATCH] kexec-tools: drop the bbappend rs
@ 2023-06-06 22:39 ` Denys Dmytriyenko
  0 siblings, 0 replies; 2+ messages in thread
From: Denys Dmytriyenko @ 2023-06-06 22:39 UTC (permalink / raw)
  To: rs; +Cc: reatmon, afd, detheridge, meta-arago

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


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-06-06 22:40 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-06 21:51 [meta-arago][master/kirkstone][PATCH] kexec-tools: drop the bbappend rs
2023-06-06 22:39 ` Denys Dmytriyenko

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.