wic: try to find bitbake using find_executable API
I forgot to cc the list. Unfortunately, this led to the commit going in with the reference to oe-classic.
Please people, get your terminology correct. The word soup in the wold is insane.
Philip
-------- Forwarded Message --------
Subject: Re: [OE-core] [wic][PATCH] wic: try to find bitbake using find_executable API
Date: Fri, 17 Apr 2015 10:32:08 -0400
From: Philip Balister <[email protected]>
To: Ed Bartosh <[email protected]>
Quoted 4 lines · Ed Bartosh
From: Ed Bartosh <[email protected]>
Current wic code was assuming that bitbake directory is on the same level as scripts, which is not the case for oe classic.
oe classic is the ancient meta-data.
I think you mean the assumption for people who do not install bitbake inside an openembedded-core checkout.
Philip
Quoted 59 lines
Using
find_executablebitbake location should be determined better as this API uss $PATH to search for executables.Fixes [YOCTO #7621]
Signed-off-by Ed Bartosh <[email protected]>Signed-off-by Ed Bartosh <[email protected]>--- scripts/wic | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-)scripts/wic +15
-5@@ -36,19 +36,25 @@ import os import sys import optparse import logging +from distutils import spawn # External modules scripts_path = os.path.abspath(os.path.dirname(__file__)) lib_path = scripts_path + '/lib' -bitbake_path = os.path.join(scripts_path, '../bitbake/lib') -sys.path = sys.path + [lib_path, bitbake_path] +sys.path.append(lib_path) + +bitbake_exe = spawn.find_executable('bitbake') +if bitbake_exe: + bitbake_path = os.path.join(os.path.dirname(bitbake_exe), '../lib') + sys.path.append(bitbake_path) + from bb import cookerdata + from bb.main import bitbake_main, BitBakeConfigParameters +else: + bitbake_main = None from image.help import * from image.engine import * -from bb import cookerdata -from bb.main import bitbake_main, BitBakeConfigParameters - def rootfs_dir_to_args(krootfs_dir): """ Get a rootfs_dir dict and serialize to string@@ -109,6 +115,10 @@ def wic_create_subcommand(args, usage_str): parser.print_help() sys.exit(1) + if options.build_rootfs and not bitbake_main: + logging.error("Can't build roofs as bitbake is not in the $PATH") + sys.exit(1) + if not options.image_name and not (options.rootfs_dir and options.bootimg_dir and options.kernel_dir and
Quoted 6 lines · Philip Balister
I forgot to cc the list. Unfortunately, this led to the commit going in with the reference to oe-classic.
Please people, get your terminology correct. The word soup in the wold is insane.
My bad. I've received your e-mail, but didn't have time to send the fix during weekend. Today I've noticed that my patch was already merged.
--
Regards,
Ed