Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-core@lists.openembedded.org,
	Richard Tollerton <rich.tollerton@ni.com>
Cc: openembedded-commits@lists.openembedded.org
Subject: Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH
Date: Thu, 8 Jan 2015 13:01:55 +0100	[thread overview]
Message-ID: <20150108120155.GH3972@jama> (raw)
In-Reply-To: <20150107141256.GM29986@jama>

[-- Attachment #1: Type: text/plain, Size: 3114 bytes --]

On Wed, Jan 07, 2015 at 03:12:56PM +0100, Martin Jansa wrote:
> On Fri, Dec 19, 2014 at 06:08:41PM +0000, git@git.openembedded.org wrote:
> > Module: openembedded-core.git
> > Branch: master
> > Commit: 89a29a3ad0742cd713e739d3d460be7711966679
> > URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=89a29a3ad0742cd713e739d3d460be7711966679
> > 
> > Author: Richard Tollerton <rich.tollerton@ni.com>
> > Date:   Fri Dec 12 13:34:00 2014 -0600
> > 
> > font-util: Fix incorrect PKG_CONFIG_PATH
> > 
> > PKG_CONFIG_PATH always defaults to /usr/lib/pkgconfig, and the host
> > /usr/lib/pkgconfig is always checked as a fallback; however,
> > PKG_CONFIG_PATH is currently (incorrectly) set to /usr/lib/pkg-config in
> > the sysroot, which doesn't exist. On host distros where the font
> 
> 
> > encoding maps are stored under a different path than OE, this will break
> > font builds, because ucs2any will attempt to read the sysroot's encoding
> > maps with the host paths.
> 
> ^ Is this description of what the patch is supposed to fix or expected
> side-effect of this change?
> 
> Because all font recipes and meta-oe are failing since this change with
> errors like this:
> 
> /home/jenkins/oe/world/shr-core/tmp-glibc/sysroots/x86_64-linux/usr/bin/ucs2any: Can't read mapping file
> '/home/jenkins/oe/world/shr-core/tmp-glibc/sysroots/x86_64-linux/usr/share/fonts/X11/util/map-ISO8859-1': No such file or directory!
> 
> See http://www.openembedded.org/wiki/Bitbake_World_Status

I can confirm that reverting this patch fixes font-* recipes
http://lists.openembedded.org/pipermail/openembedded-core/2015-January/100523.html

> 
> > 
> > Signed-off-by: Richard Tollerton <rich.tollerton@ni.com>
> > Signed-off-by: Ross Burton <ross.burton@intel.com>
> > 
> > ---
> > 
> >  meta/recipes-graphics/xorg-font/font-util_1.3.0.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta/recipes-graphics/xorg-font/font-util_1.3.0.bb b/meta/recipes-graphics/xorg-font/font-util_1.3.0.bb
> > index 8b42991..cc4258a 100644
> > --- a/meta/recipes-graphics/xorg-font/font-util_1.3.0.bb
> > +++ b/meta/recipes-graphics/xorg-font/font-util_1.3.0.bb
> > @@ -17,7 +17,7 @@ RDEPENDS_${PN}_class-native = "mkfontdir-native mkfontscale-native"
> >  PR = "${INC_PR}.0"
> >  
> >  do_configure_prepend() {
> > -        sed -i "s#MAPFILES_PATH=\`pkg-config#MAPFILES_PATH=\`PKG_CONFIG_PATH=\"${STAGING_LIBDIR_NATIVE}/pkg-config\" pkg-config#g" ${S}/fontutil.m4.in
> > +        sed -i "s#MAPFILES_PATH=\`pkg-config#MAPFILES_PATH=\`PKG_CONFIG_PATH=\"${STAGING_LIBDIR_NATIVE}/pkgconfig\" pkg-config#g" ${S}/fontutil.m4.in
> >  }
> >  
> >  BBCLASSEXTEND = "native"
> > 
> > -- 
> > _______________________________________________
> > Openembedded-commits mailing list
> > Openembedded-commits@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-commits
> 
> -- 
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com



-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]

  reply	other threads:[~2015-01-08 12:01 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20141219180841.C5F5A503BF@opal.openembedded.org>
2015-01-07 14:12 ` [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH Martin Jansa
2015-01-08 12:01   ` Martin Jansa [this message]
2015-01-08 20:00     ` Richard Tollerton
2015-01-28 13:40     ` Martin Jansa
2015-01-29  9:21       ` Richard Tollerton
2015-02-02 18:57         ` Paul Eggleton
2015-02-03  1:05           ` Richard Tollerton
2015-02-03 16:02             ` Burton, Ross
2015-02-20 12:45               ` Martin Jansa
2015-02-20 12:50                 ` Richard Purdie
2015-02-20 14:36                   ` Martin Jansa
2015-02-20 15:27                     ` Ben Shelton
2015-02-20 19:52                       ` Ben Shelton
2015-02-20 20:13                         ` Burton, Ross

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=20150108120155.GH3972@jama \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-commits@lists.openembedded.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=rich.tollerton@ni.com \
    /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