On Tue, 2024-09-03 at 07:15 +0100, Richard Purdie wrote:
> On Mon, 2024-09-02 at 22:31 +0200, Adrian Freihofer via
> lists.openembedded.org wrote:
> > From: Adrian Freihofer <adrian.freiho...@siemens.com>
> >
> > The kernel-yocto.bbclass defines some tasks tasks and it also adds
> > these
> > tasks to the SRCTREECOVEREDTASKS list. There is no need for devtool
> > to
> > duplicate this code and override what the kernel-yocto.bbclass
> > already
> > does.
> >
> > devtool modify generates a linux-yocto.6.6.bbappend containing:
> > SRCTREECOVEREDTASKS="\
> > do_fetch \
> > do_kernel_checkout \
> > do_kernel_configcheck \
> > do_unpack \
> > do_validate_branches \
> > "
> > do_patch[noexec] = "1"
> >
> > linux-yocto set SRCTREECOVEREDTASKS to
> > SRCTREECOVEREDTASKS="\
> > do_fetch \
> > do_kernel_checkout \
> > do_kernel_configcheck \
> > do_patch \
> > do_unpack \
> > do_validate_branches \
> > "
> >
> > The code in devtool modify is therefore considered as redundant and
> > removed.
> >
> > Signed-off-by: Adrian Freihofer <adrian.freiho...@siemens.com>
> > ---
> > scripts/lib/devtool/standard.py | 3 ---
> > 1 file changed, 3 deletions(-)
> >
> > diff --git a/scripts/lib/devtool/standard.py
> > b/scripts/lib/devtool/standard.py
> > index 312eb8ab506..b2e1a6ca3a5 100644
> > --- a/scripts/lib/devtool/standard.py
> > +++ b/scripts/lib/devtool/standard.py
> > @@ -952,9 +952,6 @@ def modify(args, config, basepath, workspace):
> > f.write('EXTERNALSRC_BUILD:pn-%s = "%s"\n' % (pn,
> > srctree))
> >
> > if bb.data.inherits_class('kernel', rd):
> > - f.write('SRCTREECOVEREDTASKS =
> > "do_validate_branches do_kernel_checkout '
> > - 'do_fetch do_unpack
> > do_kernel_configcheck"\n')
> > - f.write('\ndo_patch[noexec] = "1"\n')
> > f.write('\ndo_kernel_configme:prepend() {\n'
> > ' if [ -e ${S}/.config ]; then\n'
> > ' mv ${S}/.config
> > ${S}/.config.old\n'
>
>
> Note that "kernel" != "linux-yocto". Does a standard kernel still
> need fetch/unpack?
externalsrc.bbclass defaults to:
SRCTREECOVEREDTASKS ?= "do_patch do_unpack do_fetch"
https://git.yoctoproject.org/poky/tree/meta/classes/externalsrc.bbclass#n28
This should be fine for "kernel" != "linux-yocto".
I will send a v2 which mentions this in the commit message and fixes
the other finding with the uppercase PN.
Thank you for the review.
Adrian
>
> Cheers,
>
> Richard
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#204175):
https://lists.openembedded.org/g/openembedded-core/message/204175
Mute This Topic: https://lists.openembedded.org/mt/108234041/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
