Patchwork Changed SRC_URI for eglibc repo.

login
register
mail settings
Submitter Martin Ertsaas
Date July 19, 2012, 11:15 a.m.
Message ID <1342696517-28149-1-git-send-email-mertsas@cisco.com>
Download mbox | patch
Permalink /patch/32499/
State New
Headers show

Comments

Martin Ertsaas - July 19, 2012, 11:15 a.m.
The branches repo for eglibc is /branches, not /svn/branches.

Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
---
 .../eglibc/cross-localedef-native_2.15.bb          |    2 +-
 .../eglibc/cross-localedef-native_2.16.bb          |    2 +-
 meta/recipes-core/eglibc/eglibc_2.15.bb            |    2 +-
 meta/recipes-core/eglibc/eglibc_2.16.bb            |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)
Martin Ertsaas - July 19, 2012, 11:34 a.m.
On 07/19/12 13:35, Martin Jansa wrote:
> On Thu, Jul 19, 2012 at 01:15:17PM +0200, Martin Ertsaas wrote:
>> The branches repo for eglibc is /branches, not /svn/branches.
> Are you sure? wasn't this tested on broken bitbake where protocol param
> was ignored and default svn:// protocol was used instead http://?
>
> Cheers,
when was bitbake fixed for this? I have commit number
a93a58b7cea2ddb9596cb4d718c0801db5d1a295 from two days ago now. Was it a
broken bitbake?

- Martin

>
>> Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
>> ---
>>  .../eglibc/cross-localedef-native_2.15.bb          |    2 +-
>>  .../eglibc/cross-localedef-native_2.16.bb          |    2 +-
>>  meta/recipes-core/eglibc/eglibc_2.15.bb            |    2 +-
>>  meta/recipes-core/eglibc/eglibc_2.16.bb            |    2 +-
>>  4 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb b/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
>> index 422e0bb..4301174 100644
>> --- a/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
>> +++ b/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
>> @@ -16,7 +16,7 @@ inherit autotools
>>  PR = "r0"
>>  SRCREV="15225"
>>  EGLIBC_BRANCH="eglibc-2_14"
>> -SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http "
>> +SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http "
>>  S = "${WORKDIR}/${EGLIBC_BRANCH}/localedef"
>>  
>>  do_unpack_append() {
>> diff --git a/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb b/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
>> index 47f0834..12f1d8b 100644
>> --- a/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
>> +++ b/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
>> @@ -16,7 +16,7 @@ inherit autotools
>>  PR = "r0"
>>  SRCREV="19383"
>>  EGLIBC_BRANCH="eglibc-2_16"
>> -SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http "
>> +SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http "
>>  S = "${WORKDIR}/${EGLIBC_BRANCH}/localedef"
>>  
>>  do_unpack_append() {
>> diff --git a/meta/recipes-core/eglibc/eglibc_2.15.bb b/meta/recipes-core/eglibc/eglibc_2.15.bb
>> index 4d4d2cf..a72b076 100644
>> --- a/meta/recipes-core/eglibc/eglibc_2.15.bb
>> +++ b/meta/recipes-core/eglibc/eglibc_2.15.bb
>> @@ -7,7 +7,7 @@ PR = "r12"
>>  PR_append = "+svnr${SRCPV}"
>>  
>>  EGLIBC_BRANCH="eglibc-2_15"
>> -SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http \
>> +SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http \
>>             file://eglibc-svn-arm-lowlevellock-include-tls.patch \
>>             file://IO-acquire-lock-fix.patch \
>>             file://mips-rld-map-check.patch \
>> diff --git a/meta/recipes-core/eglibc/eglibc_2.16.bb b/meta/recipes-core/eglibc/eglibc_2.16.bb
>> index 7e7d68b..d0cdd68 100644
>> --- a/meta/recipes-core/eglibc/eglibc_2.16.bb
>> +++ b/meta/recipes-core/eglibc/eglibc_2.16.bb
>> @@ -7,7 +7,7 @@ PR = "r1"
>>  PR_append = "+svnr${SRCPV}"
>>  
>>  EGLIBC_BRANCH="eglibc-2_16"
>> -SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http \
>> +SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http \
>>             file://eglibc-svn-arm-lowlevellock-include-tls.patch \
>>             file://IO-acquire-lock-fix.patch \
>>             file://mips-rld-map-check.patch \
>> -- 
>> 1.7.8.6
>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Martin Jansa - July 19, 2012, 11:35 a.m.
On Thu, Jul 19, 2012 at 01:15:17PM +0200, Martin Ertsaas wrote:
> The branches repo for eglibc is /branches, not /svn/branches.

Are you sure? wasn't this tested on broken bitbake where protocol param
was ignored and default svn:// protocol was used instead http://?

Cheers,

> 
> Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
> ---
>  .../eglibc/cross-localedef-native_2.15.bb          |    2 +-
>  .../eglibc/cross-localedef-native_2.16.bb          |    2 +-
>  meta/recipes-core/eglibc/eglibc_2.15.bb            |    2 +-
>  meta/recipes-core/eglibc/eglibc_2.16.bb            |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb b/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
> index 422e0bb..4301174 100644
> --- a/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
> +++ b/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
> @@ -16,7 +16,7 @@ inherit autotools
>  PR = "r0"
>  SRCREV="15225"
>  EGLIBC_BRANCH="eglibc-2_14"
> -SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http "
> +SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http "
>  S = "${WORKDIR}/${EGLIBC_BRANCH}/localedef"
>  
>  do_unpack_append() {
> diff --git a/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb b/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
> index 47f0834..12f1d8b 100644
> --- a/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
> +++ b/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
> @@ -16,7 +16,7 @@ inherit autotools
>  PR = "r0"
>  SRCREV="19383"
>  EGLIBC_BRANCH="eglibc-2_16"
> -SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http "
> +SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http "
>  S = "${WORKDIR}/${EGLIBC_BRANCH}/localedef"
>  
>  do_unpack_append() {
> diff --git a/meta/recipes-core/eglibc/eglibc_2.15.bb b/meta/recipes-core/eglibc/eglibc_2.15.bb
> index 4d4d2cf..a72b076 100644
> --- a/meta/recipes-core/eglibc/eglibc_2.15.bb
> +++ b/meta/recipes-core/eglibc/eglibc_2.15.bb
> @@ -7,7 +7,7 @@ PR = "r12"
>  PR_append = "+svnr${SRCPV}"
>  
>  EGLIBC_BRANCH="eglibc-2_15"
> -SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http \
> +SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http \
>             file://eglibc-svn-arm-lowlevellock-include-tls.patch \
>             file://IO-acquire-lock-fix.patch \
>             file://mips-rld-map-check.patch \
> diff --git a/meta/recipes-core/eglibc/eglibc_2.16.bb b/meta/recipes-core/eglibc/eglibc_2.16.bb
> index 7e7d68b..d0cdd68 100644
> --- a/meta/recipes-core/eglibc/eglibc_2.16.bb
> +++ b/meta/recipes-core/eglibc/eglibc_2.16.bb
> @@ -7,7 +7,7 @@ PR = "r1"
>  PR_append = "+svnr${SRCPV}"
>  
>  EGLIBC_BRANCH="eglibc-2_16"
> -SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http \
> +SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http \
>             file://eglibc-svn-arm-lowlevellock-include-tls.patch \
>             file://IO-acquire-lock-fix.patch \
>             file://mips-rld-map-check.patch \
> -- 
> 1.7.8.6
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Martin Ertsaas - July 19, 2012, 11:38 a.m.
On 07/19/12 13:41, Richard Purdie wrote:
> On Thu, 2012-07-19 at 13:34 +0200, Martin Ertsås wrote:
>> On 07/19/12 13:35, Martin Jansa wrote:
>>> On Thu, Jul 19, 2012 at 01:15:17PM +0200, Martin Ertsaas wrote:
>>>> The branches repo for eglibc is /branches, not /svn/branches.
>>> Are you sure? wasn't this tested on broken bitbake where protocol param
>>> was ignored and default svn:// protocol was used instead http://?
>>>
>>> Cheers,
>> when was bitbake fixed for this? I have commit number
>> a93a58b7cea2ddb9596cb4d718c0801db5d1a295 from two days ago now. Was it a
>> broken bitbake?
> The commit immediately after that,
> http://git.openembedded.org/bitbake/commit/?id=b2017f493ab730d804ae44ec5a168d464626d046
> is the fix referred to.
>
> Cheers,
That fixed the problem. Thanks.
>
> Richard
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Richard Purdie - July 19, 2012, 11:41 a.m.
On Thu, 2012-07-19 at 13:34 +0200, Martin Ertsås wrote:
> On 07/19/12 13:35, Martin Jansa wrote:
> > On Thu, Jul 19, 2012 at 01:15:17PM +0200, Martin Ertsaas wrote:
> >> The branches repo for eglibc is /branches, not /svn/branches.
> > Are you sure? wasn't this tested on broken bitbake where protocol param
> > was ignored and default svn:// protocol was used instead http://?
> >
> > Cheers,
> when was bitbake fixed for this? I have commit number
> a93a58b7cea2ddb9596cb4d718c0801db5d1a295 from two days ago now. Was it a
> broken bitbake?

The commit immediately after that,
http://git.openembedded.org/bitbake/commit/?id=b2017f493ab730d804ae44ec5a168d464626d046
is the fix referred to.

Cheers,

Richard

Patch

diff --git a/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb b/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
index 422e0bb..4301174 100644
--- a/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
+++ b/meta/recipes-core/eglibc/cross-localedef-native_2.15.bb
@@ -16,7 +16,7 @@  inherit autotools
 PR = "r0"
 SRCREV="15225"
 EGLIBC_BRANCH="eglibc-2_14"
-SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http "
+SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http "
 S = "${WORKDIR}/${EGLIBC_BRANCH}/localedef"
 
 do_unpack_append() {
diff --git a/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb b/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
index 47f0834..12f1d8b 100644
--- a/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
+++ b/meta/recipes-core/eglibc/cross-localedef-native_2.16.bb
@@ -16,7 +16,7 @@  inherit autotools
 PR = "r0"
 SRCREV="19383"
 EGLIBC_BRANCH="eglibc-2_16"
-SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http "
+SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http "
 S = "${WORKDIR}/${EGLIBC_BRANCH}/localedef"
 
 do_unpack_append() {
diff --git a/meta/recipes-core/eglibc/eglibc_2.15.bb b/meta/recipes-core/eglibc/eglibc_2.15.bb
index 4d4d2cf..a72b076 100644
--- a/meta/recipes-core/eglibc/eglibc_2.15.bb
+++ b/meta/recipes-core/eglibc/eglibc_2.15.bb
@@ -7,7 +7,7 @@  PR = "r12"
 PR_append = "+svnr${SRCPV}"
 
 EGLIBC_BRANCH="eglibc-2_15"
-SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http \
+SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http \
            file://eglibc-svn-arm-lowlevellock-include-tls.patch \
            file://IO-acquire-lock-fix.patch \
            file://mips-rld-map-check.patch \
diff --git a/meta/recipes-core/eglibc/eglibc_2.16.bb b/meta/recipes-core/eglibc/eglibc_2.16.bb
index 7e7d68b..d0cdd68 100644
--- a/meta/recipes-core/eglibc/eglibc_2.16.bb
+++ b/meta/recipes-core/eglibc/eglibc_2.16.bb
@@ -7,7 +7,7 @@  PR = "r1"
 PR_append = "+svnr${SRCPV}"
 
 EGLIBC_BRANCH="eglibc-2_16"
-SRC_URI = "svn://www.eglibc.org/svn/branches/;module=${EGLIBC_BRANCH};protocol=http \
+SRC_URI = "svn://www.eglibc.org/branches/;module=${EGLIBC_BRANCH};protocol=http \
            file://eglibc-svn-arm-lowlevellock-include-tls.patch \
            file://IO-acquire-lock-fix.patch \
            file://mips-rld-map-check.patch \