Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 1/2] shared-mime-info, avahi, polkit: inherit perlnative
Date: Tue, 09 Aug 2011 15:48:44 +0100	[thread overview]
Message-ID: <1312901324.14274.307.camel@rex> (raw)
In-Reply-To: <4E4097A3.8090204@gmail.com>

On Mon, 2011-08-08 at 19:12 -0700, Khem Raj wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 08/08/2011 04:24 PM, Richard Purdie wrote:
> > On Sun, 2011-08-07 at 23:14 -0700, Khem Raj wrote:
> >> Otherwise we end up with configure errors like below
> >> 
> >> | checking for perl... /usr/bin/perl | checking for perl >=
> >> 5.8.1... 5.12.4 | checking for XML::Parser... configure: error:
> >> XML::Parser perl module is required for intltool | + bbfatal
> >> 'oe_runconf failed' | + echo 'ERROR: oe_runconf failed' | ERROR:
> >> oe_runconf failed
> >> 
> >> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > 
> > We really need to tweak intltool's m4 file so we don't need to add
> > in this ton of dependencies unnecessarily...
> > 
> 
> intltool.m4 uses
> 
> AC_PATH_PROG(INTLTOOL_PERL,perl)
> 
> 
> if test -z "$INTLTOOL_PERL"; then
> 
> 
>    AC_MSG_ERROR([perl not found])
> 
> 
> fi
> 
> 
> 
> its using AC_PATH_PROG to find perl which is quite right I think.
> We somehow have to make sure that the path to perl-native
> is added into environment which does not happen unless
> we inherit perlnative as of now. and perlnative class is
> 
> PATH_prepend = "${STAGING_BINDIR_NATIVE}/perl-native:"
> 
> 
> DEPENDS += "perl -native"
> 
> 
> are you proposing that we hardcode path to perl-native in intltool
> macros and assume that perl-native is already built ? or something else

I was proposing we change 
AC_PATH_PROG(INTLTOOL_PERL,perl) 
to 
AC_PATH_PROG(INTLTOOL_PERL,nativeperl)

Cheers,

Richard






  reply	other threads:[~2011-08-09 14:53 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-08  6:14 [PATCH 1/2] shared-mime-info, avahi, polkit: inherit perlnative Khem Raj
2011-08-08  6:14 ` [PATCH 2/2] qemu: Poke more paths for presence of libgl Khem Raj
2011-08-09 14:21   ` Richard Purdie
2011-08-08 23:24 ` [PATCH 1/2] shared-mime-info, avahi, polkit: inherit perlnative Richard Purdie
2011-08-09  2:12   ` Khem Raj
2011-08-09 14:48     ` Richard Purdie [this message]
2011-08-09 15:17       ` Khem Raj

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=1312901324.14274.307.camel@rex \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox