[PATCH] README: A couple of updates due to recent changes

[PATCH] README: A couple of updates due to recent changes

[Tanu Kaskinen]

Since there's now a separate mailing list for docs, I think the
yocto-docs tag doesn't need to be added to patches any more.

---

 README | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/README b/README
index 62c2a3ffb..6a7f3abe6 100644
--- a/README
+++ b/README
@@ -11,15 +11,13 @@ Where to Send Patches
 =====================
 
 When sending single patches, please using something like:
-'git send-email -1 --to docs@lists.yoctoproject.org --subject-prefix=yocto-docs][PATCH'
+'git send-email -1 --to docs@lists.yoctoproject.org
 
-These values can be set as defaults for this repository:
+The To header can be set as default for this repository:
 
 $ git config sendemail.to docs@lists.yoctoproject.org
-$ git config format.subjectPrefix yocto-docs][PATCH
 
 Now you can just do 'git send-email origin/master' to send all local patches.
 
 Git repository: http://git.yoctoproject.org/cgit/cgit.cgi/yocto-docs/
 Mailing list: docs@lists.yoctoproject.org
-Please cc: Scott Rifenbark <srifenbark@gmail.com> on changes.
-- 
2.20.1

Re: [docs] [PATCH] README: A couple of updates due to recent changes

[akuster]

[-- Attachment #1: Type: text/plain, Size: 1236 bytes --]



On 2/9/20 7:37 PM, Tanu Kaskinen wrote:
> Since there's now a separate mailing list for docs, I think the
> yocto-docs tag doesn't need to be added to patches any more.
>
> ---
>
>  README | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/README b/README
> index 62c2a3ffb..6a7f3abe6 100644
> --- a/README
> +++ b/README
> @@ -11,15 +11,13 @@ Where to Send Patches
>  =====================
>  
>  When sending single patches, please using something like:
> -'git send-email -1 --to docs@lists.yoctoproject.org --subject-prefix=yocto-docs][PATCH'
> +'git send-email -1 --to docs@lists.yoctoproject.org
We still need a subject
>  
> -These values can be set as defaults for this repository:
> +The To header can be set as default for this repository:
>  
>  $ git config sendemail.to docs@lists.yoctoproject.org
> -$ git config format.subjectPrefix yocto-docs][PATCH
We still need a subject
>  
>  Now you can just do 'git send-email origin/master' to send all local patches.
>  
>  Git repository: http://git.yoctoproject.org/cgit/cgit.cgi/yocto-docs/
>  Mailing list: docs@lists.yoctoproject.org
> -Please cc: Scott Rifenbark <srifenbark@gmail.com> on changes.
>
> 


[-- Attachment #2: Type: text/html, Size: 2472 bytes --]

Re: [docs] [PATCH] README: A couple of updates due to recent changes

[Peter Kjellerstedt]

[-- Attachment #1: Type: text/plain, Size: 1766 bytes --]

There will be a subject, but there is no reason to prefix it with “[yocto-docs]” anymore, is there?

//Peter

From: docs@lists.yoctoproject.org <docs@lists.yoctoproject.org> On Behalf Of akuster
Sent: den 10 februari 2020 18:01
To: docs@lists.yoctoproject.org
Subject: Re: [docs] [PATCH] README: A couple of updates due to recent changes


On 2/9/20 7:37 PM, Tanu Kaskinen wrote:

Since there's now a separate mailing list for docs, I think the

yocto-docs tag doesn't need to be added to patches any more.



---



 README | 6 ++----

 1 file changed, 2 insertions(+), 4 deletions(-)



diff --git a/README b/README

index 62c2a3ffb..6a7f3abe6 100644

--- a/README

+++ b/README

@@ -11,15 +11,13 @@ Where to Send Patches

 =====================



 When sending single patches, please using something like:

-'git send-email -1 --to docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org> --subject-prefix=yocto-docs][PATCH'

+'git send-email -1 --to docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org>
We still need a subject




-These values can be set as defaults for this repository:

+The To header can be set as default for this repository:



 $ git config sendemail.to docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org>

-$ git config format.subjectPrefix yocto-docs][PATCH
We still need a subject






 Now you can just do 'git send-email origin/master' to send all local patches.



 Git repository: http://git.yoctoproject.org/cgit/cgit.cgi/yocto-docs/

 Mailing list: docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org>

-Please cc: Scott Rifenbark <srifenbark@gmail.com><mailto:srifenbark@gmail.com> on changes.






[-- Attachment #2: Type: text/html, Size: 6250 bytes --]

Re: [docs] [PATCH] README: A couple of updates due to recent changes

[akuster]

[-- Attachment #1: Type: text/plain, Size: 2274 bytes --]



On 2/10/20 5:01 PM, Peter Kjellerstedt wrote:
>
> There will be a subject, but there is no reason to prefix it with
> “[yocto-docs]” anymore, is there?
>
nope.  All the other layers have --subject-prefix so it would be nice to
keep it consistent.

- armin
>
>  
>
> //Peter
>
>  
>
> *From:*docs@lists.yoctoproject.org <docs@lists.yoctoproject.org> *On
> Behalf Of *akuster
> *Sent:* den 10 februari 2020 18:01
> *To:* docs@lists.yoctoproject.org
> *Subject:* Re: [docs] [PATCH] README: A couple of updates due to
> recent changes
>
>  
>
>  
>
> On 2/9/20 7:37 PM, Tanu Kaskinen wrote:
>
>     Since there's now a separate mailing list for docs, I think the
>
>     yocto-docs tag doesn't need to be added to patches any more.
>
>      
>
>     ---
>
>      
>
>      README | 6 ++----
>
>      1 file changed, 2 insertions(+), 4 deletions(-)
>
>      
>
>     diff --git a/README b/README
>
>     index 62c2a3ffb..6a7f3abe6 100644
>
>     --- a/README
>
>     +++ b/README
>
>     @@ -11,15 +11,13 @@ Where to Send Patches
>
>      =====================
>
>      
>
>      When sending single patches, please using something like:
>
>     -'git send-email -1 --to docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org> --subject-prefix=yocto-docs][PATCH'
>
>     +'git send-email -1 --to docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org>
>
> We still need a subject
>
>      
>
>     -These values can be set as defaults for this repository:
>
>     +The To header can be set as default for this repository:
>
>      
>
>      $ git config sendemail.to docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org>
>
>     -$ git config format.subjectPrefix yocto-docs][PATCH
>
> We still need a subject
>
>      
>
>      
>
>      Now you can just do 'git send-email origin/master' to send all local patches.
>
>      
>
>      Git repository: http://git.yoctoproject.org/cgit/cgit.cgi/yocto-docs/
>
>      Mailing list: docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org>
>
>     -Please cc: Scott Rifenbark <srifenbark@gmail.com> <mailto:srifenbark@gmail.com> on changes.
>
>
>
>      
>
>  
>
>
> 


[-- Attachment #2: Type: text/html, Size: 8182 bytes --]

Re: [docs] [PATCH] README: A couple of updates due to recent changes

[Peter Kjellerstedt]

[-- Attachment #1: Type: text/plain, Size: 2755 bytes --]

Are you thinking of the layers in meta-openembedded? There it makes sense since many layers share one mailing list. However, here we are talking about the docs repository with its own mailing list. There should be no other patches sent to it than those that are intended for the docs repository, so there should be no need to add an extra [yocto-docs] prefix. And for the record, this matches the documentation for bitbake, which also has its own mailing list.

//Peter

From: docs@lists.yoctoproject.org <docs@lists.yoctoproject.org> On Behalf Of akuster
Sent: den 11 februari 2020 02:34
To: docs@lists.yoctoproject.org
Subject: Re: [docs] [PATCH] README: A couple of updates due to recent changes


On 2/10/20 5:01 PM, Peter Kjellerstedt wrote:
There will be a subject, but there is no reason to prefix it with “[yocto-docs]” anymore, is there?
nope.  All the other layers have --subject-prefix so it would be nice to keep it consistent.

- armin


//Peter

From: docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org> <docs@lists.yoctoproject.org><mailto:docs@lists.yoctoproject.org> On Behalf Of akuster
Sent: den 10 februari 2020 18:01
To: docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org>
Subject: Re: [docs] [PATCH] README: A couple of updates due to recent changes


On 2/9/20 7:37 PM, Tanu Kaskinen wrote:

Since there's now a separate mailing list for docs, I think the

yocto-docs tag doesn't need to be added to patches any more.



---



 README | 6 ++----

 1 file changed, 2 insertions(+), 4 deletions(-)



diff --git a/README b/README

index 62c2a3ffb..6a7f3abe6 100644

--- a/README

+++ b/README

@@ -11,15 +11,13 @@ Where to Send Patches

 =====================



 When sending single patches, please using something like:

-'git send-email -1 --to docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org> --subject-prefix=yocto-docs][PATCH'

+'git send-email -1 --to docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org>
We still need a subject





-These values can be set as defaults for this repository:

+The To header can be set as default for this repository:



 $ git config sendemail.to docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org>

-$ git config format.subjectPrefix yocto-docs][PATCH
We still need a subject







 Now you can just do 'git send-email origin/master' to send all local patches.



 Git repository: http://git.yoctoproject.org/cgit/cgit.cgi/yocto-docs/

 Mailing list: docs@lists.yoctoproject.org<mailto:docs@lists.yoctoproject.org>

-Please cc: Scott Rifenbark <srifenbark@gmail.com><mailto:srifenbark@gmail.com> on changes.












[-- Attachment #2: Type: text/html, Size: 8720 bytes --]

Re: [docs] [PATCH] README: A couple of updates due to recent changes

[akuster]

[-- Attachment #1: Type: text/plain, Size: 3725 bytes --]



On 2/10/20 5:57 PM, Peter Kjellerstedt wrote:
>
> Are you thinking of the layers in meta-openembedded? There it makes
> sense since many layers share one mailing list. However, here we are
> talking about the docs repository with its own mailing list. There
> should be no other patches sent to it than those that are intended for
> the docs repository, so there should be no need to add an extra
> [yocto-docs] prefix.
>
I agree. no need to include [yocto-docs] in the subject line.

I mean "git sent-email -1 -M --subject-prefix=PATCH --to
docs@lists.yoctoproject.org"

don't we want a subject? the subject part was removed from the README.

-- armin
>
> And for the record, this matches the documentation for bitbake, which
> also has its own mailing list.
>
>  
>
> //Peter
>
>  
>
> *From:*docs@lists.yoctoproject.org <docs@lists.yoctoproject.org> *On
> Behalf Of *akuster
> *Sent:* den 11 februari 2020 02:34
> *To:* docs@lists.yoctoproject.org
> *Subject:* Re: [docs] [PATCH] README: A couple of updates due to
> recent changes
>
>  
>
>  
>
> On 2/10/20 5:01 PM, Peter Kjellerstedt wrote:
>
>     There will be a subject, but there is no reason to prefix it with
>     “[yocto-docs]” anymore, is there?
>
> nope.  All the other layers have --subject-prefix so it would be nice
> to keep it consistent.
>
> - armin
>
>      
>
>     //Peter
>
>      
>
>     *From:*docs@lists.yoctoproject.org
>     <mailto:docs@lists.yoctoproject.org> <docs@lists.yoctoproject.org>
>     <mailto:docs@lists.yoctoproject.org> *On Behalf Of *akuster
>     *Sent:* den 10 februari 2020 18:01
>     *To:* docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org>
>     *Subject:* Re: [docs] [PATCH] README: A couple of updates due to
>     recent changes
>
>      
>
>      
>
>     On 2/9/20 7:37 PM, Tanu Kaskinen wrote:
>
>         Since there's now a separate mailing list for docs, I think the
>
>         yocto-docs tag doesn't need to be added to patches any more.
>
>          
>
>         ---
>
>          
>
>          README | 6 ++----
>
>          1 file changed, 2 insertions(+), 4 deletions(-)
>
>          
>
>         diff --git a/README b/README
>
>         index 62c2a3ffb..6a7f3abe6 100644
>
>         --- a/README
>
>         +++ b/README
>
>         @@ -11,15 +11,13 @@ Where to Send Patches
>
>          =====================
>
>          
>
>          When sending single patches, please using something like:
>
>         -'git send-email -1 --to docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org> --subject-prefix=yocto-docs][PATCH'
>
>         +'git send-email -1 --to docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org>
>
>     We still need a subject
>
>
>          
>
>         -These values can be set as defaults for this repository:
>
>         +The To header can be set as default for this repository:
>
>          
>
>          $ git config sendemail.to docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org>
>
>         -$ git config format.subjectPrefix yocto-docs][PATCH
>
>     We still need a subject
>
>
>          
>
>          
>
>          Now you can just do 'git send-email origin/master' to send all local patches.
>
>          
>
>          Git repository: http://git.yoctoproject.org/cgit/cgit.cgi/yocto-docs/
>
>          Mailing list: docs@lists.yoctoproject.org <mailto:docs@lists.yoctoproject.org>
>
>         -Please cc: Scott Rifenbark <srifenbark@gmail.com> <mailto:srifenbark@gmail.com> on changes.
>
>
>
>
>          
>
>      
>
>
>
>      
>
>  
>
>
> 


[-- Attachment #2: Type: text/html, Size: 12214 bytes --]

Re: [docs] [PATCH] README: A couple of updates due to recent changes

[Tanu Kaskinen]

On Mon, 2020-02-10 at 18:07 -0800, akuster wrote:
> 
> On 2/10/20 5:57 PM, Peter Kjellerstedt wrote:
> > Are you thinking of the layers in meta-openembedded? There it makes
> > sense since many layers share one mailing list. However, here we are
> > talking about the docs repository with its own mailing list. There
> > should be no other patches sent to it than those that are intended for
> > the docs repository, so there should be no need to add an extra
> > [yocto-docs] prefix.
> > 
> I agree. no need to include [yocto-docs] in the subject line.
> 
> I mean "git sent-email -1 -M --subject-prefix=PATCH --to
> docs@lists.yoctoproject.org"
> 
> don't we want a subject? the subject part was removed from the README.

If the --subject-prefix option is not set, then git send-email will by
default generate a subject containing a "[PATCH]" prefix plus the
commit title. This patch was submitted without the --subject-prefix
option, and as you can see, the subject is fine.

I forgot to add Signed-off-by to the patch, I'll resend it.

-- 
Tanu

https://www.patreon.com/tanuk
https://liberapay.com/tanuk