Patchwork [00/12] 20-May-2011 Consolidated Pull

login
register
mail settings
Submitter Saul Wold
Date May 20, 2011, 7:06 a.m.
Message ID <cover.1305874831.git.sgw@linux.intel.com>
Download mbox
Permalink /patch/4533/
State New, archived
Headers show

Pull-request

git://git.openembedded.org/openembedded-core-contrib.git sgw/stage

Comments

Saul Wold - May 20, 2011, 7:06 a.m.
Richard,

This is building green on x86 and ARM.

This contains updated pull requests from Khem and Dexuan.  I have some
non-gplv3 fixes along with a fix to the u-boot patch due to a bad checksum.

I have again included the gcc 4.6 enabler for x86 and arm, the kernel 
updates are in place. Feel free to remove it, but I think we get a good QA
pass and move forward, if not we fall back to gcc 4.5.x for RC2.

Thanks for your considersation

Sau!


The following changes since commit f2528db8f466a21ca207b310fffa7b05884b4579:

  distro/defaultsetup.conf: Append TCLIBC to TMPDIR by default (2011-05-19 23:49:37 +0100)

are available in the git repository at:
  git://git.openembedded.org/openembedded-core-contrib.git sgw/stage
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgw/stage

Dexuan Cui (1):
  package-index.bb: add support for deb and rpm.

Khem Raj (6):
  conf: Create distro overridable IMAGE varibales
  uclibc/site_config/funcs: getloadavg is unimplemented
  nfs-utils_1.2.3.bb: Fix compile failure on uclibc
  distcc_2.18.3.bb: Fix compilation on uclibc
  opkg_svn.bb: Install rcS.d into ${D} instead of ${IMAGE_ROOTFS}
  opkg-utils: Synchronise with OE

Nitin A Kamble (1):
  tcmode-default: switch to gcc 4.6.0 for x86, x86-64 & arm

Saul Wold (4):
  modutils-cross: fix install issue
  ed (GPLv2): remove --disable-silent-rules
  gettext (gplv2): add virtual/gettext to PROVIDES list
  u-boot: fix LIC_FILE_CHKSUM

 meta/classes/image.bbclass                         |    9 +++++-
 meta/classes/package_deb.bbclass                   |    4 +++
 meta/classes/package_ipk.bbclass                   |    4 +++
 meta/classes/package_rpm.bbclass                   |    3 ++
 meta/conf/distro/include/default-distrovars.inc    |   13 +++++++++
 meta/conf/distro/include/tcmode-default.inc        |    8 ++++-
 meta/recipes-bsp/uboot/u-boot_git.bb               |    2 +-
 .../nfs-utils/nfs-utils-1.0.6-uclibc.patch         |   16 ++++++-----
 .../nfs-utils/nfs-utils-1.2.3-uclibc-libio.h.patch |   26 +++++++++++++++++
 .../nfs-utils/nfs-utils_1.2.3.bb                   |    1 +
 meta/recipes-core/gettext/gettext_0.16.1.bb        |    4 +-
 meta/recipes-core/meta/package-index.bb            |    6 +---
 meta/recipes-core/tasks/task-core-boot.bb          |    4 ---
 meta/recipes-core/uclibc/site_config/funcs         |    1 -
 meta/recipes-devtools/distcc/distcc_2.18.3.bb      |   15 +++++++---
 .../opkg-utils/opkg-utils/mtime-int.patch          |   29 ++++++++++++++++++++
 meta/recipes-devtools/opkg-utils/opkg-utils_svn.bb |    7 +++-
 meta/recipes-devtools/opkg/opkg/headerfix.patch    |   19 -------------
 meta/recipes-devtools/opkg/opkg_svn.bb             |    3 +-
 meta/recipes-extended/ed/ed_0.5.bb                 |    4 ++-
 .../modutils/modutils-cross_2.4.27.bb              |    4 +-
 21 files changed, 129 insertions(+), 53 deletions(-)
 create mode 100644 meta/recipes-connectivity/nfs-utils/nfs-utils/nfs-utils-1.2.3-uclibc-libio.h.patch
 create mode 100644 meta/recipes-devtools/opkg-utils/opkg-utils/mtime-int.patch
 delete mode 100644 meta/recipes-devtools/opkg/opkg/headerfix.patch
Richard Purdie - May 20, 2011, 11:45 a.m.
On Fri, 2011-05-20 at 00:06 -0700, Saul Wold wrote:
> This is building green on x86 and ARM.
> 
> This contains updated pull requests from Khem and Dexuan.  I have some
> non-gplv3 fixes along with a fix to the u-boot patch due to a bad checksum.
> 
> I have again included the gcc 4.6 enabler for x86 and arm, the kernel 
> updates are in place. Feel free to remove it, but I think we get a good QA
> pass and move forward, if not we fall back to gcc 4.5.x for RC2.
> 
> Thanks for your considersation

I've merged some of this but I do have some concerns with a few of the
patches:

> Khem Raj (6):
>   conf: Create distro overridable IMAGE varibales

Phil had concerns about this and to be honest I agree with him. More
thought is needed with this, we need more of an overall plan.

>   distcc_2.18.3.bb: Fix compilation on uclibc

I think Phil's comment on this is valid and needs to be addressed.

>   opkg_svn.bb: Install rcS.d into ${D} instead of ${IMAGE_ROOTFS}

Should be using $D, *not* ${D}.

Cheers,

Richard
Darren Hart - May 20, 2011, 11:46 p.m.
On 05/20/2011 04:45 AM, Richard Purdie wrote:
> On Fri, 2011-05-20 at 00:06 -0700, Saul Wold wrote:
>> This is building green on x86 and ARM.
>>
>> This contains updated pull requests from Khem and Dexuan.  I have some
>> non-gplv3 fixes along with a fix to the u-boot patch due to a bad checksum.
>>
>> I have again included the gcc 4.6 enabler for x86 and arm, the kernel 
>> updates are in place. Feel free to remove it, but I think we get a good QA
>> pass and move forward, if not we fall back to gcc 4.5.x for RC2.
>>
>> Thanks for your considersation
> 
> I've merged some of this but I do have some concerns with a few of the
> patches:
> 
>> Khem Raj (6):
>>   conf: Create distro overridable IMAGE varibales
> 
> Phil had concerns about this and to be honest I agree with him. More
> thought is needed with this, we need more of an overall plan.
> 
>>   distcc_2.18.3.bb: Fix compilation on uclibc
> 
> I think Phil's comment on this is valid and needs to be addressed.
> 
>>   opkg_svn.bb: Install rcS.d into ${D} instead of ${IMAGE_ROOTFS}
> 
> Should be using $D, *not* ${D}.

Really? It was my understanding these are equivalent, with the {} form
being the less ambiguous of the two for use when concatenating strings
of variables. What am I missing?

http://tldp.org/LDP/abs/html/parameter-substitution.html

Thanks,
Saul Wold - May 20, 2011, 11:50 p.m.
On 05/20/2011 04:46 PM, Darren Hart wrote:
>
>
> On 05/20/2011 04:45 AM, Richard Purdie wrote:
>> On Fri, 2011-05-20 at 00:06 -0700, Saul Wold wrote:
>>> This is building green on x86 and ARM.
>>>
>>> This contains updated pull requests from Khem and Dexuan.  I have some
>>> non-gplv3 fixes along with a fix to the u-boot patch due to a bad checksum.
>>>
>>> I have again included the gcc 4.6 enabler for x86 and arm, the kernel
>>> updates are in place. Feel free to remove it, but I think we get a good QA
>>> pass and move forward, if not we fall back to gcc 4.5.x for RC2.
>>>
>>> Thanks for your considersation
>>
>> I've merged some of this but I do have some concerns with a few of the
>> patches:
>>
>>> Khem Raj (6):
>>>    conf: Create distro overridable IMAGE varibales
>>
>> Phil had concerns about this and to be honest I agree with him. More
>> thought is needed with this, we need more of an overall plan.
>>
>>>    distcc_2.18.3.bb: Fix compilation on uclibc
>>
>> I think Phil's comment on this is valid and needs to be addressed.
>>
>>>    opkg_svn.bb: Install rcS.d into ${D} instead of ${IMAGE_ROOTFS}
>>
>> Should be using $D, *not* ${D}.
>
> Really? It was my understanding these are equivalent, with the {} form
> being the less ambiguous of the two for use when concatenating strings
> of variables. What am I missing?
>
Bitbake parsing occurs first and translates the ${D}, but not the $D, 
see RP's comment in the main thread for this patch.

Sau!

> http://tldp.org/LDP/abs/html/parameter-substitution.html
>
> Thanks,
>