Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Martin Jansa]

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

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

> 
> 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

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

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Martin Jansa]

[-- 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 --]

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Richard Tollerton]

Martin Jansa <martin.jansa@gmail.com> writes:

> 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:
>> > 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

Thanks for the heads-up, I've reproduced it and will get back with a
fix. How soon do you need it to avoid a revert?

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

-- 
Richard Tollerton <rich.tollerton@ni.com>

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Martin Jansa]

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

On Thu, Jan 08, 2015 at 01:01:55PM +0100, Martin Jansa wrote:
> 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

Ping

> > > 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



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

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

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Richard Tollerton]

Martin Jansa <martin.jansa@gmail.com> writes:

> On Thu, Jan 08, 2015 at 01:01:55PM +0100, Martin Jansa wrote:
> > 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
>
> Ping

ACK. Sorry, I could've sworn I saw the commit get reverted on
master-next, so I figured this was a lower priority than I indicated in
my previous reply.

This build is broken because my patchset was only partially committed.
"font-util: Fix incorrect PKG_CONFIG_PATH" (the patch you reference
above) was patch 2/3, but patches 1/3 "font-util: remove MAPFILES_PATH
override" and 3/3 "xorg-font-common: fix sysroot injection for encoding
maps" appear to have never been committed. See original thread at e.g.
http://thread.gmane.org/gmane.comp.handhelds.openembedded.core/58846.

Could you try rebuilding with those other two patches?

> > > > 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
>
>
>
> -- 
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

-- 
Richard Tollerton <rich.tollerton@ni.com>

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Paul Eggleton]

On Thursday 29 January 2015 03:21:31 Richard Tollerton wrote:
> Martin Jansa <martin.jansa@gmail.com> writes:
> > On Thu, Jan 08, 2015 at 01:01:55PM +0100, Martin Jansa wrote:
> > > 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=89a
> > > > > 29a3ad0742cd713e739d3d460be7711966679
> > > > > 
> > > > > 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/bi
> > > > n/ucs2any: Can't read mapping file
> > > > '/home/jenkins/oe/world/shr-core/tmp-glibc/sysroots/x86_64-linux/usr/s
> > > > hare/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/1
> > > 00523.html> 
> > Ping
> 
> ACK. Sorry, I could've sworn I saw the commit get reverted on
> master-next, so I figured this was a lower priority than I indicated in
> my previous reply.
> 
> This build is broken because my patchset was only partially committed.
> "font-util: Fix incorrect PKG_CONFIG_PATH" (the patch you reference
> above) was patch 2/3, but patches 1/3 "font-util: remove MAPFILES_PATH
> override" and 3/3 "xorg-font-common: fix sysroot injection for encoding
> maps" appear to have never been committed. See original thread at e.g.
> http://thread.gmane.org/gmane.comp.handhelds.openembedded.core/58846.
> 
> Could you try rebuilding with those other two patches?

If these did indeed get missed, could you please rebase them on top of master 
and resend? Then we can resolve this once and for all.

Thanks,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Richard Tollerton]

Paul Eggleton <paul.eggleton@linux.intel.com> writes:

> On Thursday 29 January 2015 03:21:31 Richard Tollerton wrote:
>> Martin Jansa <martin.jansa@gmail.com> writes:
>> > On Thu, Jan 08, 2015 at 01:01:55PM +0100, Martin Jansa wrote:
>> > > 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=89a
>> > > > > 29a3ad0742cd713e739d3d460be7711966679
>> > > > > 
>> > > > > 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/bi
>> > > > n/ucs2any: Can't read mapping file
>> > > > '/home/jenkins/oe/world/shr-core/tmp-glibc/sysroots/x86_64-linux/usr/s
>> > > > hare/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/1
>> > > 00523.html> 
>> > Ping
>> 
>> ACK. Sorry, I could've sworn I saw the commit get reverted on
>> master-next, so I figured this was a lower priority than I indicated in
>> my previous reply.
>> 
>> This build is broken because my patchset was only partially committed.
>> "font-util: Fix incorrect PKG_CONFIG_PATH" (the patch you reference
>> above) was patch 2/3, but patches 1/3 "font-util: remove MAPFILES_PATH
>> override" and 3/3 "xorg-font-common: fix sysroot injection for encoding
>> maps" appear to have never been committed. See original thread at e.g.
>> http://thread.gmane.org/gmane.comp.handhelds.openembedded.core/58846.
>> 
>> Could you try rebuilding with those other two patches?
>
> If these did indeed get missed, could you please rebase them on top of master 
> and resend? Then we can resolve this once and for all.

Done (and sent), and my apologies, because this exercise has made clear
that I was wrong in several of the details here: None of the three
patches were committed; I was confusing one of them with 89a29a3a, which
was from a separate email. The confusion originated in 89a29a3a having
been committed after I had already withdrawn it, which I forgot about
over the holidays.

So, to summarize: I believe that applying all three patches of
"xorg-font: stop trying to use build machine's encoding maps" should fix
the builds, except for the font-util dependency described in that cover
letter, which I will submit patches for in meta-oe.

> Thanks,
> Paul
>
> -- 
>
> Paul Eggleton
> Intel Open Source Technology Centre

-- 
Richard Tollerton <rich.tollerton@ni.com>

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Burton, Ross]

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

On 3 February 2015 at 01:05, Richard Tollerton <rich.tollerton@ni.com>
wrote:

> Done (and sent), and my apologies, because this exercise has made clear
> that I was wrong in several of the details here: None of the three
> patches were committed; I was confusing one of them with 89a29a3a, which
> was from a separate email. The confusion originated in 89a29a3a having
> been committed after I had already withdrawn it, which I forgot about
> over the holidays.
>

That's probably my fault, sorry.  Thanks for following up and resolving
this properly.

Ross

[-- Attachment #2: Type: text/html, Size: 975 bytes --]

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Martin Jansa]

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

On Tue, Feb 03, 2015 at 04:02:36PM +0000, Burton, Ross wrote:
> On 3 February 2015 at 01:05, Richard Tollerton <rich.tollerton@ni.com>
> wrote:
> 
> > Done (and sent), and my apologies, because this exercise has made clear
> > that I was wrong in several of the details here: None of the three
> > patches were committed; I was confusing one of them with 89a29a3a, which
> > was from a separate email. The confusion originated in 89a29a3a having
> > been committed after I had already withdrawn it, which I forgot about
> > over the holidays.
> >
> 
> That's probably my fault, sorry.  Thanks for following up and resolving
> this properly.

Can you please merge those fixes before 1.8 is closed? Or at least
revert the original patch which shouldn't be merged, until your comment
for 1/3 is resolved.

FWIW I haven't noticed v4 to resolve:
https://www.mail-archive.com/openembedded-core@lists.openembedded.org/msg61018.html

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

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

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Richard Purdie]

On Fri, 2015-02-20 at 13:45 +0100, Martin Jansa wrote:
> On Tue, Feb 03, 2015 at 04:02:36PM +0000, Burton, Ross wrote:
> > On 3 February 2015 at 01:05, Richard Tollerton <rich.tollerton@ni.com>
> > wrote:
> > 
> > > Done (and sent), and my apologies, because this exercise has made clear
> > > that I was wrong in several of the details here: None of the three
> > > patches were committed; I was confusing one of them with 89a29a3a, which
> > > was from a separate email. The confusion originated in 89a29a3a having
> > > been committed after I had already withdrawn it, which I forgot about
> > > over the holidays.
> > >
> > 
> > That's probably my fault, sorry.  Thanks for following up and resolving
> > this properly.
> 
> Can you please merge those fixes before 1.8 is closed? Or at least
> revert the original patch which shouldn't be merged, until your comment
> for 1/3 is resolved.
> 
> FWIW I haven't noticed v4 to resolve:
> https://www.mail-archive.com/openembedded-core@lists.openembedded.org/msg61018.html

We don't have the "right" patch for this yet. I don't want the fonts in
the sysroots, I agree we need to resolve the other issues though. I also
agree this needs fixing for 1.8, thanks for bringing it up.

Cheers,

Richard

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Martin Jansa]

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

On Fri, Feb 20, 2015 at 12:50:03PM +0000, Richard Purdie wrote:
> On Fri, 2015-02-20 at 13:45 +0100, Martin Jansa wrote:
> > On Tue, Feb 03, 2015 at 04:02:36PM +0000, Burton, Ross wrote:
> > > On 3 February 2015 at 01:05, Richard Tollerton <rich.tollerton@ni.com>
> > > wrote:
> > > 
> > > > Done (and sent), and my apologies, because this exercise has made clear
> > > > that I was wrong in several of the details here: None of the three
> > > > patches were committed; I was confusing one of them with 89a29a3a, which
> > > > was from a separate email. The confusion originated in 89a29a3a having
> > > > been committed after I had already withdrawn it, which I forgot about
> > > > over the holidays.
> > > >
> > > 
> > > That's probably my fault, sorry.  Thanks for following up and resolving
> > > this properly.
> > 
> > Can you please merge those fixes before 1.8 is closed? Or at least
> > revert the original patch which shouldn't be merged, until your comment
> > for 1/3 is resolved.
> > 
> > FWIW I haven't noticed v4 to resolve:
> > https://www.mail-archive.com/openembedded-core@lists.openembedded.org/msg61018.html
> 
> We don't have the "right" patch for this yet. I don't want the fonts in
> the sysroots, I agree we need to resolve the other issues though. I also
> agree this needs fixing for 1.8, thanks for bringing it up.

Can you please revert:
  commit 89a29a3ad0742cd713e739d3d460be7711966679
  Author: Richard Tollerton <rich.tollerton@ni.com>
  Date:   Fri Dec 12 13:34:00 2014 -0600

    font-util: Fix incorrect PKG_CONFIG_PATH

?

It doesn't do anything useful as Richard confirmed by withdrawing it and
all font recipes are broken since it was merged.

Regards,

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

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

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Ben Shelton]

> 
> Can you please revert:
>   commit 89a29a3ad0742cd713e739d3d460be7711966679
>   Author: Richard Tollerton <rich.tollerton@ni.com>
>   Date:   Fri Dec 12 13:34:00 2014 -0600
> 
>     font-util: Fix incorrect PKG_CONFIG_PATH
> 
> ?
> 
> It doesn't do anything useful as Richard confirmed by withdrawing it and
> all font recipes are broken since it was merged.
> 
> Regards,
> 
> -- 
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

Hi all,

Is there any chance I could get you to wait until the end of the day to
do any reverts?  Rich is out right now, but I'm going to take a look at
the issue pointed out in the v3 patches and see if I can get it worked
out.

Thanks,
Ben

> -- 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Ben Shelton]

On 02/20, Ben Shelton wrote:
> > 
> > Can you please revert:
> >   commit 89a29a3ad0742cd713e739d3d460be7711966679
> >   Author: Richard Tollerton <rich.tollerton@ni.com>
> >   Date:   Fri Dec 12 13:34:00 2014 -0600
> > 
> >     font-util: Fix incorrect PKG_CONFIG_PATH
> > 
> > ?
> > 
> > It doesn't do anything useful as Richard confirmed by withdrawing it and
> > all font recipes are broken since it was merged.
> > 
> > Regards,
> > 
> > -- 
> > Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
> 
> Hi all,
> 
> Is there any chance I could get you to wait until the end of the day to
> do any reverts?  Rich is out right now, but I'm going to take a look at
> the issue pointed out in the v3 patches and see if I can get it worked
> out.
> 
> Thanks,
> Ben
> 

I submitted v4 of the patch series.  Note that with these three patches,
the commit that's in question here, 'font-util: Fix incorrect
PKG_CONFIG_PATH', is still needed and should not be reverted.

With these patches, I successfully built xorg-fonts-100dpi from
meta-oe, which previously failed.

Best,
Ben

Re: [oe-commits] Richard Tollerton : font-util: Fix incorrect PKG_CONFIG_PATH

[Burton, Ross]

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

On 20 February 2015 at 19:52, Ben Shelton <ben.shelton@ni.com> wrote:

> I submitted v4 of the patch series.  Note that with these three patches,
> the commit that's in question here, 'font-util: Fix incorrect
> PKG_CONFIG_PATH', is still needed and should not be reverted.
>

And they'll be running on the AB shortly, thanks Ben!

Ross

[-- Attachment #2: Type: text/html, Size: 769 bytes --]