* [PATCH] waf.bbclass: check that waf exists before trying to execute it
@ 2018-01-30 8:30 Martin Jansa
2018-01-30 12:03 ` Burton, Ross
0 siblings, 1 reply; 14+ messages in thread
From: Martin Jansa @ 2018-01-30 8:30 UTC (permalink / raw)
To: openembedded-core
* there are some recipes like jack and other which inherit waf-samba
which inherits waf, but they don't use waf for configure, strange isn't
it?
* fixes:
WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf --version, exit code 127. Assuming waf version without bindir/libdir support.
WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --version, exit code 127. Assuming waf version without bindir/libdir support.
WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf --version, exit code 127. Assuming waf version without bindir/libdir
support.
WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure: Unable to execute waf --version, exit code 1. Assuming waf version without bindir/libdir support.
WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf --version, exit code 127. Assuming waf version without bindir/libdir support.
WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf --version, exit code 127. Assuming waf version without bindir/libdir support.
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
meta/classes/waf.bbclass | 3 +++
1 file changed, 3 insertions(+)
diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
index c3e744e5de..eec4d9496d 100644
--- a/meta/classes/waf.bbclass
+++ b/meta/classes/waf.bbclass
@@ -29,6 +29,9 @@ python waf_preconfigure() {
from distutils.version import StrictVersion
srcsubdir = d.getVar('S')
wafbin = os.path.join(srcsubdir, 'waf')
+ if not os.path.isfile(wafbin):
+ bb.note("%s doesn't exist, cannot check whether it supports --bindir/--libdir" % wafbin)
+ return
status, result = oe.utils.getstatusoutput(wafbin + " --version")
if status != 0:
bb.warn("Unable to execute waf --version, exit code %d. Assuming waf version without bindir/libdir support." % status)
--
2.15.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-30 8:30 [PATCH] waf.bbclass: check that waf exists before trying to execute it Martin Jansa
@ 2018-01-30 12:03 ` Burton, Ross
2018-01-30 12:26 ` Martin Jansa
0 siblings, 1 reply; 14+ messages in thread
From: Burton, Ross @ 2018-01-30 12:03 UTC (permalink / raw)
To: Martin Jansa; +Cc: OE-core
[-- Attachment #1: Type: text/plain, Size: 2346 bytes --]
Why would they do that? Shouldn't it be a fatal bug if waf isn't found?
On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail.com> wrote:
> * there are some recipes like jack and other which inherit waf-samba
> which inherits waf, but they don't use waf for configure, strange isn't
> it?
>
> * fixes:
> WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf --version,
> exit code 127. Assuming waf version without bindir/libdir support.
> WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --version,
> exit code 127. Assuming waf version without bindir/libdir support.
> WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf --version,
> exit code 127. Assuming waf version without bindir/libdir
> support.
> WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure: Unable to
> execute waf --version, exit code 1. Assuming waf version without
> bindir/libdir support.
> WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf
> --version, exit code 127. Assuming waf version without bindir/libdir
> support.
> WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf --version,
> exit code 127. Assuming waf version without bindir/libdir support.
>
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
> meta/classes/waf.bbclass | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
> index c3e744e5de..eec4d9496d 100644
> --- a/meta/classes/waf.bbclass
> +++ b/meta/classes/waf.bbclass
> @@ -29,6 +29,9 @@ python waf_preconfigure() {
> from distutils.version import StrictVersion
> srcsubdir = d.getVar('S')
> wafbin = os.path.join(srcsubdir, 'waf')
> + if not os.path.isfile(wafbin):
> + bb.note("%s doesn't exist, cannot check whether it supports
> --bindir/--libdir" % wafbin)
> + return
> status, result = oe.utils.getstatusoutput(wafbin + " --version")
> if status != 0:
> bb.warn("Unable to execute waf --version, exit code %d. Assuming
> waf version without bindir/libdir support." % status)
> --
> 2.15.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
[-- Attachment #2: Type: text/html, Size: 3100 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-30 12:03 ` Burton, Ross
@ 2018-01-30 12:26 ` Martin Jansa
2018-01-30 14:30 ` Joshua Watt
2018-01-30 22:41 ` Andreas Müller
0 siblings, 2 replies; 14+ messages in thread
From: Martin Jansa @ 2018-01-30 12:26 UTC (permalink / raw)
To: Burton, Ross; +Cc: Andreas Müller, Joe Slater, OE-core
[-- Attachment #1: Type: text/plain, Size: 3052 bytes --]
+Andreas who added waf inherit in jack in:
commit 0f8f1973e229a0799098bc2334416fec8c608b50
Author: Andreas Müller <schnitzeltony@googlemail.com>
Date: Sat Oct 8 12:03:32 2016 +0200
jack: swap to jack2
+Bian who added waf samba in:
commit 962e346df99696a10657a2ea7b2166dfaf290289
Author: Bian Naimeng <biannm@cn.fujitsu.com>
Date: Tue Jun 16 14:18:25 2015 +0900
waf-samba: add new class
waf-samba is a build system used by samba related software
+Joe and Joe and who maintains samba and related recipes + meta-networking
On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@intel.com> wrote:
> Why would they do that? Shouldn't it be a fatal bug if waf isn't found?
>
> On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail.com> wrote:
>
>> * there are some recipes like jack and other which inherit waf-samba
>> which inherits waf, but they don't use waf for configure, strange isn't
>> it?
>>
>> * fixes:
>> WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf --version,
>> exit code 127. Assuming waf version without bindir/libdir support.
>> WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --version,
>> exit code 127. Assuming waf version without bindir/libdir support.
>> WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf
>> --version, exit code 127. Assuming waf version without bindir/libdir
>> support.
>> WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure: Unable to
>> execute waf --version, exit code 1. Assuming waf version without
>> bindir/libdir support.
>> WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf
>> --version, exit code 127. Assuming waf version without bindir/libdir
>> support.
>> WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf --version,
>> exit code 127. Assuming waf version without bindir/libdir support.
>>
>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>> ---
>> meta/classes/waf.bbclass | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
>> index c3e744e5de..eec4d9496d 100644
>> --- a/meta/classes/waf.bbclass
>> +++ b/meta/classes/waf.bbclass
>> @@ -29,6 +29,9 @@ python waf_preconfigure() {
>> from distutils.version import StrictVersion
>> srcsubdir = d.getVar('S')
>> wafbin = os.path.join(srcsubdir, 'waf')
>> + if not os.path.isfile(wafbin):
>> + bb.note("%s doesn't exist, cannot check whether it supports
>> --bindir/--libdir" % wafbin)
>> + return
>> status, result = oe.utils.getstatusoutput(wafbin + " --version")
>> if status != 0:
>> bb.warn("Unable to execute waf --version, exit code %d. Assuming
>> waf version without bindir/libdir support." % status)
>> --
>> 2.15.1
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>
>
[-- Attachment #2: Type: text/html, Size: 4528 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-30 12:26 ` Martin Jansa
@ 2018-01-30 14:30 ` Joshua Watt
2018-01-30 22:41 ` Andreas Müller
1 sibling, 0 replies; 14+ messages in thread
From: Joshua Watt @ 2018-01-30 14:30 UTC (permalink / raw)
To: Martin Jansa, Burton, Ross; +Cc: Andreas Müller, Joe Slater, OE-core
[-- Attachment #1: Type: text/plain, Size: 3911 bytes --]
On Tue, 2018-01-30 at 13:26 +0100, Martin Jansa wrote:
> +Andreas who added waf inherit in jack in:commit
> 0f8f1973e229a0799098bc2334416fec8c608b50
> Author: Andreas Müller <schnitzeltony@googlemail.com>
> Date: Sat Oct 8 12:03:32 2016 +0200
>
> jack: swap to jack2
>
> +Bian who added waf samba in:
> commit 962e346df99696a10657a2ea7b2166dfaf290289
> Author: Bian Naimeng <biannm@cn.fujitsu.com>
> Date: Tue Jun 16 14:18:25 2015 +0900
>
> waf-samba: add new class
>
> waf-samba is a build system used by samba related software
waf-samba looks very special purpose and not how I would expect a
normal waf workflow to work at all.... maybe it shouldn't inherit from
waf.bbclass, especially since all it is actually using is
get_waf_parallel_make() ?
> +Joe and Joe and who maintains samba and related recipes + meta-
> networking
>
>
> On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@intel.com>
> wrote:
> > Why would they do that? Shouldn't it be a fatal bug if waf isn't
> > found?
> > On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail.com>
> > wrote:
> > > * there are some recipes like jack and other which inherit waf-
> > > samba
> > >
> > > which inherits waf, but they don't use waf for configure, strange
> > > isn't
> > >
> > > it?
> > >
> > >
> > >
> > > * fixes:
> > >
> > > WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf --
> > > version, exit code 127. Assuming waf version without
> > > bindir/libdir support.
> > >
> > > WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --
> > > version, exit code 127. Assuming waf version without
> > > bindir/libdir support.
> > >
> > > WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf
> > > --version, exit code 127. Assuming waf version without
> > > bindir/libdir
> > >
> > > support.
> > >
> > > WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure:
> > > Unable to execute waf --version, exit code 1. Assuming waf
> > > version without bindir/libdir support.
> > >
> > > WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf
> > > --version, exit code 127. Assuming waf version without
> > > bindir/libdir support.
> > >
> > > WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf --
> > > version, exit code 127. Assuming waf version without
> > > bindir/libdir support.
> > >
> > >
> > >
> > > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > >
> > > ---
> > >
> > > meta/classes/waf.bbclass | 3 +++
> > >
> > > 1 file changed, 3 insertions(+)
> > >
> > >
> > >
> > > diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
> > >
> > > index c3e744e5de..eec4d9496d 100644
> > >
> > > --- a/meta/classes/waf.bbclass
> > >
> > > +++ b/meta/classes/waf.bbclass
> > >
> > > @@ -29,6 +29,9 @@ python waf_preconfigure() {
> > >
> > > from distutils.version import StrictVersion
> > >
> > > srcsubdir = d.getVar('S')
> > >
> > > wafbin = os.path.join(srcsubdir, 'waf')
> > >
> > > + if not os.path.isfile(wafbin):
> > >
> > > + bb.note("%s doesn't exist, cannot check whether it
> > > supports --bindir/--libdir" % wafbin)
> > >
> > > + return
> > >
> > > status, result = oe.utils.getstatusoutput(wafbin + " --
> > > version")
> > >
> > > if status != 0:
> > >
> > > bb.warn("Unable to execute waf --version, exit code %d.
> > > Assuming waf version without bindir/libdir support." % status)
> > >
> > >
> > > --
> > >
> > > 2.15.1
> > >
> > >
> > >
> > > --
> > >
> > > _______________________________________________
> > >
> > > Openembedded-core mailing list
> > >
> > > Openembedded-core@lists.openembedded.org
> > >
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> > >
[-- Attachment #2: Type: text/html, Size: 5248 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-30 12:26 ` Martin Jansa
2018-01-30 14:30 ` Joshua Watt
@ 2018-01-30 22:41 ` Andreas Müller
2018-01-30 23:38 ` Martin Jansa
1 sibling, 1 reply; 14+ messages in thread
From: Andreas Müller @ 2018-01-30 22:41 UTC (permalink / raw)
To: Martin Jansa; +Cc: Andreas Müller, OE-core, Joe Slater
[-- Attachment #1: Type: text/plain, Size: 3401 bytes --]
On Tuesday, January 30, 2018, Martin Jansa <martin.jansa@gmail.com> wrote:
> +Andreas who added waf inherit in jack in:
> commit 0f8f1973e229a0799098bc2334416fec8c608b50
> Author: Andreas Müller <schnitzeltony@googlemail.com>
> Date: Sat Oct 8 12:03:32 2016 +0200
>
> jack: swap to jack2
>
As far as I can see is jack2 configured with waf. Don't remember why I
chose waf-samba. Because I am on business trip I cannot take care before
next week.
Andreas
>
> +Bian who added waf samba in:
> commit 962e346df99696a10657a2ea7b2166dfaf290289
> Author: Bian Naimeng <biannm@cn.fujitsu.com>
> Date: Tue Jun 16 14:18:25 2015 +0900
>
> waf-samba: add new class
>
> waf-samba is a build system used by samba related software
>
> +Joe and Joe and who maintains samba and related recipes + meta-networking
>
>
> On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@intel.com>
> wrote:
>
>> Why would they do that? Shouldn't it be a fatal bug if waf isn't found?
>>
>> On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail.com> wrote:
>>
>>> * there are some recipes like jack and other which inherit waf-samba
>>> which inherits waf, but they don't use waf for configure, strange isn't
>>> it?
>>>
>>> * fixes:
>>> WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf --version,
>>> exit code 127. Assuming waf version without bindir/libdir support.
>>> WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --version,
>>> exit code 127. Assuming waf version without bindir/libdir support.
>>> WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf
>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>> support.
>>> WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure: Unable to
>>> execute waf --version, exit code 1. Assuming waf version without
>>> bindir/libdir support.
>>> WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf
>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>> support.
>>> WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf --version,
>>> exit code 127. Assuming waf version without bindir/libdir support.
>>>
>>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>>> ---
>>> meta/classes/waf.bbclass | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>> diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
>>> index c3e744e5de..eec4d9496d 100644
>>> --- a/meta/classes/waf.bbclass
>>> +++ b/meta/classes/waf.bbclass
>>> @@ -29,6 +29,9 @@ python waf_preconfigure() {
>>> from distutils.version import StrictVersion
>>> srcsubdir = d.getVar('S')
>>> wafbin = os.path.join(srcsubdir, 'waf')
>>> + if not os.path.isfile(wafbin):
>>> + bb.note("%s doesn't exist, cannot check whether it supports
>>> --bindir/--libdir" % wafbin)
>>> + return
>>> status, result = oe.utils.getstatusoutput(wafbin + " --version")
>>> if status != 0:
>>> bb.warn("Unable to execute waf --version, exit code %d.
>>> Assuming waf version without bindir/libdir support." % status)
>>> --
>>> 2.15.1
>>>
>>> --
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>>
>>
>>
>
[-- Attachment #2: Type: text/html, Size: 5088 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-30 22:41 ` Andreas Müller
@ 2018-01-30 23:38 ` Martin Jansa
2018-01-30 23:39 ` Martin Jansa
0 siblings, 1 reply; 14+ messages in thread
From: Martin Jansa @ 2018-01-30 23:38 UTC (permalink / raw)
To: Andreas Müller; +Cc: Andreas Müller, OE-core, Joe Slater
[-- Attachment #1: Type: text/plain, Size: 4528 bytes --]
Thanks Andreas,
jack is the only one which inherits waf directly not through waf-samba.
More surprisingly it also has ${S}/waf it seems:
https://github.com/jackaudio/jack2/blob/c1647819eed6d11f94b21981d9c869629299f357/waf
So the WARNING: shown for jack probably had different reason than the rest
of them and that is:
jack/1.9.12-r0 $ jack/1.9.12-r0/git/waf --version
Error: run waf-light from a folder containing waflib
it doesn't like running waf from outside ${S} and waf_preconfigure is
executed from ${TOPDIR} before do_configure switches to ${S}
I don't know how common this situation is, if all waf scripts are like
that, this one (after "cd ../jack/1.9.12-r0/git") says:
waf 1.8.17 (cd7579a727d1b390bf9cbf111c1b20e811370bc0)
so I'll leave that to original author to figure it out, I don't use waf
anywhere, so I cannot properly test the use case he was trying to fix.
Regards,
On Tue, Jan 30, 2018 at 11:41 PM, Andreas Müller <schnitzeltony@gmail.com>
wrote:
>
>
> On Tuesday, January 30, 2018, Martin Jansa <martin.jansa@gmail.com> wrote:
>
>> +Andreas who added waf inherit in jack in:
>> commit 0f8f1973e229a0799098bc2334416fec8c608b50
>> Author: Andreas Müller <schnitzeltony@googlemail.com>
>> Date: Sat Oct 8 12:03:32 2016 +0200
>>
>> jack: swap to jack2
>>
> As far as I can see is jack2 configured with waf. Don't remember why I
> chose waf-samba. Because I am on business trip I cannot take care before
> next week.
>
> Andreas
>
>>
>> +Bian who added waf samba in:
>> commit 962e346df99696a10657a2ea7b2166dfaf290289
>> Author: Bian Naimeng <biannm@cn.fujitsu.com>
>> Date: Tue Jun 16 14:18:25 2015 +0900
>>
>> waf-samba: add new class
>>
>> waf-samba is a build system used by samba related software
>>
>> +Joe and Joe and who maintains samba and related recipes + meta-networking
>>
>>
>> On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@intel.com>
>> wrote:
>>
>>> Why would they do that? Shouldn't it be a fatal bug if waf isn't found?
>>>
>>> On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail.com>
>>> wrote:
>>>
>>>> * there are some recipes like jack and other which inherit waf-samba
>>>> which inherits waf, but they don't use waf for configure, strange isn't
>>>> it?
>>>>
>>>> * fixes:
>>>> WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf
>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>> support.
>>>> WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --version,
>>>> exit code 127. Assuming waf version without bindir/libdir support.
>>>> WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf
>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>> support.
>>>> WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure: Unable to
>>>> execute waf --version, exit code 1. Assuming waf version without
>>>> bindir/libdir support.
>>>> WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf
>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>> support.
>>>> WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf
>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>> support.
>>>>
>>>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>>>> ---
>>>> meta/classes/waf.bbclass | 3 +++
>>>> 1 file changed, 3 insertions(+)
>>>>
>>>> diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
>>>> index c3e744e5de..eec4d9496d 100644
>>>> --- a/meta/classes/waf.bbclass
>>>> +++ b/meta/classes/waf.bbclass
>>>> @@ -29,6 +29,9 @@ python waf_preconfigure() {
>>>> from distutils.version import StrictVersion
>>>> srcsubdir = d.getVar('S')
>>>> wafbin = os.path.join(srcsubdir, 'waf')
>>>> + if not os.path.isfile(wafbin):
>>>> + bb.note("%s doesn't exist, cannot check whether it supports
>>>> --bindir/--libdir" % wafbin)
>>>> + return
>>>> status, result = oe.utils.getstatusoutput(wafbin + " --version")
>>>> if status != 0:
>>>> bb.warn("Unable to execute waf --version, exit code %d.
>>>> Assuming waf version without bindir/libdir support." % status)
>>>> --
>>>> 2.15.1
>>>>
>>>> --
>>>> _______________________________________________
>>>> Openembedded-core mailing list
>>>> Openembedded-core@lists.openembedded.org
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>>>
>>>
>>>
>>
[-- Attachment #2: Type: text/html, Size: 7315 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-30 23:38 ` Martin Jansa
@ 2018-01-30 23:39 ` Martin Jansa
2018-01-31 2:30 ` Joshua Watt
2018-01-31 17:04 ` Joshua Watt
0 siblings, 2 replies; 14+ messages in thread
From: Martin Jansa @ 2018-01-30 23:39 UTC (permalink / raw)
To: Andreas Müller, Stefan Agner
Cc: Andreas Müller, OE-core, Joe Slater
[-- Attachment #1: Type: text/plain, Size: 4780 bytes --]
+Stefan
On Wed, Jan 31, 2018 at 12:38 AM, Martin Jansa <martin.jansa@gmail.com>
wrote:
> Thanks Andreas,
>
> jack is the only one which inherits waf directly not through waf-samba.
>
> More surprisingly it also has ${S}/waf it seems:
> https://github.com/jackaudio/jack2/blob/c1647819eed6d11f94b21981d9c869
> 629299f357/waf
>
> So the WARNING: shown for jack probably had different reason than the rest
> of them and that is:
>
> jack/1.9.12-r0 $ jack/1.9.12-r0/git/waf --version
> Error: run waf-light from a folder containing waflib
>
> it doesn't like running waf from outside ${S} and waf_preconfigure is
> executed from ${TOPDIR} before do_configure switches to ${S}
>
> I don't know how common this situation is, if all waf scripts are like
> that, this one (after "cd ../jack/1.9.12-r0/git") says:
> waf 1.8.17 (cd7579a727d1b390bf9cbf111c1b20e811370bc0)
>
> so I'll leave that to original author to figure it out, I don't use waf
> anywhere, so I cannot properly test the use case he was trying to fix.
>
> Regards,
>
>
>
> On Tue, Jan 30, 2018 at 11:41 PM, Andreas Müller <schnitzeltony@gmail.com>
> wrote:
>
>>
>>
>> On Tuesday, January 30, 2018, Martin Jansa <martin.jansa@gmail.com>
>> wrote:
>>
>>> +Andreas who added waf inherit in jack in:
>>> commit 0f8f1973e229a0799098bc2334416fec8c608b50
>>> Author: Andreas Müller <schnitzeltony@googlemail.com>
>>> Date: Sat Oct 8 12:03:32 2016 +0200
>>>
>>> jack: swap to jack2
>>>
>> As far as I can see is jack2 configured with waf. Don't remember why I
>> chose waf-samba. Because I am on business trip I cannot take care before
>> next week.
>>
>> Andreas
>>
>>>
>>> +Bian who added waf samba in:
>>> commit 962e346df99696a10657a2ea7b2166dfaf290289
>>> Author: Bian Naimeng <biannm@cn.fujitsu.com>
>>> Date: Tue Jun 16 14:18:25 2015 +0900
>>>
>>> waf-samba: add new class
>>>
>>> waf-samba is a build system used by samba related software
>>>
>>> +Joe and Joe and who maintains samba and related recipes +
>>> meta-networking
>>>
>>>
>>> On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@intel.com>
>>> wrote:
>>>
>>>> Why would they do that? Shouldn't it be a fatal bug if waf isn't found?
>>>>
>>>> On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail.com>
>>>> wrote:
>>>>
>>>>> * there are some recipes like jack and other which inherit waf-samba
>>>>> which inherits waf, but they don't use waf for configure, strange isn't
>>>>> it?
>>>>>
>>>>> * fixes:
>>>>> WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf
>>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>>> support.
>>>>> WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --version,
>>>>> exit code 127. Assuming waf version without bindir/libdir support.
>>>>> WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf
>>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>>> support.
>>>>> WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure: Unable to
>>>>> execute waf --version, exit code 1. Assuming waf version without
>>>>> bindir/libdir support.
>>>>> WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf
>>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>>> support.
>>>>> WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf
>>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>>> support.
>>>>>
>>>>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>>>>> ---
>>>>> meta/classes/waf.bbclass | 3 +++
>>>>> 1 file changed, 3 insertions(+)
>>>>>
>>>>> diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
>>>>> index c3e744e5de..eec4d9496d 100644
>>>>> --- a/meta/classes/waf.bbclass
>>>>> +++ b/meta/classes/waf.bbclass
>>>>> @@ -29,6 +29,9 @@ python waf_preconfigure() {
>>>>> from distutils.version import StrictVersion
>>>>> srcsubdir = d.getVar('S')
>>>>> wafbin = os.path.join(srcsubdir, 'waf')
>>>>> + if not os.path.isfile(wafbin):
>>>>> + bb.note("%s doesn't exist, cannot check whether it supports
>>>>> --bindir/--libdir" % wafbin)
>>>>> + return
>>>>> status, result = oe.utils.getstatusoutput(wafbin + " --version")
>>>>> if status != 0:
>>>>> bb.warn("Unable to execute waf --version, exit code %d.
>>>>> Assuming waf version without bindir/libdir support." % status)
>>>>> --
>>>>> 2.15.1
>>>>>
>>>>> --
>>>>> _______________________________________________
>>>>> Openembedded-core mailing list
>>>>> Openembedded-core@lists.openembedded.org
>>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>>>>
>>>>
>>>>
>>>
>
[-- Attachment #2: Type: text/html, Size: 7848 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-30 23:39 ` Martin Jansa
@ 2018-01-31 2:30 ` Joshua Watt
2018-01-31 17:04 ` Joshua Watt
1 sibling, 0 replies; 14+ messages in thread
From: Joshua Watt @ 2018-01-31 2:30 UTC (permalink / raw)
To: Martin Jansa; +Cc: Andreas Müller, Joe Slater, Stefan Agner, OE-core
[-- Attachment #1: Type: text/plain, Size: 5137 bytes --]
On Jan 30, 2018 17:40, "Martin Jansa" <martin.jansa@gmail.com> wrote:
+Stefan
On Wed, Jan 31, 2018 at 12:38 AM, Martin Jansa <martin.jansa@gmail.com>
wrote:
> Thanks Andreas,
>
> jack is the only one which inherits waf directly not through waf-samba.
>
> More surprisingly it also has ${S}/waf it seems:
> https://github.com/jackaudio/jack2/blob/c1647819eed6d11f94b2
> 1981d9c869629299f357/waf
>
> So the WARNING: shown for jack probably had different reason than the rest
> of them and that is:
>
> jack/1.9.12-r0 $ jack/1.9.12-r0/git/waf --version
> Error: run waf-light from a folder containing waflib
>
> it doesn't like running waf from outside ${S} and waf_preconfigure is
> executed from ${TOPDIR} before do_configure switches to ${S}
>
> I don't know how common this situation is, if all waf scripts are like
> that, this one (after "cd ../jack/1.9.12-r0/git") says:
> waf 1.8.17 (cd7579a727d1b390bf9cbf111c1b20e811370bc0)
>
> so I'll leave that to original author to figure it out, I don't use waf
> anywhere, so I cannot properly test the use case he was trying to fix.
>
> Regards,
>
I have a lot of waf builds, I'll test them with the latest waf.bbclass
tomorrow.
>
>
> On Tue, Jan 30, 2018 at 11:41 PM, Andreas Müller <schnitzeltony@gmail.com>
> wrote:
>
>>
>>
>> On Tuesday, January 30, 2018, Martin Jansa <martin.jansa@gmail.com>
>> wrote:
>>
>>> +Andreas who added waf inherit in jack in:
>>> commit 0f8f1973e229a0799098bc2334416fec8c608b50
>>> Author: Andreas Müller <schnitzeltony@googlemail.com>
>>> Date: Sat Oct 8 12:03:32 2016 +0200
>>>
>>> jack: swap to jack2
>>>
>> As far as I can see is jack2 configured with waf. Don't remember why I
>> chose waf-samba. Because I am on business trip I cannot take care before
>> next week.
>>
>> Andreas
>>
>>>
>>> +Bian who added waf samba in:
>>> commit 962e346df99696a10657a2ea7b2166dfaf290289
>>> Author: Bian Naimeng <biannm@cn.fujitsu.com>
>>> Date: Tue Jun 16 14:18:25 2015 +0900
>>>
>>> waf-samba: add new class
>>>
>>> waf-samba is a build system used by samba related software
>>>
>>> +Joe and Joe and who maintains samba and related recipes +
>>> meta-networking
>>>
>>>
>>> On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@intel.com>
>>> wrote:
>>>
>>>> Why would they do that? Shouldn't it be a fatal bug if waf isn't found?
>>>>
>>>> On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail.com>
>>>> wrote:
>>>>
>>>>> * there are some recipes like jack and other which inherit waf-samba
>>>>> which inherits waf, but they don't use waf for configure, strange isn't
>>>>> it?
>>>>>
>>>>> * fixes:
>>>>> WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf
>>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>>> support.
>>>>> WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --version,
>>>>> exit code 127. Assuming waf version without bindir/libdir support.
>>>>> WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf
>>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>>> support.
>>>>> WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure: Unable to
>>>>> execute waf --version, exit code 1. Assuming waf version without
>>>>> bindir/libdir support.
>>>>> WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf
>>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>>> support.
>>>>> WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf
>>>>> --version, exit code 127. Assuming waf version without bindir/libdir
>>>>> support.
>>>>>
>>>>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>>>>> ---
>>>>> meta/classes/waf.bbclass | 3 +++
>>>>> 1 file changed, 3 insertions(+)
>>>>>
>>>>> diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
>>>>> index c3e744e5de..eec4d9496d 100644
>>>>> --- a/meta/classes/waf.bbclass
>>>>> +++ b/meta/classes/waf.bbclass
>>>>> @@ -29,6 +29,9 @@ python waf_preconfigure() {
>>>>> from distutils.version import StrictVersion
>>>>> srcsubdir = d.getVar('S')
>>>>> wafbin = os.path.join(srcsubdir, 'waf')
>>>>> + if not os.path.isfile(wafbin):
>>>>> + bb.note("%s doesn't exist, cannot check whether it supports
>>>>> --bindir/--libdir" % wafbin)
>>>>> + return
>>>>> status, result = oe.utils.getstatusoutput(wafbin + " --version")
>>>>> if status != 0:
>>>>> bb.warn("Unable to execute waf --version, exit code %d.
>>>>> Assuming waf version without bindir/libdir support." % status)
>>>>> --
>>>>> 2.15.1
>>>>>
>>>>> --
>>>>> _______________________________________________
>>>>> Openembedded-core mailing list
>>>>> Openembedded-core@lists.openembedded.org
>>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>>>>
>>>>
>>>>
>>>
>
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
[-- Attachment #2: Type: text/html, Size: 9339 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-30 23:39 ` Martin Jansa
2018-01-31 2:30 ` Joshua Watt
@ 2018-01-31 17:04 ` Joshua Watt
2018-01-31 17:07 ` Martin Jansa
1 sibling, 1 reply; 14+ messages in thread
From: Joshua Watt @ 2018-01-31 17:04 UTC (permalink / raw)
To: Martin Jansa, Andreas Müller, Stefan Agner
Cc: Andreas Müller, Joe Slater, OE-core
[-- Attachment #1: Type: text/plain, Size: 6235 bytes --]
Sorry, I wasn't sure if I should make my patch in-reply-to this
thread....
I think I fixed this correctly in http://lists.openembedded.org/piperma
il/openembedded-core/2018-January/147041.html
Thanks,Joshua Watt
On Wed, 2018-01-31 at 00:39 +0100, Martin Jansa wrote:
> +Stefan
>
> On Wed, Jan 31, 2018 at 12:38 AM, Martin Jansa <martin.jansa@gmail.co
> m> wrote:
> > Thanks Andreas,
> > jack is the only one which inherits waf directly not through waf-
> > samba.
> >
> > More surprisingly it also has ${S}/waf it seems:
> > https://github.com/jackaudio/jack2/blob/c1647819eed6d11f94b21981d9c
> > 869629299f357/waf
> >
> > So the WARNING: shown for jack probably had different reason than
> > the rest of them and that is:
> >
> > jack/1.9.12-r0 $ jack/1.9.12-r0/git/waf --version
> > Error: run waf-light from a folder containing waflib
> >
> >
> > it doesn't like running waf from outside ${S} and waf_preconfigure
> > is executed from ${TOPDIR} before do_configure switches to ${S}
> >
> > I don't know how common this situation is, if all waf scripts are
> > like that, this one (after "cd ../jack/1.9.12-r0/git") says:
> > waf 1.8.17 (cd7579a727d1b390bf9cbf111c1b20e811370bc0)
> >
> > so I'll leave that to original author to figure it out, I don't use
> > waf anywhere, so I cannot properly test the use case he was trying
> > to fix.
> >
> > Regards,
> >
> >
> >
> >
> > On Tue, Jan 30, 2018 at 11:41 PM, Andreas Müller <schnitzeltony@gma
> > il.com> wrote:
> > > On Tuesday, January 30, 2018, Martin Jansa <martin.jansa@gmail.co
> > > m> wrote:
> > > > +Andreas who added waf inherit in jack in:commit
> > > > 0f8f1973e229a0799098bc2334416fec8c608b50
> > > > Author: Andreas Müller <schnitzeltony@googlemail.com>
> > > > Date: Sat Oct 8 12:03:32 2016 +0200
> > > >
> > > > jack: swap to jack2
> > > As far as I can see is jack2 configured with waf. Don't remember
> > > why I chose waf-samba. Because I am on business trip I cannot
> > > take care before next week.
> > >
> > > Andreas
> > > > +Bian who added waf samba in:
> > > > commit 962e346df99696a10657a2ea7b2166dfaf290289
> > > > Author: Bian Naimeng <biannm@cn.fujitsu.com>
> > > > Date: Tue Jun 16 14:18:25 2015 +0900
> > > >
> > > > waf-samba: add new class
> > > >
> > > > waf-samba is a build system used by samba related software
> > > >
> > > > +Joe and Joe and who maintains samba and related recipes +
> > > > meta-networking
> > > >
> > > >
> > > > On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@inte
> > > > l.com> wrote:
> > > > > Why would they do that? Shouldn't it be a fatal bug if waf
> > > > > isn't found?
> > > > > On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail
> > > > > .com> wrote:
> > > > > > * there are some recipes like jack and other which inherit
> > > > > > waf-samba
> > > > > >
> > > > > > which inherits waf, but they don't use waf for configure,
> > > > > > strange isn't
> > > > > >
> > > > > > it?
> > > > > >
> > > > > >
> > > > > >
> > > > > > * fixes:
> > > > > >
> > > > > > WARNING: libldb-1.1.29-r0 do_configure: Unable to execute
> > > > > > waf --version, exit code 127. Assuming waf version without
> > > > > > bindir/libdir support.
> > > > > >
> > > > > > WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf
> > > > > > --version, exit code 127. Assuming waf version without
> > > > > > bindir/libdir support.
> > > > > >
> > > > > > WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute
> > > > > > waf --version, exit code 127. Assuming waf version without
> > > > > > bindir/libdir
> > > > > >
> > > > > > support.
> > > > > >
> > > > > > WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure:
> > > > > > Unable to execute waf --version, exit code 1. Assuming waf
> > > > > > version without bindir/libdir support.
> > > > > >
> > > > > > WARNING: libtevent-0.9.31-r0 do_configure: Unable to
> > > > > > execute waf --version, exit code 127. Assuming waf version
> > > > > > without bindir/libdir support.
> > > > > >
> > > > > > WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute
> > > > > > waf --version, exit code 127. Assuming waf version without
> > > > > > bindir/libdir support.
> > > > > >
> > > > > >
> > > > > >
> > > > > > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > > > > >
> > > > > > ---
> > > > > >
> > > > > > meta/classes/waf.bbclass | 3 +++
> > > > > >
> > > > > > 1 file changed, 3 insertions(+)
> > > > > >
> > > > > >
> > > > > >
> > > > > > diff --git a/meta/classes/waf.bbclass
> > > > > > b/meta/classes/waf.bbclass
> > > > > >
> > > > > > index c3e744e5de..eec4d9496d 100644
> > > > > >
> > > > > > --- a/meta/classes/waf.bbclass
> > > > > >
> > > > > > +++ b/meta/classes/waf.bbclass
> > > > > >
> > > > > > @@ -29,6 +29,9 @@ python waf_preconfigure() {
> > > > > >
> > > > > > from distutils.version import StrictVersion
> > > > > >
> > > > > > srcsubdir = d.getVar('S')
> > > > > >
> > > > > > wafbin = os.path.join(srcsubdir, 'waf')
> > > > > >
> > > > > > + if not os.path.isfile(wafbin):
> > > > > >
> > > > > > + bb.note("%s doesn't exist, cannot check whether it
> > > > > > supports --bindir/--libdir" % wafbin)
> > > > > >
> > > > > > + return
> > > > > >
> > > > > > status, result = oe.utils.getstatusoutput(wafbin + "
> > > > > > --version")
> > > > > >
> > > > > > if status != 0:
> > > > > >
> > > > > > bb.warn("Unable to execute waf --version, exit
> > > > > > code %d. Assuming waf version without bindir/libdir
> > > > > > support." % status)
> > > > > >
> > > > > >
> > > > > > --
> > > > > >
> > > > > > 2.15.1
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > >
> > > > > > _______________________________________________
> > > > > >
> > > > > > Openembedded-core mailing list
> > > > > >
> > > > > > Openembedded-core@lists.openembedded.org
> > > > > >
> > > > > > http://lists.openembedded.org/mailman/listinfo/openembedded
> > > > > > -core
> > > > > >
> > > >
> > > >
[-- Attachment #2: Type: text/html, Size: 8535 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-31 17:04 ` Joshua Watt
@ 2018-01-31 17:07 ` Martin Jansa
2018-01-31 17:15 ` Joshua Watt
2018-01-31 17:16 ` Burton, Ross
0 siblings, 2 replies; 14+ messages in thread
From: Martin Jansa @ 2018-01-31 17:07 UTC (permalink / raw)
To: Joshua Watt; +Cc: Andreas Müller, Joe Slater, Stefan Agner, OE-core
[-- Attachment #1: Type: text/plain, Size: 5026 bytes --]
What about the recipes which don't have ${S}/waf at all (and might inherit
waf from possibly wrong reasons), like all those waf-samba recipes
mentioned before?
On Wed, Jan 31, 2018 at 6:04 PM, Joshua Watt <jpewhacker@gmail.com> wrote:
> Sorry, I wasn't sure if I should make my patch in-reply-to this thread....
>
> I think I fixed this correctly in http://lists.openembedded.org/
> pipermail/openembedded-core/2018-January/147041.html
>
> Thanks,
> Joshua Watt
>
> On Wed, 2018-01-31 at 00:39 +0100, Martin Jansa wrote:
>
> +Stefan
>
> On Wed, Jan 31, 2018 at 12:38 AM, Martin Jansa <martin.jansa@gmail.com>
> wrote:
>
> Thanks Andreas,
>
> jack is the only one which inherits waf directly not through waf-samba.
>
> More surprisingly it also has ${S}/waf it seems:
> https://github.com/jackaudio/jack2/blob/c1647819eed6d11f94b2
> 1981d9c869629299f357/waf
>
> So the WARNING: shown for jack probably had different reason than the rest
> of them and that is:
>
> jack/1.9.12-r0 $ jack/1.9.12-r0/git/waf --version
> Error: run waf-light from a folder containing waflib
>
> it doesn't like running waf from outside ${S} and waf_preconfigure is
> executed from ${TOPDIR} before do_configure switches to ${S}
>
> I don't know how common this situation is, if all waf scripts are like
> that, this one (after "cd ../jack/1.9.12-r0/git") says:
> waf 1.8.17 (cd7579a727d1b390bf9cbf111c1b20e811370bc0)
>
> so I'll leave that to original author to figure it out, I don't use waf
> anywhere, so I cannot properly test the use case he was trying to fix.
>
> Regards,
>
>
>
> On Tue, Jan 30, 2018 at 11:41 PM, Andreas Müller <schnitzeltony@gmail.com>
> wrote:
>
>
>
> On Tuesday, January 30, 2018, Martin Jansa <martin.jansa@gmail.com> wrote:
>
> +Andreas who added waf inherit in jack in:
> commit 0f8f1973e229a0799098bc2334416fec8c608b50
> Author: Andreas Müller <schnitzeltony@googlemail.com>
> Date: Sat Oct 8 12:03:32 2016 +0200
>
> jack: swap to jack2
>
> As far as I can see is jack2 configured with waf. Don't remember why I
> chose waf-samba. Because I am on business trip I cannot take care before
> next week.
>
> Andreas
>
>
> +Bian who added waf samba in:
> commit 962e346df99696a10657a2ea7b2166dfaf290289
> Author: Bian Naimeng <biannm@cn.fujitsu.com>
> Date: Tue Jun 16 14:18:25 2015 +0900
>
> waf-samba: add new class
>
> waf-samba is a build system used by samba related software
>
> +Joe and Joe and who maintains samba and related recipes + meta-networking
>
>
> On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@intel.com>
> wrote:
>
> Why would they do that? Shouldn't it be a fatal bug if waf isn't found?
>
> On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@gmail.com> wrote:
>
> * there are some recipes like jack and other which inherit waf-samba
> which inherits waf, but they don't use waf for configure, strange isn't
> it?
>
> * fixes:
> WARNING: libldb-1.1.29-r0 do_configure: Unable to execute waf --version,
> exit code 127. Assuming waf version without bindir/libdir support.
> WARNING: samba-4.6.7-r0 do_configure: Unable to execute waf --version,
> exit code 127. Assuming waf version without bindir/libdir support.
> WARNING: libtalloc-2.1.9-r0 do_configure: Unable to execute waf --version,
> exit code 127. Assuming waf version without bindir/libdir
> support.
> WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0 do_configure: Unable to
> execute waf --version, exit code 1. Assuming waf version without
> bindir/libdir support.
> WARNING: libtevent-0.9.31-r0 do_configure: Unable to execute waf
> --version, exit code 127. Assuming waf version without bindir/libdir
> support.
> WARNING: libtdb-1.3.14-r0 do_configure: Unable to execute waf --version,
> exit code 127. Assuming waf version without bindir/libdir support.
>
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
> meta/classes/waf.bbclass | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/meta/classes/waf.bbclass b/meta/classes/waf.bbclass
> index c3e744e5de..eec4d9496d 100644
> --- a/meta/classes/waf.bbclass
> +++ b/meta/classes/waf.bbclass
> @@ -29,6 +29,9 @@ python waf_preconfigure() {
> from distutils.version import StrictVersion
> srcsubdir = d.getVar('S')
> wafbin = os.path.join(srcsubdir, 'waf')
> + if not os.path.isfile(wafbin):
> + bb.note("%s doesn't exist, cannot check whether it supports
> --bindir/--libdir" % wafbin)
> + return
> status, result = oe.utils.getstatusoutput(wafbin + " --version")
> if status != 0:
> bb.warn("Unable to execute waf --version, exit code %d. Assuming
> waf version without bindir/libdir support." % status)
> --
> 2.15.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
>
>
>
>
>
[-- Attachment #2: Type: text/html, Size: 9101 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-31 17:07 ` Martin Jansa
@ 2018-01-31 17:15 ` Joshua Watt
2018-01-31 17:16 ` Burton, Ross
1 sibling, 0 replies; 14+ messages in thread
From: Joshua Watt @ 2018-01-31 17:15 UTC (permalink / raw)
To: Martin Jansa; +Cc: Andreas Müller, Joe Slater, Stefan Agner, OE-core
[-- Attachment #1: Type: text/plain, Size: 7761 bytes --]
On Wed, 2018-01-31 at 18:07 +0100, Martin Jansa wrote:
> What about the recipes which don't have ${S}/waf at all (and might
> inherit waf from possibly wrong reasons), like all those waf-samba
> recipes mentioned before?
Ya, sorry I should have clarified: This should fix it for recipes that
only include waf.bbclass (e.g. jack and my local waf recipes that were
also getting the same error). I still think that perhaps waf-
samba.bbclass probably shouldn't be inheriting from waf.bbclass if the
intent is that those projects doesn't actually have ${S}/waf.... but to
be fair I haven't looked closely at the recipes that inherit waf-
samba.bbclass.
> On Wed, Jan 31, 2018 at 6:04 PM, Joshua Watt <jpewhacker@gmail.com>
> wrote:
> > Sorry, I wasn't sure if I should make my patch in-reply-to this
> > thread....
> >
> > I think I fixed this correctly in http://lists.openembedded.org/pip
> > ermail/openembedded-core/2018-January/147041.html
> >
> > Thanks,
> > Joshua Watt
> >
> > On Wed, 2018-01-31 at 00:39 +0100, Martin Jansa wrote:
> > > +Stefan
> > >
> > > On Wed, Jan 31, 2018 at 12:38 AM, Martin Jansa <martin.jansa@gmai
> > > l.com> wrote:
> > > > Thanks Andreas,
> > > > jack is the only one which inherits waf directly not through
> > > > waf-samba.
> > > >
> > > > More surprisingly it also has ${S}/waf it seems:
> > > > https://github.com/jackaudio/jack2/blob/c1647819eed6d11f94b2198
> > > > 1d9c869629299f357/waf
> > > >
> > > > So the WARNING: shown for jack probably had different reason
> > > > than the rest of them and that is:
> > > >
> > > > jack/1.9.12-r0 $ jack/1.9.12-r0/git/waf --version
> > > > Error: run waf-light from a folder containing waflib
> > > >
> > > >
> > > > it doesn't like running waf from outside ${S}
> > > > and waf_preconfigure is executed from ${TOPDIR} before
> > > > do_configure switches to ${S}
> > > >
> > > > I don't know how common this situation is, if all waf scripts
> > > > are like that, this one (after "cd ../jack/1.9.12-r0/git")
> > > > says:
> > > > waf 1.8.17 (cd7579a727d1b390bf9cbf111c1b20e811370bc0)
> > > >
> > > > so I'll leave that to original author to figure it out, I don't
> > > > use waf anywhere, so I cannot properly test the use case he was
> > > > trying to fix.
> > > >
> > > > Regards,
> > > >
> > > >
> > > >
> > > >
> > > > On Tue, Jan 30, 2018 at 11:41 PM, Andreas Müller <schnitzeltony
> > > > @gmail.com> wrote:
> > > > > On Tuesday, January 30, 2018, Martin Jansa <martin.jansa@gmai
> > > > > l.com> wrote:
> > > > > > +Andreas who added waf inherit in jack in:commit
> > > > > > 0f8f1973e229a0799098bc2334416fec8c608b50
> > > > > > Author: Andreas Müller <schnitzeltony@googlemail.com>
> > > > > > Date: Sat Oct 8 12:03:32 2016 +0200
> > > > > >
> > > > > > jack: swap to jack2
> > > > > As far as I can see is jack2 configured with waf. Don't
> > > > > remember why I chose waf-samba. Because I am on business trip
> > > > > I cannot take care before next week.
> > > > >
> > > > > Andreas
> > > > > > +Bian who added waf samba in:
> > > > > > commit 962e346df99696a10657a2ea7b2166dfaf290289
> > > > > > Author: Bian Naimeng <biannm@cn.fujitsu.com>
> > > > > > Date: Tue Jun 16 14:18:25 2015 +0900
> > > > > >
> > > > > > waf-samba: add new class
> > > > > >
> > > > > > waf-samba is a build system used by samba related
> > > > > > software
> > > > > >
> > > > > > +Joe and Joe and who maintains samba and related recipes +
> > > > > > meta-networking
> > > > > >
> > > > > >
> > > > > > On Tue, Jan 30, 2018 at 1:03 PM, Burton, Ross <ross.burton@
> > > > > > intel.com> wrote:
> > > > > > > Why would they do that? Shouldn't it be a fatal bug if
> > > > > > > waf isn't found?
> > > > > > > On 30 January 2018 at 08:30, Martin Jansa <martin.jansa@g
> > > > > > > mail.com> wrote:
> > > > > > > > * there are some recipes like jack and other which
> > > > > > > > inherit waf-samba
> > > > > > > >
> > > > > > > > which inherits waf, but they don't use waf for
> > > > > > > > configure, strange isn't
> > > > > > > >
> > > > > > > > it?
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > * fixes:
> > > > > > > >
> > > > > > > > WARNING: libldb-1.1.29-r0 do_configure: Unable to
> > > > > > > > execute waf --version, exit code 127. Assuming waf
> > > > > > > > version without bindir/libdir support.
> > > > > > > >
> > > > > > > > WARNING: samba-4.6.7-r0 do_configure: Unable to execute
> > > > > > > > waf --version, exit code 127. Assuming waf version
> > > > > > > > without bindir/libdir support.
> > > > > > > >
> > > > > > > > WARNING: libtalloc-2.1.9-r0 do_configure: Unable to
> > > > > > > > execute waf --version, exit code 127. Assuming waf
> > > > > > > > version without bindir/libdir
> > > > > > > >
> > > > > > > > support.
> > > > > > > >
> > > > > > > > WARNING: jack-1.9.10+gitAUTOINC+2d1d323505-r0
> > > > > > > > do_configure: Unable to execute waf --version, exit
> > > > > > > > code 1. Assuming waf version without bindir/libdir
> > > > > > > > support.
> > > > > > > >
> > > > > > > > WARNING: libtevent-0.9.31-r0 do_configure: Unable to
> > > > > > > > execute waf --version, exit code 127. Assuming waf
> > > > > > > > version without bindir/libdir support.
> > > > > > > >
> > > > > > > > WARNING: libtdb-1.3.14-r0 do_configure: Unable to
> > > > > > > > execute waf --version, exit code 127. Assuming waf
> > > > > > > > version without bindir/libdir support.
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > > > > > > >
> > > > > > > > ---
> > > > > > > >
> > > > > > > > meta/classes/waf.bbclass | 3 +++
> > > > > > > >
> > > > > > > > 1 file changed, 3 insertions(+)
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > diff --git a/meta/classes/waf.bbclass
> > > > > > > > b/meta/classes/waf.bbclass
> > > > > > > >
> > > > > > > > index c3e744e5de..eec4d9496d 100644
> > > > > > > >
> > > > > > > > --- a/meta/classes/waf.bbclass
> > > > > > > >
> > > > > > > > +++ b/meta/classes/waf.bbclass
> > > > > > > >
> > > > > > > > @@ -29,6 +29,9 @@ python waf_preconfigure() {
> > > > > > > >
> > > > > > > > from distutils.version import StrictVersion
> > > > > > > >
> > > > > > > > srcsubdir = d.getVar('S')
> > > > > > > >
> > > > > > > > wafbin = os.path.join(srcsubdir, 'waf')
> > > > > > > >
> > > > > > > > + if not os.path.isfile(wafbin):
> > > > > > > >
> > > > > > > > + bb.note("%s doesn't exist, cannot check
> > > > > > > > whether it supports --bindir/--libdir" % wafbin)
> > > > > > > >
> > > > > > > > + return
> > > > > > > >
> > > > > > > > status, result = oe.utils.getstatusoutput(wafbin +
> > > > > > > > " --version")
> > > > > > > >
> > > > > > > > if status != 0:
> > > > > > > >
> > > > > > > > bb.warn("Unable to execute waf --version, exit
> > > > > > > > code %d. Assuming waf version without bindir/libdir
> > > > > > > > support." % status)
> > > > > > > >
> > > > > > > >
> > > > > > > > --
> > > > > > > >
> > > > > > > > 2.15.1
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > --
> > > > > > > >
> > > > > > > > _______________________________________________
> > > > > > > >
> > > > > > > > Openembedded-core mailing list
> > > > > > > >
> > > > > > > > Openembedded-core@lists.openembedded.org
> > > > > > > >
> > > > > > > > http://lists.openembedded.org/mailman/listinfo/openembe
> > > > > > > > dded-core
> > > > > > > >
> > > > > >
> > > > > >
[-- Attachment #2: Type: text/html, Size: 9986 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-31 17:07 ` Martin Jansa
2018-01-31 17:15 ` Joshua Watt
@ 2018-01-31 17:16 ` Burton, Ross
2018-01-31 17:19 ` Martin Jansa
1 sibling, 1 reply; 14+ messages in thread
From: Burton, Ross @ 2018-01-31 17:16 UTC (permalink / raw)
To: Martin Jansa; +Cc: Andreas Müller, OE-core, Joe Slater, Stefan Agner
[-- Attachment #1: Type: text/plain, Size: 440 bytes --]
On 31 January 2018 at 17:07, Martin Jansa <martin.jansa@gmail.com> wrote:
> What about the recipes which don't have ${S}/waf at all (and might inherit
> waf from possibly wrong reasons), like all those waf-samba recipes
> mentioned before?
>
If a recipe inherits waf but doesn't use waf I'm absolutely fine with it
erroring out. Its always bothered me that autotools.bbclass doesn't object
if configure can't be found.
Ross
[-- Attachment #2: Type: text/html, Size: 800 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-31 17:16 ` Burton, Ross
@ 2018-01-31 17:19 ` Martin Jansa
2018-01-31 17:21 ` Burton, Ross
0 siblings, 1 reply; 14+ messages in thread
From: Martin Jansa @ 2018-01-31 17:19 UTC (permalink / raw)
To: Burton, Ross; +Cc: Andreas Müller, OE-core, Joe Slater, Stefan Agner
[-- Attachment #1: Type: text/plain, Size: 688 bytes --]
But shouldn't it at least show different error message in case when
${S}/waf doesn't exist and when it fails to execute it for whatever reason?
On Wed, Jan 31, 2018 at 6:16 PM, Burton, Ross <ross.burton@intel.com> wrote:
> On 31 January 2018 at 17:07, Martin Jansa <martin.jansa@gmail.com> wrote:
>
>> What about the recipes which don't have ${S}/waf at all (and might
>> inherit waf from possibly wrong reasons), like all those waf-samba recipes
>> mentioned before?
>>
>
> If a recipe inherits waf but doesn't use waf I'm absolutely fine with it
> erroring out. Its always bothered me that autotools.bbclass doesn't object
> if configure can't be found.
>
> Ross
>
[-- Attachment #2: Type: text/html, Size: 1428 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] waf.bbclass: check that waf exists before trying to execute it
2018-01-31 17:19 ` Martin Jansa
@ 2018-01-31 17:21 ` Burton, Ross
0 siblings, 0 replies; 14+ messages in thread
From: Burton, Ross @ 2018-01-31 17:21 UTC (permalink / raw)
To: Martin Jansa; +Cc: Andreas Müller, OE-core, Joe Slater, Stefan Agner
[-- Attachment #1: Type: text/plain, Size: 313 bytes --]
On 31 January 2018 at 17:19, Martin Jansa <martin.jansa@gmail.com> wrote:
> But shouldn't it at least show different error message in case when
> ${S}/waf doesn't exist and when it fails to execute it for whatever reason?
>
Sure, first thing should be to check if S/waf exists and if not, abort.
Ross
[-- Attachment #2: Type: text/html, Size: 667 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2018-01-31 17:21 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-30 8:30 [PATCH] waf.bbclass: check that waf exists before trying to execute it Martin Jansa
2018-01-30 12:03 ` Burton, Ross
2018-01-30 12:26 ` Martin Jansa
2018-01-30 14:30 ` Joshua Watt
2018-01-30 22:41 ` Andreas Müller
2018-01-30 23:38 ` Martin Jansa
2018-01-30 23:39 ` Martin Jansa
2018-01-31 2:30 ` Joshua Watt
2018-01-31 17:04 ` Joshua Watt
2018-01-31 17:07 ` Martin Jansa
2018-01-31 17:15 ` Joshua Watt
2018-01-31 17:16 ` Burton, Ross
2018-01-31 17:19 ` Martin Jansa
2018-01-31 17:21 ` Burton, Ross
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.