[PATCH] AB random failure: libpcre-native split_and_strip

[PATCH] AB random failure: libpcre-native split_and_strip

[Alex Franco]

Error handling: adding file-native as a dependency to libpre recipe
should guarantee a consistent, and more importantly POSIX-compliant
version of the "file" command is always used when checking the file
"pcre-config" within the isELF function in staging.bbclass.

Following the POSIX behavior, file returns 0 and continues, when an
error is found while checking a file.

Debug: in cases where file may still fail, the resulting error will
now be logged along with the split_and_strip_files failure.

[YOCTO #8144]

Signed-off-by: Alex Franco <alejandro.franco@linux.intel.com>
---
 meta/classes/staging.bbclass                 | 2 +-
 meta/recipes-support/libpcre/libpcre_8.37.bb | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/staging.bbclass b/meta/classes/staging.bbclass
index 967eddd..554c2e4 100644
--- a/meta/classes/staging.bbclass
+++ b/meta/classes/staging.bbclass
@@ -77,7 +77,7 @@ python sysroot_strip () {
         ret, result = oe.utils.getstatusoutput("file \"%s\"" % path.replace("\"", "\\\""))
 
         if ret:
-            bb.error("split_and_strip_files: 'file %s' failed" % path)
+            bb.error("split_and_strip_files: 'file %s' failed\nError: %s" % (path, result))
             return type
 
         # Not stripped
diff --git a/meta/recipes-support/libpcre/libpcre_8.37.bb b/meta/recipes-support/libpcre/libpcre_8.37.bb
index bcfc9e9..1a2024d 100644
--- a/meta/recipes-support/libpcre/libpcre_8.37.bb
+++ b/meta/recipes-support/libpcre/libpcre_8.37.bb
@@ -20,7 +20,7 @@ SRC_URI[sha256sum] = "51679ea8006ce31379fb0860e46dd86665d864b5020fc9cd19e71260ee
 S = "${WORKDIR}/pcre-${PV}"
 
 PROVIDES += "pcre"
-DEPENDS += "bzip2 zlib"
+DEPENDS += "bzip2 zlib file-native"
 
 PACKAGECONFIG[pcretest-readline] = "--enable-pcretest-libreadline,--disable-pcretest-libreadline,readline,"
 
-- 
2.5.3

Re: [PATCH] AB random failure: libpcre-native split_and_strip

[Richard Purdie]

On Fri, 2015-09-25 at 17:54 -0500, Alex Franco wrote:
> Error handling: adding file-native as a dependency to libpre recipe
> should guarantee a consistent, and more importantly POSIX-compliant
> version of the "file" command is always used when checking the file
> "pcre-config" within the isELF function in staging.bbclass.
> 
> Following the POSIX behavior, file returns 0 and continues, when an
> error is found while checking a file.
> 
> Debug: in cases where file may still fail, the resulting error will
> now be logged along with the split_and_strip_files failure.

I like the improvement in debug, I'm not sure the other part of this
will solve the problem though. We've seen this error on recipes other
that libpcre and also, which systems do you think doesn't have a
POSIX-compliant version of the file?

Cheers,

Richard

> [YOCTO #8144]
> 
> Signed-off-by: Alex Franco <alejandro.franco@linux.intel.com>
> ---
>  meta/classes/staging.bbclass                 | 2 +-
>  meta/recipes-support/libpcre/libpcre_8.37.bb | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/classes/staging.bbclass b/meta/classes/staging.bbclass
> index 967eddd..554c2e4 100644
> --- a/meta/classes/staging.bbclass
> +++ b/meta/classes/staging.bbclass
> @@ -77,7 +77,7 @@ python sysroot_strip () {
>          ret, result = oe.utils.getstatusoutput("file \"%s\"" % path.replace("\"", "\\\""))
>  
>          if ret:
> -            bb.error("split_and_strip_files: 'file %s' failed" % path)
> +            bb.error("split_and_strip_files: 'file %s' failed\nError: %s" % (path, result))
>              return type
>  
>          # Not stripped
> diff --git a/meta/recipes-support/libpcre/libpcre_8.37.bb b/meta/recipes-support/libpcre/libpcre_8.37.bb
> index bcfc9e9..1a2024d 100644
> --- a/meta/recipes-support/libpcre/libpcre_8.37.bb
> +++ b/meta/recipes-support/libpcre/libpcre_8.37.bb
> @@ -20,7 +20,7 @@ SRC_URI[sha256sum] = "51679ea8006ce31379fb0860e46dd86665d864b5020fc9cd19e71260ee
>  S = "${WORKDIR}/pcre-${PV}"
>  
>  PROVIDES += "pcre"
> -DEPENDS += "bzip2 zlib"
> +DEPENDS += "bzip2 zlib file-native"
>  
>  PACKAGECONFIG[pcretest-readline] = "--enable-pcretest-libreadline,--disable-pcretest-libreadline,readline,"
>  
> -- 
> 2.5.3
>