All of lore.kernel.org
 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 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.