I just got few thousands of warnings thanks to this, they might be legitimate as write_bom_data and write_abi_xml_data are our own tasks added in https://github.com/webosose/meta-webosose/blob/master/meta-webos/classes/webos_base.bbclass very long time ago (based on fetchall task which was added in 2006 :)) - there might be better way to add them now, I'll check how to do it better or else I might need to disable this warning locally. Based on "utility-tasks: Drop fetchall and checkuriall tasks" I guess the right way to do the same now will be to convert our scripts to call it with --runall. It's pity that the warning is shown repeatedly. grep -c WARNING:.*dependent.task BUILD/log/cooker/m16p3/console-latest.log 30409 and that was only very small build not even finished yet: Currently 7 running tasks (1142 of 2514) .. WARNING: cross-localedef-native: dependent task do_deploy does not exist WARNING: cross-localedef-native: dependent task do_package_write_ipk does not exist WARNING: cross-localedef-native: dependent task do_deploy does not exist WARNING: cross-localedef-native: dependent task do_package_write_ipk does not exist WARNING: cross-localedef-native: dependent task write_bom_data does not exist WARNING: cross-localedef-native: dependent task write_abi_xml_data does not exist WARNING: autoconf-archive-native: dependent task do_deploy does not exist WARNING: autoconf-archive-native: dependent task do_package_write_ipk does not exist WARNING: autoconf-archive-native: dependent task do_deploy does not exist WARNING: autoconf-archive-native: dependent task do_package_write_ipk does not exist WARNING: autoconf-archive-native: dependent task write_bom_data does not exist WARNING: autoconf-archive-native: dependent task write_abi_xml_data does not exist WARNING: cross-localedef-native: dependent task do_deploy does not exist WARNING: cross-localedef-native: dependent task do_package_write_ipk does not exist WARNING: cross-localedef-native: dependent task do_deploy does not exist WARNING: cross-localedef-native: dependent task do_package_write_ipk does not exist WARNING: cross-localedef-native: dependent task write_bom_data does not exist WARNING: cross-localedef-native: dependent task write_abi_xml_data does not exist WARNING: cross-localedef-native: dependent task do_deploy does not exist WARNING: cross-localedef-native: dependent task do_package_write_ipk does not exist WARNING: cross-localedef-native: dependent task do_deploy does not exist WARNING: cross-localedef-native: dependent task do_package_write_ipk does not exist WARNING: cross-localedef-native: dependent task write_bom_data does not exist WARNING: cross-localedef-native: dependent task write_abi_xml_data does not exist WARNING: cross-localedef-native: dependent task do_deploy does not exist WARNING: cross-localedef-native: dependent task do_package_write_ipk does not exist WARNING: cross-localedef-native: dependent task do_deploy does not exist WARNING: cross-localedef-native: dependent task do_package_write_ipk does not exist WARNING: cross-localedef-native: dependent task write_bom_data does not exist WARNING: cross-localedef-native: dependent task write_abi_xml_data does not exist .. On Fri, Apr 26, 2019 at 4:49 AM Robert Yang wrote: > > > On 4/25/19 6:50 PM, Richard Purdie wrote: > > On Thu, 2019-04-25 at 18:01 +0800, Robert Yang wrote: > >> The following command is incorrect, but was ignored silently, that may > suprise > >> users: > >> > >> addtask task after task_not_existed > >> > >> This patch can check and warn for it. It would be better to also check > "before" > >> tasks, but there is no easier way to do it. > >> > >> [YOCTO #13282] > >> > >> Signed-off-by: Robert Yang > >> --- > >> bitbake/lib/bb/build.py | 3 +++ > >> 1 file changed, 3 insertions(+) > >> > >> diff --git a/bitbake/lib/bb/build.py b/bitbake/lib/bb/build.py > >> index 7571421..861e9a9 100644 > >> --- a/bitbake/lib/bb/build.py > >> +++ b/bitbake/lib/bb/build.py > >> @@ -815,6 +815,9 @@ def add_tasks(tasklist, d): > >> task_deps['parents'][task] = [] > >> if 'deps' in flags: > >> for dep in flags['deps']: > >> + # Check and warn for "addtask task after foo" while > foo does not exist > >> + if not dep in tasklist: > >> + bb.warn('%s: dependent task %s does not exist!' % > (d.getVar('PN'), dep)) > >> dep = d.expand(dep) > >> task_deps['parents'][task].append(dep) > > > > I can't help wonder if this change has races, depending on the order of > > the addtask and deltask expressions. Do most layers parse cleanly with > > this? > > The bb.build.add_tasks() is only called by ast.py's finalize(), I think > that > everything should be ready in finalize stage, and there should be no races. > I tested this with meta-oe, python perl, selinux and others, didn't see > any problems (unless I depended on an invalid task). > > > > > At one point bitbake did support "floating" invalid tasks, I can't > > remember if we changed that or not though. > > This warning is good for checking typos during development, I think that > user can call "bb.build.addtask()" dynamically in a function to fix the > warning > for floating tasks? > > // Robert > > > > > Cheers, > > > > Richard > > > > > -- > _______________________________________________ > bitbake-devel mailing list > bitbake-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/bitbake-devel >