On 25 July 2018 at 19:35, Joshua Watt <[email protected]> wrote:
> -    cmd = d.expand("cd ${S}; kconf_check --report -o ${S}/%s/cfg/ 
> ${B}/.config ${S} %s" % (kmeta,configs))
> -    ret, result = oe.utils.getstatusoutput("%s%s" % (pathprefix, cmd))
> +    try:
> +        configs = subprocess.check_output(['scc', '--configs', '-o', s + 
> '/.kernel-meta'], env=env).decode('utf-8')
> +    except subprocess.CalledProcessError:
> +        # Is this really non-fatal?
> +        pass
> +
> +    try:
> +        subprocess.check_call(['kconf_check', '--report', '-o',
> +                '%s/%s/cfg' % (s, kmeta), d.getVar('B') + '/.config', s, 
> configs], cwd=s, env=env)
> +    except subprocess.CalledProcessError:
> +        # Is this really non-fatal?
> +        pass

CCing Bruce for his feedback.  Either one or both of these calls can
fail without it being fatal which should be documented, or it
shouldn't be catching the exceptions.

Ross
-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to