Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Phil Blundell <philb@gnu.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 1/2] libc-package bbclass: fix binary localedata dependency code
Date: Fri, 26 Aug 2011 16:30:31 +0100	[thread overview]
Message-ID: <1314372632.19905.93.camel@phil-desktop> (raw)
In-Reply-To: <EB5E7A93-679D-4108-8E6D-03D7E1865908@dominion.thruhere.net>

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

On Fri, 2011-08-26 at 17:26 +0200, Koen Kooi wrote:
> And that gives me http://git.angstrom-distribution.org/cgi-bin/cgit.cgi/testlab/commit/?h=yocto&id=9b2e08e0c4da4881ba8c83be39697a40a63b75fd
> 
> -locale_base_en_us -> eglibc_binary_localedata_en_us;
> +locale_base_en_us -> eglibc_binary_localedata_en_u_;
> 
> So something regexp is being a bit too greedy by the looks of it.

Ah whoops, the old missing backslash.  Could you try this one instead?

p.


[-- Attachment #2: 0001-libc-package-restore-correct-mangling-behavior-for-l.patch --]
[-- Type: text/x-patch, Size: 1408 bytes --]

From 1cd91a91802776ddede041cfa222b603da6c8453 Mon Sep 17 00:00:00 2001
From: Phil Blundell <philb@gnu.org>
Date: Fri, 26 Aug 2011 15:05:11 +0100
Subject: [PATCH v2] libc-package: restore correct mangling behavior for locale names

This reverts 19fb07bf337e1d724798e2eb4479c35fc45b1941 and restores
the behaviour of the code to the way it was prior to 561d875404ef1783f94f37314b6e756766db8411.

See http://lists.linuxtogo.org/pipermail/openembedded-core/2011-August/007407.html and subsequent messages.

Signed-off-by: Phil Blundell <philb@gnu.org>
---
 meta/classes/libc-package.bbclass |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/libc-package.bbclass b/meta/classes/libc-package.bbclass
index d3b33d6..f8df6a4 100644
--- a/meta/classes/libc-package.bbclass
+++ b/meta/classes/libc-package.bbclass
@@ -241,10 +241,9 @@ python package_do_split_gconvs () {
 		(locale, encoding, locale), d)
 
 	def output_locale_binary_rdepends(name, pkgname, locale, encoding):
-		m = re.match("(.*)_(.*)", name)
+		m = re.match("(.*)\.(.*)", name)
 		if m:
-			libc_name = "%s-%s" % (m.group(1), m.group(2).lower().replace("-",""))
+			libc_name = "%s.%s" % (m.group(1), m.group(2).lower().replace("-",""))
 		else:
 			libc_name = name
 		bb.data.setVar('RDEPENDS_%s' % pkgname, legitimize_package_name('%s-binary-localedata-%s' \
-- 
1.7.4.1


  reply	other threads:[~2011-08-26 15:35 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-24 18:32 [PATCH 1/2] libc-package bbclass: fix binary localedata dependency code Koen Kooi
2011-08-24 18:32 ` [PATCH 2/2] tcl: fix networking breakage Koen Kooi
2011-08-24 18:36   ` Saul Wold
2011-08-24 19:45     ` Koen Kooi
2011-08-25  2:26 ` [PATCH 1/2] libc-package bbclass: fix binary localedata dependency code Richard Purdie
2011-08-25  9:24 ` Phil Blundell
2011-08-25 14:58   ` Richard Purdie
2011-08-26 14:12     ` Phil Blundell
2011-08-26 14:54       ` Koen Kooi
2011-08-26 15:26       ` Koen Kooi
2011-08-26 15:30         ` Phil Blundell [this message]
2011-08-26 16:31           ` Koen Kooi
2011-08-30  9:26             ` Phil Blundell
2011-08-31 13:47               ` Koen Kooi
2011-08-31 17:05               ` Richard Purdie

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=1314372632.19905.93.camel@phil-desktop \
    --to=philb@gnu.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