[PATCH 0/3] meta: 3 fixes

[PATCH 0/3] meta: 3 fixes

[Robert Yang]

The following changes since commit e6f66370c6ce15aca18ef64491bab3dc92b80c57:

  classes/sstate.bbclass: Improve checkstatus using connection cache. (2015-07-12 22:53:22 +0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib rbt/3fixes
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/3fixes

Robert Yang (3):
  glibc: print PN when bbwarn
  dbus-ptest: set INSANE_SKIP for build-deps
  gnu-config: set noexec for do_compile

 meta/recipes-core/dbus/dbus.inc                    |    2 ++
 meta/recipes-core/glibc/glibc_2.21.bb              |    3 ++-
 .../gnu-config/gnu-config_20120814.bb              |    4 +---
 3 files changed, 5 insertions(+), 4 deletions(-)

-- 
1.7.9.5

[PATCH 1/3] glibc: print PN when bbwarn

[Robert Yang]

The current warning is:
WARNING: glibc: unable to generate header for spray.x

Which is easier to debug than:
WARNING: unable to generate header for spray.x

And remove the file before generate it again to fix the warning when
recompile:
file `bootparam_prot.h' already exists and may be overwritten
WARNING: unable to generate header for bootparam_prot.x
file `nlm_prot.h' already exists and may be overwritten
WARNING: unable to generate header for nlm_prot.x
[snip]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-core/glibc/glibc_2.21.bb |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/glibc/glibc_2.21.bb b/meta/recipes-core/glibc/glibc_2.21.bb
index 8197c29..30f1937 100644
--- a/meta/recipes-core/glibc/glibc_2.21.bb
+++ b/meta/recipes-core/glibc/glibc_2.21.bb
@@ -135,7 +135,8 @@ do_compile () {
 		cd ${S}/sunrpc/rpcsvc
 		for r in ${rpcsvc}; do
 			h=`echo $r|sed -e's,\.x$,.h,'`
-			rpcgen -h $r -o $h || bbwarn "unable to generate header for $r"
+			rm -f $h
+			rpcgen -h $r -o $h || bbwarn "${PN}: unable to generate header for $r"
 		done
 	)
 	echo "Adjust ldd script"
-- 
1.7.9.5

[PATCH 2/3] dbus-ptest: set INSANE_SKIP for build-deps

[Robert Yang]

Skip the warning for dbus-ptest:
dbus-1.8.18: dbus-ptest rdepends on dbus-test-ptest, but it isn't a build dependency? [build-deps]

We add dbus-test to dbus' DEPENDS to fix the problem usually, but
dbus-test DEPENDS on dbus, so we can't do that, use INSANE_SKIP to fix
the problem.

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-core/dbus/dbus.inc |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
index 01066cb..3971081 100644
--- a/meta/recipes-core/dbus/dbus.inc
+++ b/meta/recipes-core/dbus/dbus.inc
@@ -166,3 +166,5 @@ do_install_class-nativesdk() {
 	rm -rf ${D}${localstatedir}/run
 }
 BBCLASSEXTEND = "native nativesdk"
+
+INSANE_SKIP_${PN}-ptest += "build-deps"
-- 
1.7.9.5

[PATCH 3/3] gnu-config: set noexec for do_compile

[Robert Yang]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 .../gnu-config/gnu-config_20120814.bb              |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta/recipes-devtools/gnu-config/gnu-config_20120814.bb b/meta/recipes-devtools/gnu-config/gnu-config_20120814.bb
index f5fce6f..eb7e420 100644
--- a/meta/recipes-devtools/gnu-config/gnu-config_20120814.bb
+++ b/meta/recipes-devtools/gnu-config/gnu-config_20120814.bb
@@ -19,9 +19,7 @@ SRC_URI[sha256sum] = "44f99a8e76f3e8e4fec0bb5ad4762f8e44366168554ce66cb85afbe2ed
 
 CLEANBROKEN = "1"
 
-do_compile() {
-	:
-}
+do_compile[noexec] = "1"
 
 do_install () {
 	install -d ${D}${datadir}/gnu-config \
-- 
1.7.9.5

Re: [PATCH 1/3] glibc: print PN when bbwarn

[Khem Raj]

On Mon, Jul 13, 2015 at 3:02 AM, Robert Yang <liezhi.yang@windriver.com> wrote:
> The current warning is:
> WARNING: glibc: unable to generate header for spray.x
>
> Which is easier to debug than:
> WARNING: unable to generate header for spray.x
>
> And remove the file before generate it again to fix the warning when
> recompile:
> file `bootparam_prot.h' already exists and may be overwritten
> WARNING: unable to generate header for bootparam_prot.x
> file `nlm_prot.h' already exists and may be overwritten
> WARNING: unable to generate header for nlm_prot.x
> [snip]
>
> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> ---
>  meta/recipes-core/glibc/glibc_2.21.bb |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-core/glibc/glibc_2.21.bb b/meta/recipes-core/glibc/glibc_2.21.bb
> index 8197c29..30f1937 100644
> --- a/meta/recipes-core/glibc/glibc_2.21.bb
> +++ b/meta/recipes-core/glibc/glibc_2.21.bb
> @@ -135,7 +135,8 @@ do_compile () {
>                 cd ${S}/sunrpc/rpcsvc
>                 for r in ${rpcsvc}; do
>                         h=`echo $r|sed -e's,\.x$,.h,'`
> -                       rpcgen -h $r -o $h || bbwarn "unable to generate header for $r"
> +                       rm -f $h

it quashes the warning but doesnt fix underlying problem. this doesnt
seem to be right thing to do in cross builds. We use cross-rpcgen to
generate these files so this whole rpcgen run should be removed
instead. Please test such a patch and correct it.

> +                       rpcgen -h $r -o $h || bbwarn "${PN}: unable to generate header for $r"
>                 done
>         )
>         echo "Adjust ldd script"
> --
> 1.7.9.5
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH 1/3] glibc: print PN when bbwarn

[Robert Yang]

On 07/17/2015 12:45 PM, Khem Raj wrote:
> On Mon, Jul 13, 2015 at 3:02 AM, Robert Yang <liezhi.yang@windriver.com> wrote:
>> The current warning is:
>> WARNING: glibc: unable to generate header for spray.x
>>
>> Which is easier to debug than:
>> WARNING: unable to generate header for spray.x
>>
>> And remove the file before generate it again to fix the warning when
>> recompile:
>> file `bootparam_prot.h' already exists and may be overwritten
>> WARNING: unable to generate header for bootparam_prot.x
>> file `nlm_prot.h' already exists and may be overwritten
>> WARNING: unable to generate header for nlm_prot.x
>> [snip]
>>
>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>> ---
>>   meta/recipes-core/glibc/glibc_2.21.bb |    3 ++-
>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-core/glibc/glibc_2.21.bb b/meta/recipes-core/glibc/glibc_2.21.bb
>> index 8197c29..30f1937 100644
>> --- a/meta/recipes-core/glibc/glibc_2.21.bb
>> +++ b/meta/recipes-core/glibc/glibc_2.21.bb
>> @@ -135,7 +135,8 @@ do_compile () {
>>                  cd ${S}/sunrpc/rpcsvc
>>                  for r in ${rpcsvc}; do
>>                          h=`echo $r|sed -e's,\.x$,.h,'`
>> -                       rpcgen -h $r -o $h || bbwarn "unable to generate header for $r"
>> +                       rm -f $h
>
> it quashes the warning but doesnt fix underlying problem. this doesnt
> seem to be right thing to do in cross builds. We use cross-rpcgen to
> generate these files so this whole rpcgen run should be removed
> instead. Please test such a patch and correct it.
>

Hi Khem,

I will be on vacation the whole next week, will do when I come back.

// Robert

>> +                       rpcgen -h $r -o $h || bbwarn "${PN}: unable to generate header for $r"
>>                  done
>>          )
>>          echo "Adjust ldd script"
>> --
>> 1.7.9.5
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>

Re: [PATCH 1/3] glibc: print PN when bbwarn

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 506 bytes --]

On 17 July 2015 at 05:45, Khem Raj <raj.khem@gmail.com> wrote:

> it quashes the warning but doesnt fix underlying problem. this doesnt
> seem to be right thing to do in cross builds. We use cross-rpcgen to
> generate these files so this whole rpcgen run should be removed
> instead. Please test such a patch and correct it.
>

There's been a related report on the yocto@ list where it was noticed that
we're executing the host rpcgen during glibc builds, so consider this is a
ping. :)

Ross

[-- Attachment #2: Type: text/html, Size: 901 bytes --]

Re: [PATCH 1/3] glibc: print PN when bbwarn

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 295 bytes --]

On 17 August 2015 at 13:33, Burton, Ross <ross.burton@intel.com> wrote:

> There's been a related report on the yocto@ list where it was noticed
> that we're executing the host rpcgen during glibc builds, so consider
> this is a ping. :)
>

Filed #8181 so this doesn't get lost.

Ross

[-- Attachment #2: Type: text/html, Size: 667 bytes --]

Re: [PATCH 1/3] glibc: print PN when bbwarn

[Robert Yang]

Thanks, I will work on it.

// Robert

On 08/19/2015 01:02 AM, Burton, Ross wrote:
>
> On 17 August 2015 at 13:33, Burton, Ross <ross.burton@intel.com
> <mailto:ross.burton@intel.com>> wrote:
>
>     There's been a related report on the yocto@ list where it was noticed that
>     we're executing the host rpcgen during glibc builds, so consider this is a
>     ping. :)
>
>
> Filed #8181 so this doesn't get lost.
>
> Ross