On Wed, Apr 20, 2022 at 11:58 PM Russ Dill <[email protected]> wrote: > > The do_kernel_configcheck task requires a meta directory, normally > set by ${KMETA}. The meta directory is taken as a relative path > from ${S}: > > outfile = "{}/{}/cfg/mismatch.txt".format( s, kmeta ) > > However, when checking for the presence of ${KMETA} the current > working directory is searched. This will almost always fail and > "kgit --meta" is used instead. If the user does have a path in > their current working directory that matches the ${KMETA} > variable but the path is not present within the kernel source > directory, the build will fail if it tries to write config errors/ > warnings to that path. > > If ${KMETA} is not set, the same problem exists with the hard-coded > "meta" directory. > > Fix these issues by checking for ${KMETA} within ${S} rather than > the current working directory. Additionally, drop the hardcoded > backup directory "meta" as it hasn't been functioning and > probably has no users
Looks good to me. I continued to poke at this yesterday, and can't think of any users this will break .. and if it does, I'll be around to fix any fallout. Acked-by: Bruce Ashfield <[email protected]> > > Signed-off-by: Russ Dill <[email protected]> > --- > meta/classes/kernel-yocto.bbclass | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/meta/classes/kernel-yocto.bbclass > b/meta/classes/kernel-yocto.bbclass > index 1d5a8cdf29..4cb638864c 100644 > --- a/meta/classes/kernel-yocto.bbclass > +++ b/meta/classes/kernel-yocto.bbclass > @@ -521,15 +521,15 @@ python do_config_analysis() { > python do_kernel_configcheck() { > import re, string, sys, subprocess > > - # if KMETA isn't set globally by a recipe using this routine, we need to > - # set the default to 'meta'. Otherwise, kconf_check is not passed a valid > - # meta-series for processing > - kmeta = d.getVar("KMETA") or "meta" > - if not os.path.exists(kmeta): > - kmeta = subprocess.check_output(['kgit', '--meta'], > cwd=d.getVar('S')).decode('utf-8').rstrip() > - > s = d.getVar('S') > > + # if KMETA isn't set globally by a recipe using this routine, use kgit to > + # locate or create the meta directory. Otherwise, kconf_check is not > + # passed a valid meta-series for processing > + kmeta = d.getVar("KMETA") > + if not kmeta or not os.path.exists('{}/{}'.format(s,kmeta)): > + kmeta = subprocess.check_output(['kgit', '--meta'], > cwd=d.getVar('S')).decode('utf-8').rstrip() > + > env = os.environ.copy() > env['PATH'] = "%s:%s%s" % (d.getVar('PATH'), s, "/scripts/util/") > env['LD'] = d.getVar('KERNEL_LD') > -- > 2.25.1 > -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#164759): https://lists.openembedded.org/g/openembedded-core/message/164759 Mute This Topic: https://lists.openembedded.org/mt/90599712/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
