[meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)

[meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)

[Franklin S Cooper Jr]

* Adding libiconv as a dependency since external-linaro-toolchain.bb who
  PROVIDES virtual/libiconv isn't providing the needed files for PHP.
* Configuring PHP fails with
  /home/<user>/toolchain/gcc-linaro-arm-linux-gnueabi-2012.03-20120326_linux/bin/../
  lib/gcc/arm-linux-gnueabi/4.6.3/../../../../arm-linux-gnueabi/bin/ld:
  cannot find -liconv collect2: ld returned 1 exit status
* This should be a temporary fix until a proper fix is made in the toolchain.

Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
---
 .../recipes-devtools/php/php_5.3.6.bbappend        |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
index 0d16ebe..e30664d 100644
--- a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
+++ b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
@@ -4,3 +4,5 @@ FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
 PR_append = "-arago0"
 
 SRC_URI += "file://0001-php-native-Fix-host-contamination-issue.patch"
+
+DEPENDS += "libiconv"
-- 
1.7.0.4

Re: [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)

[Denys Dmytriyenko]

On Mon, Aug 20, 2012 at 11:04:43AM -0500, Franklin S Cooper Jr wrote:
> * Adding libiconv as a dependency since external-linaro-toolchain.bb who
>   PROVIDES virtual/libiconv isn't providing the needed files for PHP.
> * Configuring PHP fails with
>   /home/<user>/toolchain/gcc-linaro-arm-linux-gnueabi-2012.03-20120326_linux/bin/../
>   lib/gcc/arm-linux-gnueabi/4.6.3/../../../../arm-linux-gnueabi/bin/ld:
>   cannot find -liconv collect2: ld returned 1 exit status
> * This should be a temporary fix until a proper fix is made in the toolchain.

When the first php patch is only applied to -native, this issue never happens, 
as it's masked by configure script satisfying the check with the host iconv 
library. It's not correct, but it's only a check and the host library is not 
actually being linked during the build...

-- 
Denys


> Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
> ---
>  .../recipes-devtools/php/php_5.3.6.bbappend        |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> index 0d16ebe..e30664d 100644
> --- a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> +++ b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> @@ -4,3 +4,5 @@ FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>  PR_append = "-arago0"
>  
>  SRC_URI += "file://0001-php-native-Fix-host-contamination-issue.patch"
> +
> +DEPENDS += "libiconv"
> -- 
> 1.7.0.4
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> 

Re: [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)

[Cooper Jr., Franklin]

I agree with your suggestion did you want me to send the patch or tweak it like you did for my other patch? 



-----Original Message-----
From: Dmytriyenko, Denys 
Sent: Tuesday, August 21, 2012 1:16 PM
To: Cooper Jr., Franklin
Cc: meta-arago@arago-project.org
Subject: Re: [meta-arago] [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)

On Mon, Aug 20, 2012 at 11:04:43AM -0500, Franklin S Cooper Jr wrote:
> * Adding libiconv as a dependency since external-linaro-toolchain.bb who
>   PROVIDES virtual/libiconv isn't providing the needed files for PHP.
> * Configuring PHP fails with
>   /home/<user>/toolchain/gcc-linaro-arm-linux-gnueabi-2012.03-20120326_linux/bin/../
>   lib/gcc/arm-linux-gnueabi/4.6.3/../../../../arm-linux-gnueabi/bin/ld:
>   cannot find -liconv collect2: ld returned 1 exit status
> * This should be a temporary fix until a proper fix is made in the toolchain.

When the first php patch is only applied to -native, this issue never happens, 
as it's masked by configure script satisfying the check with the host iconv 
library. It's not correct, but it's only a check and the host library is not 
actually being linked during the build...

-- 
Denys


> Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
> ---
>  .../recipes-devtools/php/php_5.3.6.bbappend        |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> index 0d16ebe..e30664d 100644
> --- a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> +++ b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> @@ -4,3 +4,5 @@ FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>  PR_append = "-arago0"
>  
>  SRC_URI += "file://0001-php-native-Fix-host-contamination-issue.patch"
> +
> +DEPENDS += "libiconv"
> -- 
> 1.7.0.4
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> 

Re: [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)

[Denys Dmytriyenko]

On Wed, Aug 22, 2012 at 01:40:22AM +0000, Cooper Jr., Franklin wrote:
> I agree with your suggestion did you want me to send the patch or tweak it 
> like you did for my other patch?

No, can you please check if you still have this issue when building php for 
the target:

| cannot find -liconv collect2: ld returned 1 exit status

-- 
Denys

> -----Original Message-----
> From: Dmytriyenko, Denys 
> Sent: Tuesday, August 21, 2012 1:16 PM
> To: Cooper Jr., Franklin
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)
> 
> On Mon, Aug 20, 2012 at 11:04:43AM -0500, Franklin S Cooper Jr wrote:
> > * Adding libiconv as a dependency since external-linaro-toolchain.bb who
> >   PROVIDES virtual/libiconv isn't providing the needed files for PHP.
> > * Configuring PHP fails with
> >   /home/<user>/toolchain/gcc-linaro-arm-linux-gnueabi-2012.03-20120326_linux/bin/../
> >   lib/gcc/arm-linux-gnueabi/4.6.3/../../../../arm-linux-gnueabi/bin/ld:
> >   cannot find -liconv collect2: ld returned 1 exit status
> > * This should be a temporary fix until a proper fix is made in the toolchain.
> 
> When the first php patch is only applied to -native, this issue never happens, 
> as it's masked by configure script satisfying the check with the host iconv 
> library. It's not correct, but it's only a check and the host library is not 
> actually being linked during the build...
> 
> -- 
> Denys
> 
> 
> > Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
> > ---
> >  .../recipes-devtools/php/php_5.3.6.bbappend        |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> > index 0d16ebe..e30664d 100644
> > --- a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> > +++ b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
> > @@ -4,3 +4,5 @@ FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> >  PR_append = "-arago0"
> >  
> >  SRC_URI += "file://0001-php-native-Fix-host-contamination-issue.patch"
> > +
> > +DEPENDS += "libiconv"
> > -- 
> > 1.7.0.4
> > 
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> 

Re: [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)

[Cooper Jr., Franklin]

Before I left today I noticed this also but didn't have a chance to debug it to see why this is an issue all of a sudden. Before I sent the patch on Monday I did a cleanall for php php-native and iconv several times and had no issue building php. Last night while I was trying to debug qt issues I cleared my entire temp directory to clear out everything. The very first time ( before i left today) I tried to build php I received a bunch of errors saying various libiconv functions couldn't be referenced I didn't receive this issue for php-native so I am not sure what has changed since initially php native was the one having the issues.

On Aug 22, 2012, at 4:51 PM, "Dmytriyenko, Denys" <denys@ti.com> wrote:

> On Wed, Aug 22, 2012 at 01:40:22AM +0000, Cooper Jr., Franklin wrote:
>> I agree with your suggestion did you want me to send the patch or tweak it 
>> like you did for my other patch?
> 
> No, can you please check if you still have this issue when building php for 
> the target:
> 
> | cannot find -liconv collect2: ld returned 1 exit status
> 
> -- 
> Denys
> 
>> -----Original Message-----
>> From: Dmytriyenko, Denys 
>> Sent: Tuesday, August 21, 2012 1:16 PM
>> To: Cooper Jr., Franklin
>> Cc: meta-arago@arago-project.org
>> Subject: Re: [meta-arago] [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)
>> 
>> On Mon, Aug 20, 2012 at 11:04:43AM -0500, Franklin S Cooper Jr wrote:
>>> * Adding libiconv as a dependency since external-linaro-toolchain.bb who
>>>  PROVIDES virtual/libiconv isn't providing the needed files for PHP.
>>> * Configuring PHP fails with
>>>  /home/<user>/toolchain/gcc-linaro-arm-linux-gnueabi-2012.03-20120326_linux/bin/../
>>>  lib/gcc/arm-linux-gnueabi/4.6.3/../../../../arm-linux-gnueabi/bin/ld:
>>>  cannot find -liconv collect2: ld returned 1 exit status
>>> * This should be a temporary fix until a proper fix is made in the toolchain.
>> 
>> When the first php patch is only applied to -native, this issue never happens, 
>> as it's masked by configure script satisfying the check with the host iconv 
>> library. It's not correct, but it's only a check and the host library is not 
>> actually being linked during the build...
>> 
>> -- 
>> Denys
>> 
>> 
>>> Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
>>> ---
>>> .../recipes-devtools/php/php_5.3.6.bbappend        |    2 ++
>>> 1 files changed, 2 insertions(+), 0 deletions(-)
>>> 
>>> diff --git a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
>>> index 0d16ebe..e30664d 100644
>>> --- a/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
>>> +++ b/meta-arago-distro/recipes-devtools/php/php_5.3.6.bbappend
>>> @@ -4,3 +4,5 @@ FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>>> PR_append = "-arago0"
>>> 
>>> SRC_URI += "file://0001-php-native-Fix-host-contamination-issue.patch"
>>> +
>>> +DEPENDS += "libiconv"
>>> -- 
>>> 1.7.0.4
>>> 
>>> _______________________________________________
>>> meta-arago mailing list
>>> meta-arago@arago-project.org
>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>>> 
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>>