All of lore.kernel.org
 help / color / mirror / Atom feed
From: Saul Wold <sgw@linux.intel.com>
To: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 4/8] Revert "Add ptest for glib."
Date: Tue, 05 Feb 2013 11:59:08 -0800	[thread overview]
Message-ID: <5111648C.5010301@linux.intel.com> (raw)
In-Reply-To: <CAC1BbcSeJ_yg7Cqoai=aoP+w_ba+MjkZ=QWdPJQAjWfPFCo1OQ@mail.gmail.com>

On 02/05/2013 11:36 AM, Bernhard Reutner-Fischer wrote:
> On 5 February 2013 20:16, Bernhard Reutner-Fischer
> <rep.dot.nop@gmail.com> wrote:
>> On 5 February 2013 19:39, Saul Wold <sgw@linux.intel.com> wrote:
>>> On 02/05/2013 09:13 AM, Bernhard Reutner-Fischer wrote:
>>>>
>>>> On 5 February 2013 17:46, Saul Wold <sgw@linux.intel.com> wrote:
>>>>>
>>>>> On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
>>>>>>
>>>>>>
>>>>>> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>>>>>>
>>>>>> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>>>>>>
>>>>> What's the issue?  Is it fixable instead of reverting the ptest patch? I
>>>>
>>>>
>>>> The issue is that it incorrectly and unconditionally pulls in eglibc
>>>> stuff:
>>>>
>>>> -RDEPENDS_${PN}-ptest += "\
>>>> -            eglibc-gconv-utf-16 \
>>>> -            eglibc-charmap-utf-8 \
>>>> -            eglibc-gconv-cp1255 \
>>>> -            eglibc-charmap-cp1255 \
>>>> -            eglibc-gconv-utf-32 \
>>>> -            eglibc-gconv-utf-7 \
>>>> -            eglibc-charmap-invariant \
>>>> -            eglibc-localedata-translit-cjk-variants \
>>>> -            tzdata \
>>>> -            tzdata-americas \
>>>> -            tzdata-asia \
>>>> -            tzdata-europe \
>>>> -            tzdata-posix \
>>>> -            python-pygobject \
>>>> -            python-dbus \
>>>> -           "
>>>>
>>> Bjorn,
>>>
>>> Do you have some time to look at this and see if you can make use of TCLIBC
>>> to use the correct library and locale package info based on which libc is
>>> being used?
>>
>> would most likely DTRT, yep.
>
> Well, given that i see e.g. glibc-gconv-utf-16 nowhere in
> openembedded-core i suspect they are misplaced and should live in the
> layer that provides them.
>
It's a DYNAMIC package built in eglibc-locale.inc.

>> I'm curious what you would need from uClibc to satisfy that ptest thing?
>> What does it do, what is it good for? :)
>
Bjorn question at this point!

> Is e.g. glibc-gconv-utf-16 locale support or something like that?
>
probably, could the uclibc-gconv provide equivalent?

Sau!


> thanks,
>>
>> RDEPENDS_${PN}-ptest += "\
>>              tzdata \
>>              tzdata-americas \
>>              tzdata-asia \
>>              tzdata-europe \
>>              tzdata-posix \
>>              python-pygobject \
>>              python-dbus \
>>             "
>> RDEPENDS_${PN}-ptest_append_libc-eglibc = "\
>>              eglibc-gconv-utf-16 \
>>              eglibc-charmap-utf-8 \
>>              eglibc-gconv-cp1255 \
>>              eglibc-charmap-cp1255 \
>>              eglibc-gconv-utf-32 \
>>              eglibc-gconv-utf-7 \
>>              eglibc-charmap-invariant \
>>              eglibc-localedata-translit-cjk-variants \
>>             "
>> RDEPENDS_${PN}-ptest_append_libc-eglibc = "did not look"
>>
>> TIA && cheers,
>
>



  reply	other threads:[~2013-02-05 20:15 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 1/8] base.bbclass: Commentary typo fix Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 2/8] uclibc: add c6x arch Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 3/8] uclibc: turn on kernel module support Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 4/8] Revert "Add ptest for glib." Bernhard Reutner-Fischer
2013-02-05 16:44   ` Richard Purdie
2013-02-05 16:44   ` Burton, Ross
2013-02-05 16:47     ` Bernhard Reutner-Fischer
2013-02-05 16:59       ` Burton, Ross
2013-02-05 17:17         ` Bernhard Reutner-Fischer
2013-02-08  8:21           ` Björn Stenberg
2013-02-05 16:46   ` Saul Wold
2013-02-05 17:13     ` Bernhard Reutner-Fischer
2013-02-05 18:39       ` Saul Wold
2013-02-05 19:16         ` Bernhard Reutner-Fischer
2013-02-05 19:36           ` Bernhard Reutner-Fischer
2013-02-05 19:59             ` Saul Wold [this message]
2013-02-06  0:01               ` Khem Raj
2013-02-06  9:02         ` Björn Stenberg
2013-02-05 15:55 ` [PATCH 5/8] busybox: add ifup's ifstate dir to package Bernhard Reutner-Fischer
2013-02-12  5:46   ` Saul Wold
2013-02-12  9:14     ` Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 6/8] busybox: update-modules script needs id(1) Bernhard Reutner-Fischer
2013-02-05 16:38   ` Richard Purdie
2013-02-05 16:46     ` Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 7/8] base.bbclass: remove redundant setting of MAKE Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 8/8] create-pull-request: Error message on missing -u Bernhard Reutner-Fischer

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=5111648C.5010301@linux.intel.com \
    --to=sgw@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=rep.dot.nop@gmail.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 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.