[PATCH] u-boot 2011.12: add directory prefix to patch URIs

[PATCH] u-boot 2011.12: add directory prefix to patch URIs

[Cliff Brake]

From: Cliff Brake <cbrake@bec-systems.com>


Signed-off-by: Cliff Brake <cbrake@bec-systems.com>
---
 recipes-bsp/u-boot/u-boot_2011.12.bb |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/recipes-bsp/u-boot/u-boot_2011.12.bb b/recipes-bsp/u-boot/u-boot_2011.12.bb
index 39f77bd..45c64a6 100644
--- a/recipes-bsp/u-boot/u-boot_2011.12.bb
+++ b/recipes-bsp/u-boot/u-boot_2011.12.bb
@@ -30,9 +30,9 @@ SRC_URI = "git://www.denx.de/git/u-boot.git;protocol=git \
            ${CACHEFIX} \
            file://2011.12/0009-Beagleboard-Correct-memory-size-on-rev-C4.patch \
            file://2011.12/0010-OMAP3-Correct-get_sdr_cs_offset-mask.patch \
-           file://0011-ext2load-increase-read-speed.patch \
-           file://0012-ext4fs-ls-load-support.patch \
-           file://0013-beagleboard-switch-mmcroots-to-ext4.patch \
+           file://2011.12/0011-ext2load-increase-read-speed.patch \
+           file://2011.12/0012-ext4fs-ls-load-support.patch \
+           file://2011.12/0013-beagleboard-switch-mmcroots-to-ext4.patch \
            ${FWENV} \
           "
 
-- 
1.7.1

Re: [PATCH] u-boot 2011.12: add directory prefix to patch URIs

[Denys Dmytriyenko]

Cliff,

Thanks for the patch. But it is already the case of proper paths being used in 
the recipe:

http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/diff/recipes-bsp/u-boot/u-boot_2011.12.bb?id=07913cca6cfd7f35a9bb5d6b2cca41c6b5f07ec3

diff --git a/recipes-bsp/u-boot/u-boot_2011.12.bb b/recipes-bsp/u-boot/u-boot_2011.12.bb
index 9616a05..45c64a6 100644
--- a/recipes-bsp/u-boot/u-boot_2011.12.bb
+++ b/recipes-bsp/u-boot/u-boot_2011.12.bb
@@ -6,7 +6,7 @@ UBOOT_IMAGE = "u-boot-${MACHINE}-${PV}-${PR}.img"
 UBOOT_SYMLINK = "u-boot-${MACHINE}.img"
 
 PV = "2011.12"
-PR = "r7"
+PR = "r8"
 
 # No patches for other machines yet
 COMPATIBLE_MACHINE = "(beagleboard|pandaboard|hawkboard|am3517-evm|am37x-evm|omap3evm)"
@@ -30,6 +30,9 @@ SRC_URI = "git://www.denx.de/git/u-boot.git;protocol=git \
            ${CACHEFIX} \
            file://2011.12/0009-Beagleboard-Correct-memory-size-on-rev-C4.patch \
            file://2011.12/0010-OMAP3-Correct-get_sdr_cs_offset-mask.patch \
+           file://2011.12/0011-ext2load-increase-read-speed.patch \
+           file://2011.12/0012-ext4fs-ls-load-support.patch \
+           file://2011.12/0013-beagleboard-switch-mmcroots-to-ext4.patch \
            ${FWENV} \
           "

Did you happen to pick an intermediate version from Koen's WIP tree on github? 
That's not how it was submitted and eventually went into the official tree. 
Sorry for confusion.

-- 
Denys


On Thu, May 10, 2012 at 05:48:19PM -0400, Cliff Brake wrote:
> From: Cliff Brake <cbrake@bec-systems.com>
> 
> 
> Signed-off-by: Cliff Brake <cbrake@bec-systems.com>
> ---
>  recipes-bsp/u-boot/u-boot_2011.12.bb |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/recipes-bsp/u-boot/u-boot_2011.12.bb b/recipes-bsp/u-boot/u-boot_2011.12.bb
> index 39f77bd..45c64a6 100644
> --- a/recipes-bsp/u-boot/u-boot_2011.12.bb
> +++ b/recipes-bsp/u-boot/u-boot_2011.12.bb
> @@ -30,9 +30,9 @@ SRC_URI = "git://www.denx.de/git/u-boot.git;protocol=git \
>             ${CACHEFIX} \
>             file://2011.12/0009-Beagleboard-Correct-memory-size-on-rev-C4.patch \
>             file://2011.12/0010-OMAP3-Correct-get_sdr_cs_offset-mask.patch \
> -           file://0011-ext2load-increase-read-speed.patch \
> -           file://0012-ext4fs-ls-load-support.patch \
> -           file://0013-beagleboard-switch-mmcroots-to-ext4.patch \
> +           file://2011.12/0011-ext2load-increase-read-speed.patch \
> +           file://2011.12/0012-ext4fs-ls-load-support.patch \
> +           file://2011.12/0013-beagleboard-switch-mmcroots-to-ext4.patch \
>             ${FWENV} \
>            "
>  
> -- 
> 1.7.1
> 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
> 

Re: [PATCH] u-boot 2011.12: add directory prefix to patch URIs

[Cliff Brake]

Denys,

Thanks for the clarification -- I made the mistake of thinking the
meta-ti master was on Github (saw it there when Angstrom moved).

https://github.com/Angstrom-distribution/meta-ti

I notice now at the top of the above page, it says it is a mirror, but
it may be a good idea to mention in the README where the master repo
is to avoid confusion.

Thanks,
Cliff

On Fri, May 11, 2012 at 12:47 AM, Denys Dmytriyenko <denis@denix.org> wrote:
> Cliff,
>
> Thanks for the patch. But it is already the case of proper paths being used in
> the recipe:
>
> http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/diff/recipes-bsp/u-boot/u-boot_2011.12.bb?id=07913cca6cfd7f35a9bb5d6b2cca41c6b5f07ec3
>
> diff --git a/recipes-bsp/u-boot/u-boot_2011.12.bb b/recipes-bsp/u-boot/u-boot_2011.12.bb
> index 9616a05..45c64a6 100644
> --- a/recipes-bsp/u-boot/u-boot_2011.12.bb
> +++ b/recipes-bsp/u-boot/u-boot_2011.12.bb
> @@ -6,7 +6,7 @@ UBOOT_IMAGE = "u-boot-${MACHINE}-${PV}-${PR}.img"
>  UBOOT_SYMLINK = "u-boot-${MACHINE}.img"
>
>  PV = "2011.12"
> -PR = "r7"
> +PR = "r8"
>
>  # No patches for other machines yet
>  COMPATIBLE_MACHINE = "(beagleboard|pandaboard|hawkboard|am3517-evm|am37x-evm|omap3evm)"
> @@ -30,6 +30,9 @@ SRC_URI = "git://www.denx.de/git/u-boot.git;protocol=git \
>            ${CACHEFIX} \
>            file://2011.12/0009-Beagleboard-Correct-memory-size-on-rev-C4.patch \
>            file://2011.12/0010-OMAP3-Correct-get_sdr_cs_offset-mask.patch \
> +           file://2011.12/0011-ext2load-increase-read-speed.patch \
> +           file://2011.12/0012-ext4fs-ls-load-support.patch \
> +           file://2011.12/0013-beagleboard-switch-mmcroots-to-ext4.patch \
>            ${FWENV} \
>           "
>
> Did you happen to pick an intermediate version from Koen's WIP tree on github?
> That's not how it was submitted and eventually went into the official tree.
> Sorry for confusion.
>
> --
> Denys
>
>
> On Thu, May 10, 2012 at 05:48:19PM -0400, Cliff Brake wrote:
>> From: Cliff Brake <cbrake@bec-systems.com>
>>
>>
>> Signed-off-by: Cliff Brake <cbrake@bec-systems.com>
>> ---
>>  recipes-bsp/u-boot/u-boot_2011.12.bb |    6 +++---
>>  1 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/recipes-bsp/u-boot/u-boot_2011.12.bb b/recipes-bsp/u-boot/u-boot_2011.12.bb
>> index 39f77bd..45c64a6 100644
>> --- a/recipes-bsp/u-boot/u-boot_2011.12.bb
>> +++ b/recipes-bsp/u-boot/u-boot_2011.12.bb
>> @@ -30,9 +30,9 @@ SRC_URI = "git://www.denx.de/git/u-boot.git;protocol=git \
>>             ${CACHEFIX} \
>>             file://2011.12/0009-Beagleboard-Correct-memory-size-on-rev-C4.patch \
>>             file://2011.12/0010-OMAP3-Correct-get_sdr_cs_offset-mask.patch \
>> -           file://0011-ext2load-increase-read-speed.patch \
>> -           file://0012-ext4fs-ls-load-support.patch \
>> -           file://0013-beagleboard-switch-mmcroots-to-ext4.patch \
>> +           file://2011.12/0011-ext2load-increase-read-speed.patch \
>> +           file://2011.12/0012-ext4fs-ls-load-support.patch \
>> +           file://2011.12/0013-beagleboard-switch-mmcroots-to-ext4.patch \
>>             ${FWENV} \
>>            "
>>
>> --
>> 1.7.1
>>
>> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti
>>



-- 
=================
http://bec-systems.com

Re: [PATCH] u-boot 2011.12: add directory prefix to patch URIs

[Steffen Sledz]

On 11.05.2012 13:10, Cliff Brake wrote:
> Denys,
> 
> Thanks for the clarification -- I made the mistake of thinking the
> meta-ti master was on Github (saw it there when Angstrom moved).
> 
> https://github.com/Angstrom-distribution/meta-ti
> 
> I notice now at the top of the above page, it says it is a mirror, but
> it may be a good idea to mention in the README where the master repo
> is to avoid confusion.

I hit the same confusion and mentioned it in thread <https://lists.yoctoproject.org/pipermail/meta-ti/2012-April/000975.html>.

Unfortunately there was no final clarification by the meta-ti maintainers until now, just a short "And you're thinking wrong". :(

Steffen

-- 
DResearch Fahrzeugelektronik GmbH
Otto-Schmirgal-Str. 3, 10319 Berlin, Germany
Tel: +49 30 515932-237 mailto:sledz@dresearch-fe.de
Fax: +49 30 515932-299
Geschäftsführer: Dr. Michael Weber, Werner Mögle;
Amtsgericht Berlin Charlottenburg; HRB 130120 B;
Ust.-IDNr. DE273952058

Re: [PATCH] u-boot 2011.12: add directory prefix to patch URIs

[Koen Kooi]

Op 11 mei 2012, om 13:10 heeft Cliff Brake het volgende geschreven:

> Denys,
> 
> Thanks for the clarification -- I made the mistake of thinking the
> meta-ti master was on Github (saw it there when Angstrom moved).
> 
> https://github.com/Angstrom-distribution/meta-ti
> 
> I notice now at the top of the above page, it says it is a mirror, but
> it may be a good idea to mention in the README where the master repo

The master repo is behind the TI firewall so listing that is only of use if you can VPN into TI :)

regards,

Koen