[OE-core] [PATCH 1/1] waf.bbclass: Warn if waf doesn't exist

[Joshua Watt]

On Mar 20, 2018 18:33, "Martin Jansa" <martin.jansa at gmail.com> wrote:

see https://patchwork.openembedded.org/patch/147693/

and updated version (just rebased after other waf related changes were
merged):
http://git.openembedded.org/openembedded-core-contrib/
commit/?h=jansa/master-submitted&id=3f70b555e3f6033b807e03247605f3fdad868646

but it was rejected by Ross before, maybe now it won't be when it has Yocto
bug number in commit message..


Ah started my reply before I saw yours. I thought this looked familiar.
IIRC, the other patch was before we changed waf-samba.bbclass to no longer
inherit from waf.bbclass.

I didn't remember exactly why the previous one was rejected.... But looking
back I think Ross was right that inheriting waf.bbclass without actually
having waf should be an error (at some point). I was (perhaps incorrectly)
presuming that even if you got the FileNotFoundError, do_configure would
still fail. Is that correct? If not I think something strange is going on.



On Wed, Mar 21, 2018 at 12:07 AM, Amanda Brindle <amanda.r.brindle at intel.com
> wrote:

> Before, waf.bbclass would fail to catch FileNotFoundError. Now, it will
> catch this error and warn that waf doesn't exist.
>
> Fixes [YOCTO 12553]
>
> Signed-off-by: Amanda Brindle <amanda.r.brindle at intel.com>
> ---
>  meta/classes/waf.bbclass | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
> index f9a851d..5bd5913 100644
> --- a/meta/classes/waf.bbclass
> +++ b/meta/classes/waf.bbclass
> @@ -15,6 +15,8 @@ python waf_preconfigure() {
>              d.setVar("WAF_EXTRA_CONF", "--bindir=${bindir}
> --libdir=${libdir}")
>      except subprocess.CalledProcessError as e:
>          bb.warn("Unable to execute waf --version, exit code %d. Assuming
> waf version without bindir/libdir support." % e.returncode)
> +    except FileNotFoundError:
> +        bb.warn("waf does not exist in %s" % subsrcdir)
>  }
>
>  do_configure[prefuncs] += "waf_preconfigure"
> --
> 2.7.4
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>


--
_______________________________________________
Openembedded-core mailing list
Openembedded-core at lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20180320/bc4edc17/attachment-0002.html>