[PATCH 0/1] epiphany: inherit perlnative

[PATCH 0/1] epiphany: inherit perlnative

[Robert Yang]

The following changes since commit 0a2df616a5c3316704742f1dcf37b450920e0280:

  boost: fix CVE-2012-2677 (2016-07-21 07:45:01 +0100)

are available in the git repository at:

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

Robert Yang (1):
  epiphany: inherit perlnative

 meta/recipes-gnome/epiphany/epiphany_3.20.3.bb | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

-- 
2.9.0

[PATCH 1/1] epiphany: inherit perlnative

[Robert Yang]

Fixed:
checking for XML::Parser... configure: error: XML::Parser perl module is required for intltool

The module is installed to sysroot, but need use native perl to find it.

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-gnome/epiphany/epiphany_3.20.3.bb | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/recipes-gnome/epiphany/epiphany_3.20.3.bb b/meta/recipes-gnome/epiphany/epiphany_3.20.3.bb
index eba480b..a61e2cf 100644
--- a/meta/recipes-gnome/epiphany/epiphany_3.20.3.bb
+++ b/meta/recipes-gnome/epiphany/epiphany_3.20.3.bb
@@ -5,7 +5,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 DEPENDS = "libsoup-2.4 webkitgtk gtk+3 iso-codes ca-certificates avahi libnotify gcr libwnck3 \
 	   gsettings-desktop-schemas gnome-desktop3 libxml2-native intltool-native"
 
-inherit gnomebase gsettings distro_features_check upstream-version-is-even
+inherit gnomebase gsettings distro_features_check upstream-version-is-even perlnative
 # libwnck3 is x11 only
 REQUIRED_DISTRO_FEATURES = "x11"
 
@@ -22,4 +22,3 @@ do_configure_prepend() {
 FILES_${PN} += "${datadir}/appdata ${datadir}/dbus-1 ${datadir}/gnome-shell/search-providers"
 RDEPENDS_${PN} = "iso-codes adwaita-icon-theme"
 RRECOMMENDS_${PN} = "ca-certificates"
-
-- 
2.9.0

Re: [PATCH 1/1] epiphany: inherit perlnative

[Burton, Ross]

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

On 24 July 2016 at 06:16, Robert Yang <liezhi.yang@windriver.com> wrote:

> Fixed:
> checking for XML::Parser... configure: error: XML::Parser perl module is
> required for intltool
>
> The module is installed to sysroot, but need use native perl to find it.
>

This is fixed by [PATCH 2/2] intltool: remove broken XML::Parser detection.

Ross

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

Re: [PATCH 1/1] epiphany: inherit perlnative

[Robert Yang]

On 07/26/2016 07:33 AM, Burton, Ross wrote:
>
> On 24 July 2016 at 06:16, Robert Yang <liezhi.yang@windriver.com
> <mailto:liezhi.yang@windriver.com>> wrote:
>
>     Fixed:
>     checking for XML::Parser... configure: error: XML::Parser perl module is
>     required for intltool
>
>     The module is installed to sysroot, but need use native perl to find it.
>
>
> This is fixed by [PATCH 2/2] intltool: remove broken XML::Parser detection.

Hi Ross,

I still get the same error after your patch applied:

configure:16513: checking for perl
configure:16531: found /usr/bin/perl
configure:16543: result: /usr/bin/perl
configure:16554: checking for perl >= 5.8.1
configure:16561: result: 5.18.2
configure:16565: checking for XML::Parser
configure:16571: error: XML::Parser perl module is required for intltool

// Robert

>
> Ross

Re: [PATCH 1/1] epiphany: inherit perlnative

[Khem Raj]

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


> On Jul 26, 2016, at 6:17 PM, Robert Yang <liezhi.yang@windriver.com> wrote:
> 
> 
> 
> On 07/26/2016 07:33 AM, Burton, Ross wrote:
>> 
>> On 24 July 2016 at 06:16, Robert Yang <liezhi.yang@windriver.com
>> <mailto:liezhi.yang@windriver.com>> wrote:
>> 
>>    Fixed:
>>    checking for XML::Parser... configure: error: XML::Parser perl module is
>>    required for intltool
>> 
>>    The module is installed to sysroot, but need use native perl to find it.
>> 
>> 
>> This is fixed by [PATCH 2/2] intltool: remove broken XML::Parser detection.
> 
> Hi Ross,
> 
> I still get the same error after your patch applied:
> 
> configure:16513: checking for perl
> configure:16531: found /usr/bin/perl
> configure:16543: result: /usr/bin/perl
> configure:16554: checking for perl >= 5.8.1
> configure:16561: result: 5.18.2
> configure:16565: checking for XML::Parser
> configure:16571: error: XML::Parser perl module is required for intltool

same here

http://errors.yoctoproject.org/Errors/Details/73330/


[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

Re: [PATCH 1/1] epiphany: inherit perlnative

[Burton, Ross]

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

On 27 July 2016 at 10:34, Khem Raj <raj.khem@gmail.com> wrote:

> > I still get the same error after your patch applied:
> >
> > configure:16513: checking for perl
> > configure:16531: found /usr/bin/perl
> > configure:16543: result: /usr/bin/perl
> > configure:16554: checking for perl >= 5.8.1
> > configure:16561: result: 5.18.2
> > configure:16565: checking for XML::Parser
> > configure:16571: error: XML::Parser perl module is required for intltool
>
> same here
>
> http://errors.yoctoproject.org/Errors/Details/73330/


[shakes fist at intltool]

So intltool is basically deprecated and I'm seriously considering spending
the time porting to modern gettext instead of battling intltool.

Ross

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

Re: [PATCH 1/1] epiphany: inherit perlnative

[Burton, Ross]

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

On 27 July 2016 at 10:34, Khem Raj <raj.khem@gmail.com> wrote:

> > I still get the same error after your patch applied:
> >
> > configure:16513: checking for perl
> > configure:16531: found /usr/bin/perl
> > configure:16543: result: /usr/bin/perl
> > configure:16554: checking for perl >= 5.8.1
> > configure:16561: result: 5.18.2
> > configure:16565: checking for XML::Parser
> > configure:16571: error: XML::Parser perl module is required for intltool
>
> same here
>
> http://errors.yoctoproject.org/Errors/Details/73330/


I don't understand how as my patch literally removes that block of code
from the intltool macros.  Can you still replicate this, and if so can you
save the build tree?

Ross

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

Re: [PATCH 1/1] epiphany: inherit perlnative

[Randle, William C]

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

On Thu, 2016-08-04 at 16:17 +0100, Burton, Ross wrote:

On 27 July 2016 at 10:34, Khem Raj <raj.khem@gmail.com<mailto:raj.khem@gmail.com>> wrote:
> I still get the same error after your patch applied:
>
> configure:16513: checking for perl
> configure:16531: found /usr/bin/perl
> configure:16543: result: /usr/bin/perl
> configure:16554: checking for perl >= 5.8.1
> configure:16561: result: 5.18.2
> configure:16565: checking for XML::Parser
> configure:16571: error: XML::Parser perl module is required for intltool

same here

http://errors.yoctoproject.org/Errors/Details/73330/

I don't understand how as my patch literally removes that block of code from the intltool macros.  Can you still replicate this, and if so can you save the build tree?

Ross

>

Here's the one that was on the Intel AB the other day: https://autobuilder.yoctoproject.org/intel/builders/nightly-meta-intel-world/builds/124/steps/BuildImages_1/logs/stdio . Search for "ERROR:" then scroll down a bit. The build tree is still on yocto-ab01.ostc.intel.com under nightly-meta-intel-world.

    -Bill



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

Re: [PATCH 1/1] epiphany: inherit perlnative

[Robert Yang]

On 08/04/2016 11:17 PM, Burton, Ross wrote:
>
> On 27 July 2016 at 10:34, Khem Raj <raj.khem@gmail.com
> <mailto:raj.khem@gmail.com>> wrote:
>
>     > I still get the same error after your patch applied:
>     >
>     > configure:16513: checking for perl
>     > configure:16531: found /usr/bin/perl
>     > configure:16543: result: /usr/bin/perl
>     > configure:16554: checking for perl >= 5.8.1
>     > configure:16561: result: 5.18.2
>     > configure:16565: checking for XML::Parser
>     > configure:16571: error: XML::Parser perl module is required forintltool
>
>     same here
>
>     http://errors.yoctoproject.org/Errors/Details/73330/
>
>
> I don't understand how as my patch literally removes that block of code from the
> intltool macros.  Can you still replicate this, and if so can you save the build
> tree?

I can reproduce the problem every time on my host Ubuntu 14.04 64bit:

local.conf:
MACHINE = "qemux86-64"
INHERIT += "packagefeed-stability"
PRSERV_HOST = "localhost:0"
require conf/multilib.conf
MULTILIBS = "multilib:lib32"
DEFAULTTUNE_virtclass-multilib-lib32 = "x86"

Error log:
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=rbt/errlog&id=9e34fa2d6eed6332d376c341113d5dd386c5d200

I guess that you can't reproduce the problem is because you have XML::Parser
installed on host ?

// Robert

>
> Ross

Re: [PATCH 1/1] epiphany: inherit perlnative

[Burton, Ross]

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

On 5 August 2016 at 03:47, Robert Yang <liezhi.yang@windriver.com> wrote:

> I can reproduce the problem every time on my host Ubuntu 14.04 64bit:
>

Just discovered that ephy for some reason ships its own copies of
intltool.m4 (and a few others).  We'll need to delete those.  Patch
incoming.

Ross

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

Re: [PATCH 1/1] epiphany: inherit perlnative

[Khem Raj]

On 8/5/16 1:20 AM, Burton, Ross wrote:
> 
> On 5 August 2016 at 03:47, Robert Yang <liezhi.yang@windriver.com
> <mailto:liezhi.yang@windriver.com>> wrote:
> 
>     I can reproduce the problem every time on my host Ubuntu 14.04 64bit:
> 
> 
> Just discovered that ephy for some reason ships its own copies of
> intltool.m4 (and a few others).  We'll need to delete those.  Patch
> incoming.

ah good find. I was still seeing the errors on my boxes.
> 
> Ross

Re: [PATCH 1/1] epiphany: inherit perlnative

[Burton, Ross]

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

On 5 August 2016 at 15:49, Khem Raj <raj.khem@gmail.com> wrote:

> > Just discovered that ephy for some reason ships its own copies of
> > intltool.m4 (and a few others).  We'll need to delete those.  Patch
> > incoming.
>
> ah good find. I was still seeing the errors on my boxes.
>

I've now a few patches locally that I'll send shortly, and want to change
autoconf a little too.

Ross

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