docs.lists.yoctoproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] documentation: update sources mirror URL
@ 2021-10-05 22:45 Jon Mason
  2021-10-05 22:45 ` [PATCH 2/2] bitbake: bitbake-user-manual: " Jon Mason
  2021-10-06 14:41 ` [docs] [PATCH 1/2] documentation: " Michael Opdenacker
  0 siblings, 2 replies; 9+ messages in thread
From: Jon Mason @ 2021-10-05 22:45 UTC (permalink / raw)
  To: docs, bitbake-devel

The URL for the sources mirror no longer works.  Update to the new
location.

Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
 documentation/ref-manual/faq.rst       | 15 ++++++++-------
 documentation/ref-manual/variables.rst |  8 ++++----
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
index d3a603d4a438..f06abc84ee6f 100644
--- a/documentation/ref-manual/faq.rst
+++ b/documentation/ref-manual/faq.rst
@@ -302,10 +302,10 @@ attempt before any others by adding something like the following to the
 ``local.conf`` configuration file::
 
    PREMIRRORS:prepend = "\
-       git://.*/.* http://www.yoctoproject.org/sources/ \n \
-       ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
-       http://.*/.* http://www.yoctoproject.org/sources/ \n \
-       https://.*/.* http://www.yoctoproject.org/sources/ \n"
+       git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+       ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+       http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+       https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
 
 These changes cause the build system to intercept Git, FTP, HTTP, and
 HTTPS requests and direct them to the ``http://`` sources mirror. You
@@ -342,9 +342,10 @@ You could make the following changes to the ``local.conf`` configuration
 file as long as the :term:`PREMIRRORS` server is current::
 
    PREMIRRORS:prepend = "\
-       ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
-       http://.*/.* http://www.yoctoproject.org/sources/ \n \
-       https://.*/.* http://www.yoctoproject.org/sources/ \n"
+       git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+       ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+       http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+       https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
    BB_FETCH_PREMIRRORONLY = "1"
 
 These changes would cause the build system to successfully fetch source
diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
index 5f6f91146bf0..97782d8ec7ed 100644
--- a/documentation/ref-manual/variables.rst
+++ b/documentation/ref-manual/variables.rst
@@ -5838,10 +5838,10 @@ system and gives an overview of their function and contents.
       :term:`Build Directory`::
 
          PREMIRRORS:prepend = "\
-             git://.*/.* http://www.yoctoproject.org/sources/ \n \
-             ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
-             http://.*/.* http://www.yoctoproject.org/sources/ \n \
-             https://.*/.* http://www.yoctoproject.org/sources/ \n"
+             git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+             ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+             http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+             https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
 
       These changes cause the
       build system to intercept Git, FTP, HTTP, and HTTPS requests and
-- 
2.20.1



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 2/2] bitbake: bitbake-user-manual: update sources mirror URL
  2021-10-05 22:45 [PATCH 1/2] documentation: update sources mirror URL Jon Mason
@ 2021-10-05 22:45 ` Jon Mason
  2021-10-06 14:41 ` [docs] [PATCH 1/2] documentation: " Michael Opdenacker
  1 sibling, 0 replies; 9+ messages in thread
From: Jon Mason @ 2021-10-05 22:45 UTC (permalink / raw)
  To: docs, bitbake-devel

The URL for the sources mirror no longer works.  Update to the new
location.

Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
 .../bitbake-user-manual-ref-variables.rst                 | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-ref-variables.rst b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-ref-variables.rst
index 8862e16b83cb..11f8fdcde003 100644
--- a/bitbake/doc/bitbake-user-manual/bitbake-user-manual-ref-variables.rst
+++ b/bitbake/doc/bitbake-user-manual/bitbake-user-manual-ref-variables.rst
@@ -1123,10 +1123,10 @@ overview of their function and contents.
       your configuration::
 
          PREMIRRORS:prepend = "\
-         git://.*/.* http://www.yoctoproject.org/sources/ \n \
-         ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
-         http://.*/.* http://www.yoctoproject.org/sources/ \n \
-         https://.*/.* http://www.yoctoproject.org/sources/ \n"
+         git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+         ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+         http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
+         https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
 
       These changes cause the build system to intercept Git, FTP, HTTP, and
       HTTPS requests and direct them to the ``http://`` sources mirror. You can
-- 
2.20.1



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [docs] [PATCH 1/2] documentation: update sources mirror URL
  2021-10-05 22:45 [PATCH 1/2] documentation: update sources mirror URL Jon Mason
  2021-10-05 22:45 ` [PATCH 2/2] bitbake: bitbake-user-manual: " Jon Mason
@ 2021-10-06 14:41 ` Michael Opdenacker
  2021-10-06 19:53   ` Jon Mason
  1 sibling, 1 reply; 9+ messages in thread
From: Michael Opdenacker @ 2021-10-06 14:41 UTC (permalink / raw)
  To: Jon Mason, docs, bitbake-devel

Hi Jon

Thanks for the proposed update !

On 10/6/21 12:45 AM, Jon Mason wrote:
> The URL for the sources mirror no longer works.  Update to the new
> location.
>
> Signed-off-by: Jon Mason <jdmason@kudzu.us>
> ---
>  documentation/ref-manual/faq.rst       | 15 ++++++++-------
>  documentation/ref-manual/variables.rst |  8 ++++----
>  2 files changed, 12 insertions(+), 11 deletions(-)
>
> diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
> index d3a603d4a438..f06abc84ee6f 100644
> --- a/documentation/ref-manual/faq.rst
> +++ b/documentation/ref-manual/faq.rst
> @@ -302,10 +302,10 @@ attempt before any others by adding something like the following to the
>  ``local.conf`` configuration file::
>  
>     PREMIRRORS:prepend = "\
> -       git://.*/.* http://www.yoctoproject.org/sources/ \n \
> -       ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
> -       http://.*/.* http://www.yoctoproject.org/sources/ \n \
> -       https://.*/.* http://www.yoctoproject.org/sources/ \n"
> +       git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> +       ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> +       http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> +       https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"


Hey, doesn't it sound weird to show an example that proposes to use http
when https is usually recommended?
What about proposing https://downloads.yoctoproject.org/mirror/sources/
instead?

Or we don't really care because we use source checksums anyway?

Just my two cents. I don't mind accepting the patch as is.

Thanks again,
Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [docs] [PATCH 1/2] documentation: update sources mirror URL
  2021-10-06 14:41 ` [docs] [PATCH 1/2] documentation: " Michael Opdenacker
@ 2021-10-06 19:53   ` Jon Mason
  2021-10-06 22:17     ` [bitbake-devel] " Richard Purdie
  0 siblings, 1 reply; 9+ messages in thread
From: Jon Mason @ 2021-10-06 19:53 UTC (permalink / raw)
  To: Michael Opdenacker; +Cc: docs, bitbake-devel

On Wed, Oct 6, 2021 at 10:41 AM Michael Opdenacker
<michael.opdenacker@bootlin.com> wrote:
>
> Hi Jon
>
> Thanks for the proposed update !
>
> On 10/6/21 12:45 AM, Jon Mason wrote:
> > The URL for the sources mirror no longer works.  Update to the new
> > location.
> >
> > Signed-off-by: Jon Mason <jdmason@kudzu.us>
> > ---
> >  documentation/ref-manual/faq.rst       | 15 ++++++++-------
> >  documentation/ref-manual/variables.rst |  8 ++++----
> >  2 files changed, 12 insertions(+), 11 deletions(-)
> >
> > diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
> > index d3a603d4a438..f06abc84ee6f 100644
> > --- a/documentation/ref-manual/faq.rst
> > +++ b/documentation/ref-manual/faq.rst
> > @@ -302,10 +302,10 @@ attempt before any others by adding something like the following to the
> >  ``local.conf`` configuration file::
> >
> >     PREMIRRORS:prepend = "\
> > -       git://.*/.* http://www.yoctoproject.org/sources/ \n \
> > -       ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
> > -       http://.*/.* http://www.yoctoproject.org/sources/ \n \
> > -       https://.*/.* http://www.yoctoproject.org/sources/ \n"
> > +       git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > +       ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > +       http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > +       https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
>
>
> Hey, doesn't it sound weird to show an example that proposes to use http
> when https is usually recommended?
> What about proposing https://downloads.yoctoproject.org/mirror/sources/
> instead?
>
> Or we don't really care because we use source checksums anyway?
>
> Just my two cents. I don't mind accepting the patch as is.

A quick and dirty grep of the trees shows many occurrences of
downloads.yoctoproject.org without https.  Perhaps it would be good to
correct all of these in a follow-on patch.  I can do a quick patch of
all of these and get that out in the next day or so.

Also, looking at documentation/profile-manual/usage.rst (where most of
the occurrences in the docs are), shouldn't we have a newer kernel
than 2.6.19.2 as an example?  I mean, it's almost 15 years old.

Thanks,
Jon

> Thanks again,
> Michael.
>
> --
> Michael Opdenacker, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [bitbake-devel] [docs] [PATCH 1/2] documentation: update sources mirror URL
  2021-10-06 19:53   ` Jon Mason
@ 2021-10-06 22:17     ` Richard Purdie
  2021-10-07  0:48       ` Jon Mason
                         ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Richard Purdie @ 2021-10-06 22:17 UTC (permalink / raw)
  To: Jon Mason, Michael Opdenacker; +Cc: docs, bitbake-devel

On Wed, 2021-10-06 at 15:53 -0400, Jon Mason wrote:
> On Wed, Oct 6, 2021 at 10:41 AM Michael Opdenacker
> <michael.opdenacker@bootlin.com> wrote:
> > 
> > Hi Jon
> > 
> > Thanks for the proposed update !
> > 
> > On 10/6/21 12:45 AM, Jon Mason wrote:
> > > The URL for the sources mirror no longer works.  Update to the new
> > > location.
> > > 
> > > Signed-off-by: Jon Mason <jdmason@kudzu.us>
> > > ---
> > >  documentation/ref-manual/faq.rst       | 15 ++++++++-------
> > >  documentation/ref-manual/variables.rst |  8 ++++----
> > >  2 files changed, 12 insertions(+), 11 deletions(-)
> > > 
> > > diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
> > > index d3a603d4a438..f06abc84ee6f 100644
> > > --- a/documentation/ref-manual/faq.rst
> > > +++ b/documentation/ref-manual/faq.rst
> > > @@ -302,10 +302,10 @@ attempt before any others by adding something like the following to the
> > >  ``local.conf`` configuration file::
> > > 
> > >     PREMIRRORS:prepend = "\
> > > -       git://.*/.* http://www.yoctoproject.org/sources/ \n \
> > > -       ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
> > > -       http://.*/.* http://www.yoctoproject.org/sources/ \n \
> > > -       https://.*/.* http://www.yoctoproject.org/sources/ \n"
> > > +       git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > > +       ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > > +       http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > > +       https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
> > 
> > 
> > Hey, doesn't it sound weird to show an example that proposes to use http
> > when https is usually recommended?
> > What about proposing https://downloads.yoctoproject.org/mirror/sources/
> > instead?
> > 
> > Or we don't really care because we use source checksums anyway?
> > 
> > Just my two cents. I don't mind accepting the patch as is.
> 
> A quick and dirty grep of the trees shows many occurrences of
> downloads.yoctoproject.org without https.  Perhaps it would be good to
> correct all of these in a follow-on patch.  I can do a quick patch of
> all of these and get that out in the next day or so.
> 
> Also, looking at documentation/profile-manual/usage.rst (where most of
> the occurrences in the docs are), shouldn't we have a newer kernel
> than 2.6.19.2 as an example?  I mean, it's almost 15 years old.

Careful, some of us remember working on the 2.6.19 kernel and have patches in it
:)

I don't think we've worried too much about http vs https for the downloads as
they are checksumed but it is probably cleaner just to convert them.

Cheers,

Richard







^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [bitbake-devel] [docs] [PATCH 1/2] documentation: update sources mirror URL
  2021-10-06 22:17     ` [bitbake-devel] " Richard Purdie
@ 2021-10-07  0:48       ` Jon Mason
  2021-10-07  8:14         ` Quentin Schulz
  2021-10-07  8:42       ` Michael Opdenacker
       [not found]       ` <16ABB308824CB520.3789@lists.yoctoproject.org>
  2 siblings, 1 reply; 9+ messages in thread
From: Jon Mason @ 2021-10-07  0:48 UTC (permalink / raw)
  To: Richard Purdie; +Cc: Michael Opdenacker, docs, bitbake-devel

On Wed, Oct 6, 2021 at 6:17 PM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> On Wed, 2021-10-06 at 15:53 -0400, Jon Mason wrote:
> > On Wed, Oct 6, 2021 at 10:41 AM Michael Opdenacker
> > <michael.opdenacker@bootlin.com> wrote:
> > >
> > > Hi Jon
> > >
> > > Thanks for the proposed update !
> > >
> > > On 10/6/21 12:45 AM, Jon Mason wrote:
> > > > The URL for the sources mirror no longer works.  Update to the new
> > > > location.
> > > >
> > > > Signed-off-by: Jon Mason <jdmason@kudzu.us>
> > > > ---
> > > >  documentation/ref-manual/faq.rst       | 15 ++++++++-------
> > > >  documentation/ref-manual/variables.rst |  8 ++++----
> > > >  2 files changed, 12 insertions(+), 11 deletions(-)
> > > >
> > > > diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
> > > > index d3a603d4a438..f06abc84ee6f 100644
> > > > --- a/documentation/ref-manual/faq.rst
> > > > +++ b/documentation/ref-manual/faq.rst
> > > > @@ -302,10 +302,10 @@ attempt before any others by adding something like the following to the
> > > >  ``local.conf`` configuration file::
> > > >
> > > >     PREMIRRORS:prepend = "\
> > > > -       git://.*/.* http://www.yoctoproject.org/sources/ \n \
> > > > -       ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
> > > > -       http://.*/.* http://www.yoctoproject.org/sources/ \n \
> > > > -       https://.*/.* http://www.yoctoproject.org/sources/ \n"
> > > > +       git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > > > +       ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > > > +       http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
> > > > +       https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
> > >
> > >
> > > Hey, doesn't it sound weird to show an example that proposes to use http
> > > when https is usually recommended?
> > > What about proposing https://downloads.yoctoproject.org/mirror/sources/
> > > instead?
> > >
> > > Or we don't really care because we use source checksums anyway?
> > >
> > > Just my two cents. I don't mind accepting the patch as is.
> >
> > A quick and dirty grep of the trees shows many occurrences of
> > downloads.yoctoproject.org without https.  Perhaps it would be good to
> > correct all of these in a follow-on patch.  I can do a quick patch of
> > all of these and get that out in the next day or so.
> >
> > Also, looking at documentation/profile-manual/usage.rst (where most of
> > the occurrences in the docs are), shouldn't we have a newer kernel
> > than 2.6.19.2 as an example?  I mean, it's almost 15 years old.
>
> Careful, some of us remember working on the 2.6.19 kernel and have patches in it
> :)
>
> I don't think we've worried too much about http vs https for the downloads as
> they are checksumed but it is probably cleaner just to convert them.

It's a fool's errand, and I'm just the fool to do it.  Changes all
over the place, broken URLs, etc.  Patches coming shortly (as soon as
my sanity tests make sure I didn't break anything obvious).

Thanks,
Jon


>
> Cheers,
>
> Richard
>
>
>
>
>


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [bitbake-devel] [docs] [PATCH 1/2] documentation: update sources mirror URL
  2021-10-07  0:48       ` Jon Mason
@ 2021-10-07  8:14         ` Quentin Schulz
  0 siblings, 0 replies; 9+ messages in thread
From: Quentin Schulz @ 2021-10-07  8:14 UTC (permalink / raw)
  To: Jon Mason; +Cc: Richard Purdie, Michael Opdenacker, docs, bitbake-devel

Hi Jon,

On Wed, Oct 06, 2021 at 08:48:05PM -0400, Jon Mason wrote:
> On Wed, Oct 6, 2021 at 6:17 PM Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >
> > On Wed, 2021-10-06 at 15:53 -0400, Jon Mason wrote:
> > > On Wed, Oct 6, 2021 at 10:41 AM Michael Opdenacker
> > > <michael.opdenacker@bootlin.com> wrote:
> > > >
> > > > Hi Jon
> > > >
> > > > Thanks for the proposed update !
> > > >
> > > > On 10/6/21 12:45 AM, Jon Mason wrote:
[...]
> > > A quick and dirty grep of the trees shows many occurrences of
> > > downloads.yoctoproject.org without https.  Perhaps it would be good to
> > > correct all of these in a follow-on patch.  I can do a quick patch of
> > > all of these and get that out in the next day or so.
> > >
> > > Also, looking at documentation/profile-manual/usage.rst (where most of
> > > the occurrences in the docs are), shouldn't we have a newer kernel
> > > than 2.6.19.2 as an example?  I mean, it's almost 15 years old.
> >
> > Careful, some of us remember working on the 2.6.19 kernel and have patches in it
> > :)
> >
> > I don't think we've worried too much about http vs https for the downloads as
> > they are checksumed but it is probably cleaner just to convert them.
> 
> It's a fool's errand, and I'm just the fool to do it.  Changes all
> over the place, broken URLs, etc.  Patches coming shortly (as soon as
> my sanity tests make sure I didn't break anything obvious).
> 

For non-literal blocks (where extlinks aren't replaced), please use
:yocto_dl: to replace downloads.yoctoproject.org :)

Thanks!
Quentin


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [bitbake-devel] [docs] [PATCH 1/2] documentation: update sources mirror URL
  2021-10-06 22:17     ` [bitbake-devel] " Richard Purdie
  2021-10-07  0:48       ` Jon Mason
@ 2021-10-07  8:42       ` Michael Opdenacker
       [not found]       ` <16ABB308824CB520.3789@lists.yoctoproject.org>
  2 siblings, 0 replies; 9+ messages in thread
From: Michael Opdenacker @ 2021-10-07  8:42 UTC (permalink / raw)
  To: Richard Purdie, Jon Mason; +Cc: docs, bitbake-devel

Hi,

On 10/7/21 12:17 AM, Richard Purdie wrote:
> On Wed, 2021-10-06 at 15:53 -0400, Jon Mason wrote:
>>
>>> Thanks for the proposed update !
>>>
>>> On 10/6/21 12:45 AM, Jon Mason wrote:
>>>> The URL for the sources mirror no longer works.  Update to the new
>>>> location.
>>>>
>>>> Signed-off-by: Jon Mason <jdmason@kudzu.us>
>>>> ---
>>>>  documentation/ref-manual/faq.rst       | 15 ++++++++-------
>>>>  documentation/ref-manual/variables.rst |  8 ++++----
>>>>  2 files changed, 12 insertions(+), 11 deletions(-)
>>>>
>>>> diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
>>>> index d3a603d4a438..f06abc84ee6f 100644
>>>> --- a/documentation/ref-manual/faq.rst
>>>> +++ b/documentation/ref-manual/faq.rst
>>>> @@ -302,10 +302,10 @@ attempt before any others by adding something like the following to the
>>>>  ``local.conf`` configuration file::
>>>>
>>>>     PREMIRRORS:prepend = "\
>>>> -       git://.*/.* http://www.yoctoproject.org/sources/ \n \
>>>> -       ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
>>>> -       http://.*/.* http://www.yoctoproject.org/sources/ \n \
>>>> -       https://.*/.* http://www.yoctoproject.org/sources/ \n"
>>>> +       git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
>>>> +       ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
>>>> +       http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
>>>> +       https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
>>>
>>> Hey, doesn't it sound weird to show an example that proposes to use http
>>> when https is usually recommended?
>>> What about proposing https://downloads.yoctoproject.org/mirror/sources/
>>> instead?
>>>
>>> Or we don't really care because we use source checksums anyway?
>>>
>>> Just my two cents. I don't mind accepting the patch as is.
>> A quick and dirty grep of the trees shows many occurrences of
>> downloads.yoctoproject.org without https.  Perhaps it would be good to
>> correct all of these in a follow-on patch.  I can do a quick patch of
>> all of these and get that out in the next day or so.



OK to do it as a follow-up patch. This way, I can already take yours (at
least the ones for yocto-docs, which I maintain).

>>
>> Also, looking at documentation/profile-manual/usage.rst (where most of
>> the occurrences in the docs are), shouldn't we have a newer kernel
>> than 2.6.19.2 as an example?  I mean, it's almost 15 years old.
> Careful, some of us remember working on the 2.6.19 kernel and have patches in it
> :)


I can take care of this one too.

>
> I don't think we've worried too much about http vs https for the downloads as
> they are checksumed but it is probably cleaner just to convert them.


Agreed, but let's do it as a follow-up patch.

Cheers,
Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [bitbake-devel] [docs] [PATCH 1/2] documentation: update sources mirror URL
       [not found]       ` <16ABB308824CB520.3789@lists.yoctoproject.org>
@ 2021-10-07  9:04         ` Michael Opdenacker
  0 siblings, 0 replies; 9+ messages in thread
From: Michael Opdenacker @ 2021-10-07  9:04 UTC (permalink / raw)
  To: Richard Purdie, Jon Mason; +Cc: docs, bitbake-devel


On 10/7/21 10:42 AM, Michael Opdenacker wrote:
> Hi,
>
> On 10/7/21 12:17 AM, Richard Purdie wrote:
>> On Wed, 2021-10-06 at 15:53 -0400, Jon Mason wrote:
>>>> Thanks for the proposed update !
>>>>
>>>> On 10/6/21 12:45 AM, Jon Mason wrote:
>>>>> The URL for the sources mirror no longer works.  Update to the new
>>>>> location.
>>>>>
>>>>> Signed-off-by: Jon Mason <jdmason@kudzu.us>
>>>>> ---
>>>>>  documentation/ref-manual/faq.rst       | 15 ++++++++-------
>>>>>  documentation/ref-manual/variables.rst |  8 ++++----
>>>>>  2 files changed, 12 insertions(+), 11 deletions(-)
>>>>>
>>>>> diff --git a/documentation/ref-manual/faq.rst b/documentation/ref-manual/faq.rst
>>>>> index d3a603d4a438..f06abc84ee6f 100644
>>>>> --- a/documentation/ref-manual/faq.rst
>>>>> +++ b/documentation/ref-manual/faq.rst
>>>>> @@ -302,10 +302,10 @@ attempt before any others by adding something like the following to the
>>>>>  ``local.conf`` configuration file::
>>>>>
>>>>>     PREMIRRORS:prepend = "\
>>>>> -       git://.*/.* http://www.yoctoproject.org/sources/ \n \
>>>>> -       ftp://.*/.* http://www.yoctoproject.org/sources/ \n \
>>>>> -       http://.*/.* http://www.yoctoproject.org/sources/ \n \
>>>>> -       https://.*/.* http://www.yoctoproject.org/sources/ \n"
>>>>> +       git://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
>>>>> +       ftp://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
>>>>> +       http://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n \
>>>>> +       https://.*/.* http://downloads.yoctoproject.org/mirror/sources/ \n"
>>>> Hey, doesn't it sound weird to show an example that proposes to use http
>>>> when https is usually recommended?
>>>> What about proposing https://downloads.yoctoproject.org/mirror/sources/
>>>> instead?
>>>>
>>>> Or we don't really care because we use source checksums anyway?
>>>>
>>>> Just my two cents. I don't mind accepting the patch as is.
>>> A quick and dirty grep of the trees shows many occurrences of
>>> downloads.yoctoproject.org without https.  Perhaps it would be good to
>>> correct all of these in a follow-on patch.  I can do a quick patch of
>>> all of these and get that out in the next day or so.
>
>
> OK to do it as a follow-up patch. This way, I can already take yours (at
> least the ones for yocto-docs, which I maintain).


Merged your first patch into master-next. If you make a follow-up patch
to replace "http" by "https" (I could do that if you wish), that's
better if you base it on master-next.

Thanks again
Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2021-10-07  9:04 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-05 22:45 [PATCH 1/2] documentation: update sources mirror URL Jon Mason
2021-10-05 22:45 ` [PATCH 2/2] bitbake: bitbake-user-manual: " Jon Mason
2021-10-06 14:41 ` [docs] [PATCH 1/2] documentation: " Michael Opdenacker
2021-10-06 19:53   ` Jon Mason
2021-10-06 22:17     ` [bitbake-devel] " Richard Purdie
2021-10-07  0:48       ` Jon Mason
2021-10-07  8:14         ` Quentin Schulz
2021-10-07  8:42       ` Michael Opdenacker
     [not found]       ` <16ABB308824CB520.3789@lists.yoctoproject.org>
2021-10-07  9:04         ` Michael Opdenacker

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).