[PULL] fixes for bash and nfs-utils

[PULL] fixes for bash and nfs-utils

[Qing He]

this fixes bug 483 and bug 387.

Thanks,
Qing

---
 meta/recipes-connectivity/nfs-utils/nfs-utils/nfsserver |    6 +++---
 meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.2.bb  |    2 +-
 meta/recipes-extended/bash/bash.inc                     |    2 +-
 meta/recipes-extended/bash/bash_3.2.48.bb               |    2 +-
 meta/recipes-extended/bash/bash_4.1.bb                  |    2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

Qing He (2):
      bash: fix job control
      nfs-utils: fix nfs server initscript

Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=qhe/fix

Re: [PULL] fixes for bash and nfs-utils

[Richard Purdie]

On Wed, 2010-10-20 at 16:56 +0800, Qing He wrote:
> this fixes bug 483 and bug 387.
> 
> Thanks,
> Qing
> 
> ---
>  meta/recipes-connectivity/nfs-utils/nfs-utils/nfsserver |    6 +++---
>  meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.2.bb  |    2 +-
>  meta/recipes-extended/bash/bash.inc                     |    2 +-
>  meta/recipes-extended/bash/bash_3.2.48.bb               |    2 +-
>  meta/recipes-extended/bash/bash_4.1.bb                  |    2 +-
>  5 files changed, 7 insertions(+), 7 deletions(-)
> 
> Qing He (2):
>       bash: fix job control
>       nfs-utils: fix nfs server initscript
> 
> Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=qhe/fix

Merged into master, thanks!

Richard

[PULL] libtheora: add DEPENDS on libogg

[Cui, Dexuan]

$ scripts/create-pull-request -r 27d06858b5f3138039bddecc0eaaee1e05e21b48 -b dcui/master
Note: <commit_id> parameter assumed as 'HEAD'

 meta/recipes-multimedia/libtheora/libtheora_1.1.1.bb |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Dexuan Cui (1):
      libtheora: add DEPENDS on libogg

Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master

commit 2ecf311fbca970f6ea2d80accc3f925b097b664e
Author: Dexuan Cui <dexuan.cui@intel.com>
Date:   Thu Oct 21 14:54:34 2010 +0800

    libtheora: add DEPENDS on libogg

    This is used to fix the following build failure:

     checking for oggpackB_read... no
    | configure: error: newer libogg version (1.1 or later) required

    Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>

Re: [PULL] libtheora: add DEPENDS on libogg

[Richard Purdie]

On Fri, 2010-10-22 at 08:54 +0800, Cui, Dexuan wrote:
> $ scripts/create-pull-request -r 27d06858b5f3138039bddecc0eaaee1e05e21b48 -b dcui/master
> Note: <commit_id> parameter assumed as 'HEAD'
> 
>  meta/recipes-multimedia/libtheora/libtheora_1.1.1.bb |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> Dexuan Cui (1):
>       libtheora: add DEPENDS on libogg
> 
> Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master

Merged into master, thanks!

Richard

[PULL] busybox: enable more options for tar to support more file formats

[Cui, Dexuan]

$ scripts/create-pull-request -r 2ecf311fbca970f6ea2d80accc3f925b097b664e -b dcui/master
Note: <commit_id> parameter assumed as 'HEAD'

 meta/recipes-core/busybox/busybox-1.16.2/defconfig |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

Dexuan Cui (1):
      busybox: enable more options for tar to support more file formats.

Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master


commit b952fae48c45e39a14ec9c24b0ee544a741c3f69
Author: Dexuan Cui <dexuan.cui@intel.com>
Date:   Tue Oct 26 15:10:26 2010 +0800

    busybox: enable more options for tar to support more file formats.

    Fixes [BUGID #495]

    Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>

[PULL] fixes to eglibc and busybox

[Cui, Dexuan]

BTW: The fix to busybox is a RESEND since looks it was neglected. :-)

$ scripts/create-pull-request -r 2ecf311fbca970f6ea2d80accc3f925b097b664e -b dcui/master
Note: <commit_id> parameter assumed as 'HEAD'

 meta/recipes-core/busybox/busybox-1.16.2/defconfig |   14 +++++++-------
 meta/recipes-core/eglibc/eglibc-package.inc        |    2 +-
 meta/recipes-core/eglibc/eglibc_2.12.bb            |    2 +-
 3 files changed, 9 insertions(+), 9 deletions(-)

Dexuan Cui (2):
      busybox: enable more options for tar to support more file formats.
      eglibc: install the missing /etc/ld.so.conf into target

Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master

 
commit 61e77cb2e9a240110c50925d1b3f908d14b02f24
Author: Dexuan Cui <dexuan.cui@intel.com>
Date:   Tue Nov 2 15:42:09 2010 +0800

    eglibc: install the missing /etc/ld.so.conf into target

    ${sysconfdir} should be added into FILES_${PN}.

    Fixes [BUGID #511]

    Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>

commit b952fae48c45e39a14ec9c24b0ee544a741c3f69
Author: Dexuan Cui <dexuan.cui@intel.com>
Date:   Tue Oct 26 15:10:26 2010 +0800

    busybox: enable more options for tar to support more file formats.

    Fixes [BUGID #495]

    Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>

Re: [PULL] fixes to eglibc and busybox

[Mark Hatle]

On 11/3/10 7:55 PM, Cui, Dexuan wrote:
> BTW: The fix to busybox is a RESEND since looks it was neglected. :-)
>
> $ scripts/create-pull-request -r 2ecf311fbca970f6ea2d80accc3f925b097b664e -b dcui/master
> Note:<commit_id>  parameter assumed as 'HEAD'
>
>   meta/recipes-core/busybox/busybox-1.16.2/defconfig |   14 +++++++-------
>   meta/recipes-core/eglibc/eglibc-package.inc        |    2 +-
>   meta/recipes-core/eglibc/eglibc_2.12.bb            |    2 +-
>   3 files changed, 9 insertions(+), 9 deletions(-)
>
> Dexuan Cui (2):
>        busybox: enable more options for tar to support more file formats.
>        eglibc: install the missing /etc/ld.so.conf into target
>
> Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master
>
>
> commit 61e77cb2e9a240110c50925d1b3f908d14b02f24
> Author: Dexuan Cui<dexuan.cui@intel.com>
> Date:   Tue Nov 2 15:42:09 2010 +0800
>
>      eglibc: install the missing /etc/ld.so.conf into target

My preference is that /etc/ld.so.conf not always be installed onto the target. 
It's not required for anything to function, and as long as only default library 
directories are used, even ldconfig will run properly with it not existing (or 
empty).

If we do install it, we need to make sure it's empty.  I don't think it's 
appropriate to advocate alternative library directories on an embedded platform.

>      ${sysconfdir} should be added into FILES_${PN}.
>
>      Fixes [BUGID #511]
>
>      Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
>
> commit b952fae48c45e39a14ec9c24b0ee544a741c3f69
> Author: Dexuan Cui<dexuan.cui@intel.com>
> Date:   Tue Oct 26 15:10:26 2010 +0800
>
>      busybox: enable more options for tar to support more file formats.
>
>      Fixes [BUGID #495]
>
>      Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
> _______________________________________________
> poky mailing list
> poky@pokylinux.org
> https://lists.pokylinux.org/listinfo/poky

Re: [PULL] fixes to eglibc and busybox

[Saul Wold]

On 11/04/2010 08:13 AM, Mark Hatle wrote:
> On 11/3/10 7:55 PM, Cui, Dexuan wrote:
>> BTW: The fix to busybox is a RESEND since looks it was neglected. :-)
>>
>> $ scripts/create-pull-request -r 2ecf311fbca970f6ea2d80accc3f925b097b664e -b dcui/master
>> Note:<commit_id>   parameter assumed as 'HEAD'
>>
>>    meta/recipes-core/busybox/busybox-1.16.2/defconfig |   14 +++++++-------
>>    meta/recipes-core/eglibc/eglibc-package.inc        |    2 +-
>>    meta/recipes-core/eglibc/eglibc_2.12.bb            |    2 +-
>>    3 files changed, 9 insertions(+), 9 deletions(-)
>>
>> Dexuan Cui (2):
>>         busybox: enable more options for tar to support more file formats.
>>         eglibc: install the missing /etc/ld.so.conf into target
>>
>> Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master
>>
>>
>> commit 61e77cb2e9a240110c50925d1b3f908d14b02f24
>> Author: Dexuan Cui<dexuan.cui@intel.com>
>> Date:   Tue Nov 2 15:42:09 2010 +0800
>>
>>       eglibc: install the missing /etc/ld.so.conf into target
>
> My preference is that /etc/ld.so.conf not always be installed onto the target.
> It's not required for anything to function, and as long as only default library
> directories are used, even ldconfig will run properly with it not existing (or
> empty).
>
> If we do install it, we need to make sure it's empty.  I don't think it's
> appropriate to advocate alternative library directories on an embedded platform.
>
Mark,

The default one that we are installing contains entries for 
/usr/local/lib and /usr/X11R6/lib

I understand you would rather it be empty, is that correct?


Sau!


>>       ${sysconfdir} should be added into FILES_${PN}.
>>
>>       Fixes [BUGID #511]
>>
>>       Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
>>
>> commit b952fae48c45e39a14ec9c24b0ee544a741c3f69
>> Author: Dexuan Cui<dexuan.cui@intel.com>
>> Date:   Tue Oct 26 15:10:26 2010 +0800
>>
>>       busybox: enable more options for tar to support more file formats.
>>
>>       Fixes [BUGID #495]
>>
>>       Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
>> _______________________________________________
>> poky mailing list
>> poky@pokylinux.org
>> https://lists.pokylinux.org/listinfo/poky
>
> _______________________________________________
> poky mailing list
> poky@pokylinux.org
> https://lists.pokylinux.org/listinfo/poky
>

Re: [PULL] fixes to eglibc and busybox

[Mark Hatle]

On 11/4/10 1:10 PM, Saul Wold wrote:
> On 11/04/2010 08:13 AM, Mark Hatle wrote:
>> On 11/3/10 7:55 PM, Cui, Dexuan wrote:
>>> BTW: The fix to busybox is a RESEND since looks it was neglected. :-)
>>>
>>> $ scripts/create-pull-request -r 2ecf311fbca970f6ea2d80accc3f925b097b664e -b dcui/master
>>> Note:<commit_id>    parameter assumed as 'HEAD'
>>>
>>>     meta/recipes-core/busybox/busybox-1.16.2/defconfig |   14 +++++++-------
>>>     meta/recipes-core/eglibc/eglibc-package.inc        |    2 +-
>>>     meta/recipes-core/eglibc/eglibc_2.12.bb            |    2 +-
>>>     3 files changed, 9 insertions(+), 9 deletions(-)
>>>
>>> Dexuan Cui (2):
>>>          busybox: enable more options for tar to support more file formats.
>>>          eglibc: install the missing /etc/ld.so.conf into target
>>>
>>> Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master
>>>
>>>
>>> commit 61e77cb2e9a240110c50925d1b3f908d14b02f24
>>> Author: Dexuan Cui<dexuan.cui@intel.com>
>>> Date:   Tue Nov 2 15:42:09 2010 +0800
>>>
>>>        eglibc: install the missing /etc/ld.so.conf into target
>>
>> My preference is that /etc/ld.so.conf not always be installed onto the target.
>> It's not required for anything to function, and as long as only default library
>> directories are used, even ldconfig will run properly with it not existing (or
>> empty).
>>
>> If we do install it, we need to make sure it's empty.  I don't think it's
>> appropriate to advocate alternative library directories on an embedded platform.
>>
> Mark,
>
> The default one that we are installing contains entries for
> /usr/local/lib and /usr/X11R6/lib
>
> I understand you would rather it be empty, is that correct?

Generally speaking, we should not be placing software into /usr/local/lib as 
part of the Yocto work.  That location, and /opt/* are reserved for system 
administrators to place their own software...  So that location, if necessary, 
should be done by the system admin.

The /usr/X11R6/lib location should be a symlink to /usr/lib.  As all of the X 
libraries are installed into /usr/lib... so there is no reason to have this 
location in the ld.so.conf.  (Note I have not verified that that is a symlink... 
or if the directory is empty -- but it should be...)

So yes, my preference is that it's either absent or empty for embedded systems.

--Mark

>
> Sau!
>
>
>>>        ${sysconfdir} should be added into FILES_${PN}.
>>>
>>>        Fixes [BUGID #511]
>>>
>>>        Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
>>>
>>> commit b952fae48c45e39a14ec9c24b0ee544a741c3f69
>>> Author: Dexuan Cui<dexuan.cui@intel.com>
>>> Date:   Tue Oct 26 15:10:26 2010 +0800
>>>
>>>        busybox: enable more options for tar to support more file formats.
>>>
>>>        Fixes [BUGID #495]
>>>
>>>        Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
>>> _______________________________________________
>>> poky mailing list
>>> poky@pokylinux.org
>>> https://lists.pokylinux.org/listinfo/poky
>>
>> _______________________________________________
>> poky mailing list
>> poky@pokylinux.org
>> https://lists.pokylinux.org/listinfo/poky
>>
>
> _______________________________________________
> poky mailing list
> poky@pokylinux.org
> https://lists.pokylinux.org/listinfo/poky

Re: [PULL] fixes to eglibc and busybox

[Saul Wold]

On 11/04/2010 11:21 AM, Mark Hatle wrote:
> On 11/4/10 1:10 PM, Saul Wold wrote:
>> On 11/04/2010 08:13 AM, Mark Hatle wrote:
>>> On 11/3/10 7:55 PM, Cui, Dexuan wrote:
>>>> BTW: The fix to busybox is a RESEND since looks it was neglected. :-)
>>>>
>>>> $ scripts/create-pull-request -r 2ecf311fbca970f6ea2d80accc3f925b097b664e -b dcui/master
>>>> Note:<commit_id>     parameter assumed as 'HEAD'
>>>>
>>>>      meta/recipes-core/busybox/busybox-1.16.2/defconfig |   14 +++++++-------
>>>>      meta/recipes-core/eglibc/eglibc-package.inc        |    2 +-
>>>>      meta/recipes-core/eglibc/eglibc_2.12.bb            |    2 +-
>>>>      3 files changed, 9 insertions(+), 9 deletions(-)
>>>>
>>>> Dexuan Cui (2):
>>>>           busybox: enable more options for tar to support more file formats.
>>>>           eglibc: install the missing /etc/ld.so.conf into target
>>>>
>>>> Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master
>>>>
>>>>
>>>> commit 61e77cb2e9a240110c50925d1b3f908d14b02f24
>>>> Author: Dexuan Cui<dexuan.cui@intel.com>
>>>> Date:   Tue Nov 2 15:42:09 2010 +0800
>>>>
>>>>         eglibc: install the missing /etc/ld.so.conf into target
>>>
>>> My preference is that /etc/ld.so.conf not always be installed onto the target.
>>> It's not required for anything to function, and as long as only default library
>>> directories are used, even ldconfig will run properly with it not existing (or
>>> empty).
>>>
>>> If we do install it, we need to make sure it's empty.  I don't think it's
>>> appropriate to advocate alternative library directories on an embedded platform.
>>>
>> Mark,
>>
>> The default one that we are installing contains entries for
>> /usr/local/lib and /usr/X11R6/lib
>>
>> I understand you would rather it be empty, is that correct?
>
> Generally speaking, we should not be placing software into /usr/local/lib as
> part of the Yocto work.  That location, and /opt/* are reserved for system
> administrators to place their own software...  So that location, if necessary,
> should be done by the system admin.
>
Agreed

> The /usr/X11R6/lib location should be a symlink to /usr/lib.  As all of the X
> libraries are installed into /usr/lib... so there is no reason to have this
> location in the ld.so.conf.  (Note I have not verified that that is a symlink...
> or if the directory is empty -- but it should be...)
>
I don't think we even install an X11R6 directory anymore, so it can be 
removed!  CONFIRMED no /usr/X11R6 dir in a yocto image.

Sau!

> So yes, my preference is that it's either absent or empty for embedded systems.
>
> --Mark
>
>>
>> Sau!
>>
>>
>>>>         ${sysconfdir} should be added into FILES_${PN}.
>>>>
>>>>         Fixes [BUGID #511]
>>>>
>>>>         Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
>>>>
>>>> commit b952fae48c45e39a14ec9c24b0ee544a741c3f69
>>>> Author: Dexuan Cui<dexuan.cui@intel.com>
>>>> Date:   Tue Oct 26 15:10:26 2010 +0800
>>>>
>>>>         busybox: enable more options for tar to support more file formats.
>>>>
>>>>         Fixes [BUGID #495]
>>>>
>>>>         Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
>>>> _______________________________________________
>>>> poky mailing list
>>>> poky@pokylinux.org
>>>> https://lists.pokylinux.org/listinfo/poky
>>>
>>> _______________________________________________
>>> poky mailing list
>>> poky@pokylinux.org
>>> https://lists.pokylinux.org/listinfo/poky
>>>
>>
>> _______________________________________________
>> poky mailing list
>> poky@pokylinux.org
>> https://lists.pokylinux.org/listinfo/poky
>
> _______________________________________________
> poky mailing list
> poky@pokylinux.org
> https://lists.pokylinux.org/listinfo/poky
>

Re: [PULL] fixes to eglibc and busybox

[Cui, Dexuan]

>> On 11/04/2010 11:21 AM, Mark Hatle wrote:
>> 
>> So yes, my preference is that it's either absent or empty for
>> embedded systems. 

Hi Mark and Saul,
Thanks very much for the detailed explanation!!
So I'll make a new patch to install an empty ld.so.conf (to avoid the annoying spurious warning from ldconfig).

BTW: I'll send the new PULL mail to yocto@yoctoproject.org as this is not related to poky core.

Thanks,
-- Dexuan

[PULL] fixes to eglibc and busybox

[Cui, Dexuan]

This is a rebased and resent version of that I sent to poky@pokylinux.org (Thanks to Mark and Saul's suggestion).
Since this is a non poky core change, I post this version in this mailing list.
Please review it and comment.

Thanks,
-- Dexuan

$ scripts/create-pull-request -r 2ecf311fbca970f6ea2d80accc3f925b097b664e -b dcui/master
Note: <commit_id> parameter assumed as 'HEAD'

 meta/recipes-core/busybox/busybox-1.16.2/defconfig  |   14 +++++++-------
 meta/recipes-core/eglibc/eglibc-2.12/etc/ld.so.conf |    2 --
 meta/recipes-core/eglibc/eglibc-package.inc         |    2 +-
 meta/recipes-core/eglibc/eglibc_2.12.bb             |    2 +-
 4 files changed, 9 insertions(+), 11 deletions(-)

Dexuan Cui (2):
      busybox: enable more options for tar to support more file formats.
      eglibc: install an empty /etc/ld.so.conf into target

Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master

commit 032856f340075390052eeeee15d1e1495fe2d94b
Author: Dexuan Cui <dexuan.cui@intel.com>
Date:   Fri Nov 5 08:35:54 2010 +0800

    eglibc: install an empty /etc/ld.so.conf into target

    We add ${sysconfdir} into FILES_${PN} to avoid the below spurious warning

    ldconfig: Can't open configuration file /etc/ld.so.conf: No such file or
    directory

    The file ld.so.conf can be empty because now we don't install
    /usr/local/lib and /usr/X11R6/.

    Fixes [BUGID #511]

    Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>

commit b952fae48c45e39a14ec9c24b0ee544a741c3f69
Author: Dexuan Cui <dexuan.cui@intel.com>
Date:   Tue Oct 26 15:10:26 2010 +0800

    busybox: enable more options for tar to support more file formats.

    Fixes [BUGID #495]

    Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>