All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: "Cooper Jr., Franklin" <fcooper@ti.com>
Cc: "meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary)
Date: Wed, 22 Aug 2012 17:51:16 -0400	[thread overview]
Message-ID: <20120822215116.GA8846@denix.org> (raw)
In-Reply-To: <8F29D6B095ED194EA1980491A5E029710C2A7DE4@DFLE09.ent.ti.com>

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
> 


  reply	other threads:[~2012-08-22 21:51 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-20 16:04 [meta-arago-distro][PATCH] php: Add libiconv as a dependency. (Temporary) Franklin S Cooper Jr
2012-08-21 18:15 ` Denys Dmytriyenko
2012-08-22  1:40   ` Cooper Jr., Franklin
2012-08-22 21:51     ` Denys Dmytriyenko [this message]
2012-08-22 22:13       ` Cooper Jr., Franklin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120822215116.GA8846@denix.org \
    --to=denys@ti.com \
    --cc=fcooper@ti.com \
    --cc=meta-arago@arago-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.