u-boot: update to 2013.07

Submitted by Laszlo Papp on Aug. 23, 2013, 9:48 a.m.

Details

Message ID 1377251290-4290-1-git-send-email-lpapp@kde.org
State Accepted
Commit b2d6f08ff4f03079973eab76a790bf4555e32236
Headers show

Commit Message

Laszlo Papp Aug. 23, 2013, 9:48 a.m.
Signed-off-by: Laszlo Papp <lpapp@kde.org>
---
 meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb | 27 +++++++++++++++++++++++
 meta/recipes-bsp/u-boot/u-boot_2013.07.bb         | 23 +++++++++++++++++++
 2 files changed, 50 insertions(+)
 create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
 create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb

Patch hide | download patch | download mbox

diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
new file mode 100644
index 0000000..0108e00
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
@@ -0,0 +1,27 @@ 
+DESCRIPTION = "U-boot bootloader mkimage tool"
+LICENSE = "GPLv2+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
+                    file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
+SECTION = "bootloader"
+
+SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2"
+
+SRC_URI[md5sum] = "8445162690052e6afd4b8f87af2bb557"
+SRC_URI[sha256sum] = "2bbd440c1b29928d230604eac372fda5152d3ae23f52b1910c77fc5bf531250b"
+
+S = "${WORKDIR}/u-boot-${PV}"
+
+BBCLASSEXTEND = "native nativesdk"
+
+EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}" HOSTLDFLAGS="${LDFLAGS}" HOSTSTRIP=true'
+
+do_compile () {
+  oe_runmake tools
+}
+
+do_install () {
+  install -d ${D}${bindir}
+  install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
+  ln -sf uboot-mkimage ${D}${bindir}/mkimage
+}
+
diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
new file mode 100644
index 0000000..f3f4b62
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
@@ -0,0 +1,23 @@ 
+require u-boot.inc
+
+# To build u-boot for your machine, provide the following lines in your machine
+# config, replacing the assignments as appropriate for your machine.
+# UBOOT_MACHINE = "omap3_beagle_config"
+# UBOOT_ENTRYPOINT = "0x80008000"
+# UBOOT_LOADADDRESS = "0x80008000"
+
+LICENSE = "GPLv2+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
+                    file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
+
+# This revision corresponds to the tag "v2013.07"
+# We use the revision in order to avoid having to fetch it from the repo during parse
+SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
+
+PV = "v2013.07+git${SRCPV}"
+
+SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
+
+S = "${WORKDIR}/git"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"

Comments

Laszlo Papp Aug. 23, 2013, 4:57 p.m.
Any update? It would be nice not to miss the freeze ... ;-)


On Fri, Aug 23, 2013 at 10:48 AM, Laszlo Papp <lpapp@kde.org> wrote:

> Signed-off-by: Laszlo Papp <lpapp@kde.org>
> ---
>  meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb | 27
> +++++++++++++++++++++++
>  meta/recipes-bsp/u-boot/u-boot_2013.07.bb         | 23
> +++++++++++++++++++
>  2 files changed, 50 insertions(+)
>  create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>  create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>
> diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bbb/meta/recipes-bsp/u-boot/
> u-boot-mkimage_2013.07.bb
> new file mode 100644
> index 0000000..0108e00
> --- /dev/null
> +++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
> @@ -0,0 +1,27 @@
> +DESCRIPTION = "U-boot bootloader mkimage tool"
> +LICENSE = "GPLv2+"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
> +
>  file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
> +SECTION = "bootloader"
> +
> +SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2"
> +
> +SRC_URI[md5sum] = "8445162690052e6afd4b8f87af2bb557"
> +SRC_URI[sha256sum] =
> "2bbd440c1b29928d230604eac372fda5152d3ae23f52b1910c77fc5bf531250b"
> +
> +S = "${WORKDIR}/u-boot-${PV}"
> +
> +BBCLASSEXTEND = "native nativesdk"
> +
> +EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}" HOSTLDFLAGS="${LDFLAGS}"
> HOSTSTRIP=true'
> +
> +do_compile () {
> +  oe_runmake tools
> +}
> +
> +do_install () {
> +  install -d ${D}${bindir}
> +  install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
> +  ln -sf uboot-mkimage ${D}${bindir}/mkimage
> +}
> +
> diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bbb/meta/recipes-bsp/u-boot/
> u-boot_2013.07.bb
> new file mode 100644
> index 0000000..f3f4b62
> --- /dev/null
> +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> @@ -0,0 +1,23 @@
> +require u-boot.inc
> +
> +# To build u-boot for your machine, provide the following lines in your
> machine
> +# config, replacing the assignments as appropriate for your machine.
> +# UBOOT_MACHINE = "omap3_beagle_config"
> +# UBOOT_ENTRYPOINT = "0x80008000"
> +# UBOOT_LOADADDRESS = "0x80008000"
> +
> +LICENSE = "GPLv2+"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
> +
>  file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
> +
> +# This revision corresponds to the tag "v2013.07"
> +# We use the revision in order to avoid having to fetch it from the repo
> during parse
> +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
> +
> +PV = "v2013.07+git${SRCPV}"
> +
> +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
> +
> +S = "${WORKDIR}/git"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> --
> 1.8.3.4
>
>
Saul Wold Aug. 23, 2013, 9:56 p.m.
On 08/23/2013 09:57 AM, Laszlo Papp wrote:
> Any update? It would be nice not to miss the freeze ... ;-)
>
I had some networking issues this morning, and I am pulling together 
patches now.  This was build on the Autobuilders last night and there a 
a problem with meta-fsl-ppc-lsb build, I am not sure if it's a transient 
issue or a real one.

I have not look into it yet.  The high level logs can be found here:
http://autobuilder.yoctoproject.org:8011/builders/nightly-fsl-ppc-lsb/builds/231/steps/Building%20Images/logs/stdio

As you might have seen I rejected Otavio's patch and am working to 
include this one.  It will still need the rest of the maintainers to review.

Thanks for your time and effort.

Sau!

>
> On Fri, Aug 23, 2013 at 10:48 AM, Laszlo Papp <lpapp@kde.org
> <mailto:lpapp@kde.org>> wrote:
>
>     Signed-off-by: Laszlo Papp <lpapp@kde.org <mailto:lpapp@kde.org>>
>     ---
>       meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb> | 27 +++++++++++++++++++++++
>       meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>     <http://u-boot_2013.07.bb>         | 23 +++++++++++++++++++
>       2 files changed, 50 insertions(+)
>       create mode 100644
>     meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb>
>       create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>     <http://u-boot_2013.07.bb>
>
>     diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb>
>     b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb>
>     new file mode 100644
>     index 0000000..0108e00
>     --- /dev/null
>     +++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb>
>     @@ -0,0 +1,27 @@
>     +DESCRIPTION = "U-boot bootloader mkimage tool"
>     +LICENSE = "GPLv2+"
>     +LIC_FILES_CHKSUM =
>     "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>     +
>       file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
>     +SECTION = "bootloader"
>     +
>     +SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2"
>     +
>     +SRC_URI[md5sum] = "8445162690052e6afd4b8f87af2bb557"
>     +SRC_URI[sha256sum] =
>     "2bbd440c1b29928d230604eac372fda5152d3ae23f52b1910c77fc5bf531250b"
>     +
>     +S = "${WORKDIR}/u-boot-${PV}"
>     +
>     +BBCLASSEXTEND = "native nativesdk"
>     +
>     +EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}"
>     HOSTLDFLAGS="${LDFLAGS}" HOSTSTRIP=true'
>     +
>     +do_compile () {
>     +  oe_runmake tools
>     +}
>     +
>     +do_install () {
>     +  install -d ${D}${bindir}
>     +  install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
>     +  ln -sf uboot-mkimage ${D}${bindir}/mkimage
>     +}
>     +
>     diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>     <http://u-boot_2013.07.bb>
>     b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb <http://u-boot_2013.07.bb>
>     new file mode 100644
>     index 0000000..f3f4b62
>     --- /dev/null
>     +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>     <http://u-boot_2013.07.bb>
>     @@ -0,0 +1,23 @@
>     +require u-boot.inc
>     +
>     +# To build u-boot for your machine, provide the following lines in
>     your machine
>     +# config, replacing the assignments as appropriate for your machine.
>     +# UBOOT_MACHINE = "omap3_beagle_config"
>     +# UBOOT_ENTRYPOINT = "0x80008000"
>     +# UBOOT_LOADADDRESS = "0x80008000"
>     +
>     +LICENSE = "GPLv2+"
>     +LIC_FILES_CHKSUM =
>     "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>     +
>       file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
>     +
>     +# This revision corresponds to the tag "v2013.07"
>     +# We use the revision in order to avoid having to fetch it from the
>     repo during parse
>     +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
>     +
>     +PV = "v2013.07+git${SRCPV}"
>     +
>     +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git
>     <http://git.denx.de/u-boot.git;branch=master;protocol=git>"
>     +
>     +S = "${WORKDIR}/git"
>     +
>     +PACKAGE_ARCH = "${MACHINE_ARCH}"
>     --
>     1.8.3.4
>
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
Saul Wold Aug. 24, 2013, 4:31 a.m.
On 08/23/2013 02:56 PM, Saul Wold wrote:
> On 08/23/2013 09:57 AM, Laszlo Papp wrote:
>> Any update? It would be nice not to miss the freeze ... ;-)
>>
> I had some networking issues this morning, and I am pulling together
> patches now.  This was build on the Autobuilders last night and there a
> a problem with meta-fsl-ppc-lsb build, I am not sure if it's a transient
> issue or a real one.
>
> I have not look into it yet.  The high level logs can be found here:
> http://autobuilder.yoctoproject.org:8011/builders/nightly-fsl-ppc-lsb/builds/231/steps/Building%20Images/logs/stdio
>
As I said above I had not looked into this yet, after looking at it a 
little further this is an issue with the u-boot_git recipe that 
FreeScale PPC is using, so not an issue for you to worry about.

Sorry for the alarm.

Sau!

>
> As you might have seen I rejected Otavio's patch and am working to
> include this one.  It will still need the rest of the maintainers to
> review.
>
> Thanks for your time and effort.
>
> Sau!
>
>>
>> On Fri, Aug 23, 2013 at 10:48 AM, Laszlo Papp <lpapp@kde.org
>> <mailto:lpapp@kde.org>> wrote:
>>
>>     Signed-off-by: Laszlo Papp <lpapp@kde.org <mailto:lpapp@kde.org>>
>>     ---
>>       meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>>     <http://u-boot-mkimage_2013.07.bb> | 27 +++++++++++++++++++++++
>>       meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>     <http://u-boot_2013.07.bb>         | 23 +++++++++++++++++++
>>       2 files changed, 50 insertions(+)
>>       create mode 100644
>>     meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>>     <http://u-boot-mkimage_2013.07.bb>
>>       create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>     <http://u-boot_2013.07.bb>
>>
>>     diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>>     <http://u-boot-mkimage_2013.07.bb>
>>     b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>>     <http://u-boot-mkimage_2013.07.bb>
>>     new file mode 100644
>>     index 0000000..0108e00
>>     --- /dev/null
>>     +++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>>     <http://u-boot-mkimage_2013.07.bb>
>>     @@ -0,0 +1,27 @@
>>     +DESCRIPTION = "U-boot bootloader mkimage tool"
>>     +LICENSE = "GPLv2+"
>>     +LIC_FILES_CHKSUM =
>>     "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>>     +
>>
>> file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
>>
>>     +SECTION = "bootloader"
>>     +
>>     +SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2"
>>     +
>>     +SRC_URI[md5sum] = "8445162690052e6afd4b8f87af2bb557"
>>     +SRC_URI[sha256sum] =
>>     "2bbd440c1b29928d230604eac372fda5152d3ae23f52b1910c77fc5bf531250b"
>>     +
>>     +S = "${WORKDIR}/u-boot-${PV}"
>>     +
>>     +BBCLASSEXTEND = "native nativesdk"
>>     +
>>     +EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}"
>>     HOSTLDFLAGS="${LDFLAGS}" HOSTSTRIP=true'
>>     +
>>     +do_compile () {
>>     +  oe_runmake tools
>>     +}
>>     +
>>     +do_install () {
>>     +  install -d ${D}${bindir}
>>     +  install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
>>     +  ln -sf uboot-mkimage ${D}${bindir}/mkimage
>>     +}
>>     +
>>     diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>     <http://u-boot_2013.07.bb>
>>     b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>> <http://u-boot_2013.07.bb>
>>     new file mode 100644
>>     index 0000000..f3f4b62
>>     --- /dev/null
>>     +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>     <http://u-boot_2013.07.bb>
>>     @@ -0,0 +1,23 @@
>>     +require u-boot.inc
>>     +
>>     +# To build u-boot for your machine, provide the following lines in
>>     your machine
>>     +# config, replacing the assignments as appropriate for your machine.
>>     +# UBOOT_MACHINE = "omap3_beagle_config"
>>     +# UBOOT_ENTRYPOINT = "0x80008000"
>>     +# UBOOT_LOADADDRESS = "0x80008000"
>>     +
>>     +LICENSE = "GPLv2+"
>>     +LIC_FILES_CHKSUM =
>>     "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>>     +
>>
>> file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
>>
>>     +
>>     +# This revision corresponds to the tag "v2013.07"
>>     +# We use the revision in order to avoid having to fetch it from the
>>     repo during parse
>>     +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
>>     +
>>     +PV = "v2013.07+git${SRCPV}"
>>     +
>>     +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git
>>     <http://git.denx.de/u-boot.git;branch=master;protocol=git>"
>>     +
>>     +S = "${WORKDIR}/git"
>>     +
>>     +PACKAGE_ARCH = "${MACHINE_ARCH}"
>>     --
>>     1.8.3.4
>>
>>
>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
Saul Wold Aug. 26, 2013, 5:55 p.m.
On 08/23/2013 09:57 AM, Laszlo Papp wrote:
> Any update? It would be nice not to miss the freeze ... ;-)
>
This has been merged, and I was informed by the QA team that it works 
correctly for the mpc8315e-rdb and beagleboard, but NOT the beagleboard XM.

So maintaining this as DEFAULT_PREFERENCE = "-1" is correct for now, 
until the XM issue can be resolved.

Thanks for the contribution.

Sau!

>
> On Fri, Aug 23, 2013 at 10:48 AM, Laszlo Papp <lpapp@kde.org
> <mailto:lpapp@kde.org>> wrote:
>
>     Signed-off-by: Laszlo Papp <lpapp@kde.org <mailto:lpapp@kde.org>>
>     ---
>       meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb> | 27 +++++++++++++++++++++++
>       meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>     <http://u-boot_2013.07.bb>         | 23 +++++++++++++++++++
>       2 files changed, 50 insertions(+)
>       create mode 100644
>     meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb>
>       create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>     <http://u-boot_2013.07.bb>
>
>     diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb>
>     b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb>
>     new file mode 100644
>     index 0000000..0108e00
>     --- /dev/null
>     +++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>     <http://u-boot-mkimage_2013.07.bb>
>     @@ -0,0 +1,27 @@
>     +DESCRIPTION = "U-boot bootloader mkimage tool"
>     +LICENSE = "GPLv2+"
>     +LIC_FILES_CHKSUM =
>     "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>     +
>       file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
>     +SECTION = "bootloader"
>     +
>     +SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2"
>     +
>     +SRC_URI[md5sum] = "8445162690052e6afd4b8f87af2bb557"
>     +SRC_URI[sha256sum] =
>     "2bbd440c1b29928d230604eac372fda5152d3ae23f52b1910c77fc5bf531250b"
>     +
>     +S = "${WORKDIR}/u-boot-${PV}"
>     +
>     +BBCLASSEXTEND = "native nativesdk"
>     +
>     +EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTLD="${LD}"
>     HOSTLDFLAGS="${LDFLAGS}" HOSTSTRIP=true'
>     +
>     +do_compile () {
>     +  oe_runmake tools
>     +}
>     +
>     +do_install () {
>     +  install -d ${D}${bindir}
>     +  install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage
>     +  ln -sf uboot-mkimage ${D}${bindir}/mkimage
>     +}
>     +
>     diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>     <http://u-boot_2013.07.bb>
>     b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb <http://u-boot_2013.07.bb>
>     new file mode 100644
>     index 0000000..f3f4b62
>     --- /dev/null
>     +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>     <http://u-boot_2013.07.bb>
>     @@ -0,0 +1,23 @@
>     +require u-boot.inc
>     +
>     +# To build u-boot for your machine, provide the following lines in
>     your machine
>     +# config, replacing the assignments as appropriate for your machine.
>     +# UBOOT_MACHINE = "omap3_beagle_config"
>     +# UBOOT_ENTRYPOINT = "0x80008000"
>     +# UBOOT_LOADADDRESS = "0x80008000"
>     +
>     +LICENSE = "GPLv2+"
>     +LIC_FILES_CHKSUM =
>     "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>     +
>       file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
>     +
>     +# This revision corresponds to the tag "v2013.07"
>     +# We use the revision in order to avoid having to fetch it from the
>     repo during parse
>     +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
>     +
>     +PV = "v2013.07+git${SRCPV}"
>     +
>     +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git
>     <http://git.denx.de/u-boot.git;branch=master;protocol=git>"
>     +
>     +S = "${WORKDIR}/git"
>     +
>     +PACKAGE_ARCH = "${MACHINE_ARCH}"
>     --
>     1.8.3.4
>
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
Peter Bigot Aug. 26, 2013, 9:48 p.m.
I can confirm this also works with the Gumstix Overo in a layer bbappend 
that used SRC_URI_append_overo for additional patches.  Thanks to you 
both for getting it merged.

I did come up with a question, though.  The recipe uses:

   PV = "v2013.07+git${SRCPV}"

which creates an informative but ugly ${BPN}.  I'd like to use something 
like:

   FILESEXTRAPATHS_prepend := "${THISDIR}/${BPN}:"

in the bbappend to simplify updates, but I don't want to have a 
directory named u-boot-v2013.07+gitAUTOINC+62c175fbb8.  For linux-yocto, 
this is solved through introduction of ${LINUX_VERSION} allowing 
${PN}-${LINUX_VERSION}.

Is there reason to create a parallel UBOOT_VERSION?

Is there reason to have some way to access the default ${PV} inferred 
from the recipe name before the base recipe overrode it? (Is there 
already some way to get that value?)

Is there a better approach than bbappend-in-layer that I should use for 
adding patches required for a specific u-boot machine?

Peter

On 08/26/2013 12:55 PM, Saul Wold wrote:
> On 08/23/2013 09:57 AM, Laszlo Papp wrote:
>> Any update? It would be nice not to miss the freeze ... ;-)
>>
> This has been merged, and I was informed by the QA team that it works 
> correctly for the mpc8315e-rdb and beagleboard, but NOT the 
> beagleboard XM.
Laszlo Papp Aug. 26, 2013, 9:55 p.m.
This email contains some nuances, so I will try to reply to the main points
(IMO).

On Mon, Aug 26, 2013 at 10:48 PM, Peter A. Bigot <pab@pabigot.com> wrote:

> Is there reason to create a parallel UBOOT_VERSION?
>

Yes. fw-utils is not available with this u-boot version just yet. Also, I
have not tested all the reference BSPs.


> Is there a better approach than bbappend-in-layer that I should use for
> adding patches required for a specific u-boot machine?
>

That is hard to answer without details. fw-utils could be some help layer,
maybe. Other than that, what do you mean? Something like config fragments
(see busybox)?

Cheers,
Laszlo
Bruce Ashfield Aug. 26, 2013, 10:03 p.m.
On Mon, Aug 26, 2013 at 5:48 PM, Peter A. Bigot <pab@pabigot.com> wrote:
> I can confirm this also works with the Gumstix Overo in a layer bbappend
> that used SRC_URI_append_overo for additional patches.  Thanks to you both
> for getting it merged.
>
> I did come up with a question, though.  The recipe uses:
>
>   PV = "v2013.07+git${SRCPV}"
>
> which creates an informative but ugly ${BPN}.  I'd like to use something
> like:
>
>   FILESEXTRAPATHS_prepend := "${THISDIR}/${BPN}:"
>
> in the bbappend to simplify updates, but I don't want to have a directory
> named u-boot-v2013.07+gitAUTOINC+62c175fbb8.  For linux-yocto, this is
> solved through introduction of ${LINUX_VERSION} allowing
> ${PN}-${LINUX_VERSION}.
>
> Is there reason to create a parallel UBOOT_VERSION?
>
> Is there reason to have some way to access the default ${PV} inferred from
> the recipe name before the base recipe overrode it? (Is there already some
> way to get that value?)
>
> Is there a better approach than bbappend-in-layer that I should use for
> adding patches required for a specific u-boot machine?

That's the right approach. Only when complexity warrants is it worth
the effort to maintain
a targeted git tree that manages changes via branches. So until the
directories full of patches
and conflicting features collapse on themselves, go with SRC_URIs and bbappends.

Cheers,

Bruce

>
> Peter
>
>
> On 08/26/2013 12:55 PM, Saul Wold wrote:
>>
>> On 08/23/2013 09:57 AM, Laszlo Papp wrote:
>>>
>>> Any update? It would be nice not to miss the freeze ... ;-)
>>>
>> This has been merged, and I was informed by the QA team that it works
>> correctly for the mpc8315e-rdb and beagleboard, but NOT the beagleboard XM.
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
Laszlo Papp Aug. 27, 2013, 10:45 a.m.
On Mon, Aug 26, 2013 at 6:55 PM, Saul Wold <sgw@linux.intel.com> wrote:

> Thanks for the contribution.
>

Looks my first contribution which is in then! Yay, thanks. :)

-- Laszlo
Randy MacLeod Aug. 27, 2013, 7:57 p.m.
On 13-08-23 05:48 AM, Laszlo Papp wrote:
> Signed-off-by: Laszlo Papp <lpapp@kde.org>
> ---
>   meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb | 27 +++++++++++++++++++++++
>   meta/recipes-bsp/u-boot/u-boot_2013.07.bb         | 23 +++++++++++++++++++
>   2 files changed, 50 insertions(+)
>   create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>   create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>
...
> diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> new file mode 100644
> index 0000000..f3f4b62
> --- /dev/null
> +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> @@ -0,0 +1,23 @@
> +require u-boot.inc
> +
> +# To build u-boot for your machine, provide the following lines in your machine
> +# config, replacing the assignments as appropriate for your machine.
> +# UBOOT_MACHINE = "omap3_beagle_config"
> +# UBOOT_ENTRYPOINT = "0x80008000"
> +# UBOOT_LOADADDRESS = "0x80008000"
> +
> +LICENSE = "GPLv2+"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
> +                    file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
> +
> +# This revision corresponds to the tag "v2013.07"
> +# We use the revision in order to avoid having to fetch it from the repo during parse
> +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
> +
> +PV = "v2013.07+git${SRCPV}"
> +
> +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"

Just curious...
I dont' see a good reason to specify the git protocol twice:
    git://.... and ; protocol=git

This occurs in many recipes and we could get rid of the clutter
with a script if it bothers anyone else. :)

Am I missing something or are people just copying and pasting the text?

// Randy

> +
> +S = "${WORKDIR}/git"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>
Laszlo Papp Aug. 27, 2013, 8:03 p.m.
On Tue, Aug 27, 2013 at 8:57 PM, Randy MacLeod
<randy.macleod@windriver.com>wrote:

> Am I missing something or are people just copying and pasting the text?
>

I was just copying and pasting it.
Martin Jansa Aug. 27, 2013, 8:08 p.m.
On Tue, Aug 27, 2013 at 03:57:09PM -0400, Randy MacLeod wrote:
> On 13-08-23 05:48 AM, Laszlo Papp wrote:
> > Signed-off-by: Laszlo Papp <lpapp@kde.org>
> > ---
> >   meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb | 27 +++++++++++++++++++++++
> >   meta/recipes-bsp/u-boot/u-boot_2013.07.bb         | 23 +++++++++++++++++++
> >   2 files changed, 50 insertions(+)
> >   create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
> >   create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> >
> ...
> > diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> > new file mode 100644
> > index 0000000..f3f4b62
> > --- /dev/null
> > +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> > @@ -0,0 +1,23 @@
> > +require u-boot.inc
> > +
> > +# To build u-boot for your machine, provide the following lines in your machine
> > +# config, replacing the assignments as appropriate for your machine.
> > +# UBOOT_MACHINE = "omap3_beagle_config"
> > +# UBOOT_ENTRYPOINT = "0x80008000"
> > +# UBOOT_LOADADDRESS = "0x80008000"
> > +
> > +LICENSE = "GPLv2+"
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
> > +                    file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
> > +
> > +# This revision corresponds to the tag "v2013.07"
> > +# We use the revision in order to avoid having to fetch it from the repo during parse
> > +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
> > +
> > +PV = "v2013.07+git${SRCPV}"
> > +
> > +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
> 
> Just curious...
> I dont' see a good reason to specify the git protocol twice:
>     git://.... and ; protocol=git

It's not twice, git:// is to select bitbake fetcher (git.py) and one
fetcher can support multiple protocols (e.g. over git, http or ssh)

but you're right that in most cases "protocol=git" isn't needed, because
it's default protocol for git fetcher when host is specified in that
SRC_URI.

bitbake/lib/bb/fetch2/git.py:
        if 'protocol' in ud.parm:
            ud.proto = ud.parm['protocol']
        elif not ud.host:
            ud.proto = 'file'
        else:
            ud.proto = "git"

And I would support removing it in one go as it clearly confuses people
:).

> 
> This occurs in many recipes and we could get rid of the clutter
> with a script if it bothers anyone else. :)
> 
> Am I missing something or are people just copying and pasting the text?
> 
> // Randy
> 
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +PACKAGE_ARCH = "${MACHINE_ARCH}"
> >
> 
> 
> -- 
> # Randy MacLeod. SMTS, Linux, Wind River
> Direct: 613.963.1350
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
Richard Purdie Aug. 27, 2013, 11:29 p.m.
On Tue, 2013-08-27 at 22:08 +0200, Martin Jansa wrote:
> On Tue, Aug 27, 2013 at 03:57:09PM -0400, Randy MacLeod wrote:
> > On 13-08-23 05:48 AM, Laszlo Papp wrote:
> > > Signed-off-by: Laszlo Papp <lpapp@kde.org>
> > > ---
> > >   meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb | 27 +++++++++++++++++++++++
> > >   meta/recipes-bsp/u-boot/u-boot_2013.07.bb         | 23 +++++++++++++++++++
> > >   2 files changed, 50 insertions(+)
> > >   create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
> > >   create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> > >
> > ...
> > > diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> > > new file mode 100644
> > > index 0000000..f3f4b62
> > > --- /dev/null
> > > +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
> > > @@ -0,0 +1,23 @@
> > > +require u-boot.inc
> > > +
> > > +# To build u-boot for your machine, provide the following lines in your machine
> > > +# config, replacing the assignments as appropriate for your machine.
> > > +# UBOOT_MACHINE = "omap3_beagle_config"
> > > +# UBOOT_ENTRYPOINT = "0x80008000"
> > > +# UBOOT_LOADADDRESS = "0x80008000"
> > > +
> > > +LICENSE = "GPLv2+"
> > > +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
> > > +                    file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
> > > +
> > > +# This revision corresponds to the tag "v2013.07"
> > > +# We use the revision in order to avoid having to fetch it from the repo during parse
> > > +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
> > > +
> > > +PV = "v2013.07+git${SRCPV}"
> > > +
> > > +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
> > 
> > Just curious...
> > I dont' see a good reason to specify the git protocol twice:
> >     git://.... and ; protocol=git
> 
> It's not twice, git:// is to select bitbake fetcher (git.py) and one
> fetcher can support multiple protocols (e.g. over git, http or ssh)
> 
> but you're right that in most cases "protocol=git" isn't needed, because
> it's default protocol for git fetcher when host is specified in that
> SRC_URI.
> 
> bitbake/lib/bb/fetch2/git.py:
>         if 'protocol' in ud.parm:
>             ud.proto = ud.parm['protocol']
>         elif not ud.host:
>             ud.proto = 'file'
>         else:
>             ud.proto = "git"
> 
> And I would support removing it in one go as it clearly confuses people
> :).

The reason is that the default for the git fetcher was once rsync. Why?
The fetcher was written when git was still a new and unexplored
landscape. We decided in the end to break that insane default and fix
it.

We should kill off the now unnecessary parameters though.

Cheers,

Richard
Randy MacLeod Aug. 28, 2013, 3:50 a.m.
On 13-08-27 07:29 PM, Richard Purdie wrote:
> On Tue, 2013-08-27 at 22:08 +0200, Martin Jansa wrote:
>> On Tue, Aug 27, 2013 at 03:57:09PM -0400, Randy MacLeod wrote:
>>> On 13-08-23 05:48 AM, Laszlo Papp wrote:
>>>> Signed-off-by: Laszlo Papp <lpapp@kde.org>
>>>> ---
>>>>    meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb | 27 +++++++++++++++++++++++
>>>>    meta/recipes-bsp/u-boot/u-boot_2013.07.bb         | 23 +++++++++++++++++++
>>>>    2 files changed, 50 insertions(+)
>>>>    create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>>>>    create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>>>
>>> ...
>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>>> new file mode 100644
>>>> index 0000000..f3f4b62
>>>> --- /dev/null
>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>>> @@ -0,0 +1,23 @@
>>>> +require u-boot.inc
>>>> +
>>>> +# To build u-boot for your machine, provide the following lines in your machine
>>>> +# config, replacing the assignments as appropriate for your machine.
>>>> +# UBOOT_MACHINE = "omap3_beagle_config"
>>>> +# UBOOT_ENTRYPOINT = "0x80008000"
>>>> +# UBOOT_LOADADDRESS = "0x80008000"
>>>> +
>>>> +LICENSE = "GPLv2+"
>>>> +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>>>> +                    file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
>>>> +
>>>> +# This revision corresponds to the tag "v2013.07"
>>>> +# We use the revision in order to avoid having to fetch it from the repo during parse
>>>> +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
>>>> +
>>>> +PV = "v2013.07+git${SRCPV}"
>>>> +
>>>> +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
>>>
>>> Just curious...
>>> I dont' see a good reason to specify the git protocol twice:
>>>      git://.... and ; protocol=git
>>
>> It's not twice, git:// is to select bitbake fetcher (git.py) and one
>> fetcher can support multiple protocols (e.g. over git, http or ssh)
>>
>> but you're right that in most cases "protocol=git" isn't needed, because
>> it's default protocol for git fetcher when host is specified in that
>> SRC_URI.
>>
>> bitbake/lib/bb/fetch2/git.py:
>>          if 'protocol' in ud.parm:
>>              ud.proto = ud.parm['protocol']
>>          elif not ud.host:
>>              ud.proto = 'file'
>>          else:
>>              ud.proto = "git"
>>
>> And I would support removing it in one go as it clearly confuses people
>> :).
>
> The reason is that the default for the git fetcher was once rsync. Why?
> The fetcher was written when git was still a new and unexplored
> landscape. We decided in the end to break that insane default and fix
> it.
>
> We should kill off the now unnecessary parameters though.

Ok. Jackie will have time to make a first pass tomorrow.
Jackie, please include the replacement script in your email.

// Randy

>
> Cheers,
>
> Richard
>
>
>
Khem Raj Aug. 28, 2013, 4 p.m.
On Aug 27, 2013, at 12:57 PM, Randy MacLeod <randy.macleod@windriver.com> wrote:

> Just curious...
> I dont' see a good reason to specify the git protocol twice:
>   git://.... and ; protocol=git

since its explicit. it does provide readability benefits which does have value
when newbies read the recipes.

> 
> This occurs in many recipes and we could get rid of the clutter
> with a script if it bothers anyone else. :)
> 
> Am I missing something or are people just copying and pasting the text?
jackie huang Aug. 29, 2013, 7:16 a.m.
On 8/28/2013 11:50 AM, Randy MacLeod wrote:
> On 13-08-27 07:29 PM, Richard Purdie wrote:
>> On Tue, 2013-08-27 at 22:08 +0200, Martin Jansa wrote:
>>> On Tue, Aug 27, 2013 at 03:57:09PM -0400, Randy MacLeod wrote:
>>>> On 13-08-23 05:48 AM, Laszlo Papp wrote:
>>>>> Signed-off-by: Laszlo Papp <lpapp@kde.org>
>>>>> ---
>>>>>    meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb | 27
>>>>> +++++++++++++++++++++++
>>>>>    meta/recipes-bsp/u-boot/u-boot_2013.07.bb         | 23
>>>>> +++++++++++++++++++
>>>>>    2 files changed, 50 insertions(+)
>>>>>    create mode 100644
>>>>> meta/recipes-bsp/u-boot/u-boot-mkimage_2013.07.bb
>>>>>    create mode 100644 meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>>>>
>>>> ...
>>>>> diff --git a/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>>>> b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>>>> new file mode 100644
>>>>> index 0000000..f3f4b62
>>>>> --- /dev/null
>>>>> +++ b/meta/recipes-bsp/u-boot/u-boot_2013.07.bb
>>>>> @@ -0,0 +1,23 @@
>>>>> +require u-boot.inc
>>>>> +
>>>>> +# To build u-boot for your machine, provide the following lines in
>>>>> your machine
>>>>> +# config, replacing the assignments as appropriate for your machine.
>>>>> +# UBOOT_MACHINE = "omap3_beagle_config"
>>>>> +# UBOOT_ENTRYPOINT = "0x80008000"
>>>>> +# UBOOT_LOADADDRESS = "0x80008000"
>>>>> +
>>>>> +LICENSE = "GPLv2+"
>>>>> +LIC_FILES_CHKSUM =
>>>>> "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb \
>>>>> +
>>>>> file://README;beginline=1;endline=22;md5=78b195c11cb6ef63e6985140db7d7bab"
>>>>>
>>>>> +
>>>>> +# This revision corresponds to the tag "v2013.07"
>>>>> +# We use the revision in order to avoid having to fetch it from
>>>>> the repo during parse
>>>>> +SRCREV = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
>>>>> +
>>>>> +PV = "v2013.07+git${SRCPV}"
>>>>> +
>>>>> +SRC_URI = "git://git.denx.de/u-boot.git;branch=master;protocol=git"
>>>>
>>>> Just curious...
>>>> I dont' see a good reason to specify the git protocol twice:
>>>>      git://.... and ; protocol=git
>>>
>>> It's not twice, git:// is to select bitbake fetcher (git.py) and one
>>> fetcher can support multiple protocols (e.g. over git, http or ssh)
>>>
>>> but you're right that in most cases "protocol=git" isn't needed, because
>>> it's default protocol for git fetcher when host is specified in that
>>> SRC_URI.
>>>
>>> bitbake/lib/bb/fetch2/git.py:
>>>          if 'protocol' in ud.parm:
>>>              ud.proto = ud.parm['protocol']
>>>          elif not ud.host:
>>>              ud.proto = 'file'
>>>          else:
>>>              ud.proto = "git"
>>>
>>> And I would support removing it in one go as it clearly confuses people
>>> :).
>>
>> The reason is that the default for the git fetcher was once rsync. Why?
>> The fetcher was written when git was still a new and unexplored
>> landscape. We decided in the end to break that insane default and fix
>> it.
>>
>> We should kill off the now unnecessary parameters though.
>
> Ok. Jackie will have time to make a first pass tomorrow.
> Jackie, please include the replacement script in your email.

I think it should be the for the fetchers of svn, cvs, hg and so on:

$ grep protocol bitbake/lib/bb/fetch2/svn.py
         proto = ud.parm.get('protocol', 'svn')

$ grep protocol bitbake/lib/bb/fetch2/hg.py
         proto = ud.parm.get('protocol', 'http')


I checked that we only use git and svn:
$ find meta \( -name *.bb -o -name *.inc -o -name *.bbclass \) -exec sed 
-n 's/.*"\(\w*:\/\/\).*\(;protocol=\w*\)/\1 \2/p' {} \; | sort | uniq
git:// ;protocol=git \
git:// ;protocol=git"
git:// 
;protocol=git;bareclone=1;branch=${KBRANCH},${KMETA};name=machine,meta"
git:// ;protocol=git;bareclone=1;branch=${KBRANCH},meta;name=machine,meta"
git:// ;protocol=git" % (gstpkg_basename, branch))
git:// 
;protocol=git;nocheckout=1;branch=${KBRANCH},${KMETA};name=machine,meta"
git:// ;protocol=http \
svn:// ;protocol=http \
svn:// ;protocol=http;module=trunk \
svn:// ;protocol=svn"

So I think it's safe to simply remove ";protocol=git" and ";protocol=svn":

$ find meta \( -name *.bb -o -name *.inc -o -name *.bbclass \) -exec sed 
-i 's/;protocol=git//; s/;protocol=svn//' {} \;


What left with protocol parameter after removal is:
$ find meta \( -name *.bb -o -name *.inc -o -name *.bbclass \) -exec sed 
-n 's/.*"\(\w*:\/\/\).*\(;protocol=\w*\)/\1 \2/p' {} \; | sort | uniq
git:// ;protocol=http \
svn:// ;protocol=http \
svn:// ;protocol=http;module=trunk \

I think it should be what we want, I will send the patch after my 
"bitbake world" complete.

Thanks,
Jackie

>
> // Randy
>
>>
>> Cheers,
>>
>> Richard
>>
>>
>>
>
>