All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: "Jens Rehsack" <rehsack@gmail.com>,
	"Aníbal Limón" <anibal.limon@linux.intel.com>
Cc: "yocto@yoctoproject.org" <yocto@yoctoproject.org>
Subject: Re: [PATCH] Perl: fix PERL5LIB settings
Date: Thu, 20 Nov 2014 15:16:17 +0000	[thread overview]
Message-ID: <1416496577.2068.3.camel@linuxfoundation.org> (raw)
In-Reply-To: <37719788-60D1-4C79-BCC4-4F4FA3DFDEB1@gmail.com>

On Sat, 2014-11-08 at 13:47 +0100, Jens Rehsack wrote:
> Am 07.11.2014 um 12:55 schrieb Burton, Ross <ross.burton@intel.com>:
> 
> > Hi Wolfgang,
> > 
> > On 5 November 2014 19:29, Wolfgang Denk <wd@denx.de> wrote:
> >  meta/recipes-devtools/perl/perl-native_5.14.3.bb | 4 ++--
> >  meta/recipes-devtools/perl/perl_5.14.3.bb        | 2 +-
> >  2 files changed, 3 insertions(+), 3 deletions(-)
> > 
> > Perl was upgraded to 5.20.0 in July, can you rebase and verify that this is still required?
> 
> I can finally confirm that it fixes the issues in daisy (bitbaking Dancer2 using https://github.com/rehsack/meta-cpan/tree/master/recipes-www/dancer2-perl).
> You can try whether Dancer2 bitbakes for dizzy and/or master to verify - for me it's impossible to create a dizzy build - to many daisy adoptions and release pressure for next weeks.
> If you know someone having dizzy - bitbake Dancer2, Unix::Statgrab or alike to prove ...

I was asked why this patch hasn't gone in. I agree there is a problem
here which needs fixing. The trouble is that it doesn't easily apply to
master, we don't have many perl experts to review this and patches
against oe-core really need to go to the openembedded-core mailing list.

I believe Aníbal did fix part of the problem with:

http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=6839a39912d483c6a403bd52e5f889547f1f4807

which leaves the perl-native part. I don't normally do this however I've
updated the remaining part of the patch to apply against master and
included it in master-next since I hate to see things getting lost.

Cheers,

Richard




      parent reply	other threads:[~2014-11-20 15:21 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-03  8:33 PERL5LIB not properly reflecting target_sdk_dir Joseph Andrew de la Peña
2014-10-08  6:05 ` Joseph Andrew de la Peña
2014-10-27  8:07   ` Wolfgang Denk
2014-11-02 23:35     ` Joseph Andrew de la Peña
2014-11-05 19:29     ` [PATCH] Perl: fix PERL5LIB settings Wolfgang Denk
2014-11-07 11:55       ` Burton, Ross
2014-11-08 12:47         ` Jens Rehsack
2014-11-20 14:37           ` Jens Rehsack
2014-11-20 15:32             ` Richard Purdie
2014-11-20 15:16           ` Richard Purdie [this message]

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=1416496577.2068.3.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=anibal.limon@linux.intel.com \
    --cc=rehsack@gmail.com \
    --cc=yocto@yoctoproject.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.