diff mbox

[slof] README: Update with patch submittion note

Message ID 1436155750-16815-1-git-send-email-aik@ozlabs.ru (mailing list archive)
State Not Applicable
Headers show

Commit Message

Alexey Kardashevskiy July 6, 2015, 4:09 a.m. UTC
This adds a maillist and emails to be used when posting a patch for SLOF.

Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Thomas Huth <thuth@redhat.com>
Cc: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
---

I was asked to describe somewhere how to post new patches so this is the first
attempt. Please fix my english and add/remove personal emails and any other
comments are also welcome. Thanks!

---
 README | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

Nikunj A Dadhania July 6, 2015, 5:03 a.m. UTC | #1
Alexey Kardashevskiy <aik@ozlabs.ru> writes:

> This adds a maillist and emails to be used when posting a patch for SLOF.
>
> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
> ---
>
> I was asked to describe somewhere how to post new patches so this is the first
> attempt. Please fix my english and add/remove personal emails and any other
> comments are also welcome. Thanks!
>
> ---
>  README | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/README b/README
> index 58e9294..b81fd08 100644
> --- a/README
> +++ b/README
> @@ -11,6 +11,7 @@ Index
>  2.2 Overview of the source code
>  2.4 Extending the Forth engine
>  3.0 Limitations
> +4.0 Submitting patches
>
>
>  1.0 Introduction to Slimline Open Firmware
> @@ -236,6 +237,15 @@ To add primitives:
>   On a JS21 all memory configurations should work.
>
>
> +4.0 Submitting patches
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> +
> +Patches for SLOF should be made against https://github.com/aik/SLOF,
> +the master branch and posted to linuxppc-dev@ozlabs.org.

> It is recommended also cc: to

It is recommended to cc:

> + aik@ozlabs.ru, nikunj@linux.vnet.ibm.com,
> + thuth@redhat.com.
> +
> +
>  Documentation
>  +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>
> -- 
> 2.4.0.rc3.8.gfb3e7d5
Thomas Huth July 6, 2015, 7:22 a.m. UTC | #2
On Mon,  6 Jul 2015 14:09:10 +1000
Alexey Kardashevskiy <aik@ozlabs.ru> wrote:

> This adds a maillist and emails to be used when posting a patch for SLOF.
> 
> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
> ---
> 
> I was asked to describe somewhere how to post new patches so this is the first
> attempt. Please fix my english and add/remove personal emails and any other
> comments are also welcome. Thanks!

I think you should add least add a sentence that the subject should
contain "[SLOF PATCH...]", so that people on the list can see at a
first glance that the patch is about SLOF.

And IMHO it would be really better to finally have a dedicated mailing
list for SLOF - even if it's low traffic only. It would help filtering
your mails and maybe also attract people who do not want to receive the
whole other traffic from the linux-ppc mailing list.

 Thomas
Alexey Kardashevskiy July 6, 2015, 8:02 a.m. UTC | #3
On 07/06/2015 05:22 PM, Thomas Huth wrote:
> On Mon,  6 Jul 2015 14:09:10 +1000
> Alexey Kardashevskiy <aik@ozlabs.ru> wrote:
>
>> This adds a maillist and emails to be used when posting a patch for SLOF.
>>
>> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>> Cc: Thomas Huth <thuth@redhat.com>
>> Cc: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
>> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
>> ---
>>
>> I was asked to describe somewhere how to post new patches so this is the first
>> attempt. Please fix my english and add/remove personal emails and any other
>> comments are also welcome. Thanks!
>
> I think you should add least add a sentence that the subject should
> contain "[SLOF PATCH...]", so that people on the list can see at a
> first glance that the patch is about SLOF.

===
Patches for SLOF should be made against https://github.com/aik/SLOF,
the master branch and posted to slof@ozlabs.org.
It is recommended cc: to aik@ozlabs.ru, nikunj@linux.vnet.ibm.com,
thuth@redhat.com.
===

> And IMHO it would be really better to finally have a dedicated mailing
> list for SLOF - even if it's low traffic only. It would help filtering
> your mails and maybe also attract people who do not want to receive the
> whole other traffic from the linux-ppc mailing list.

slof@lists.ozlabs.org it is then, I changed the message above.
Thomas Huth July 6, 2015, 8:14 a.m. UTC | #4
On Mon, 6 Jul 2015 18:02:00 +1000
Alexey Kardashevskiy <aik@ozlabs.ru> wrote:

> On 07/06/2015 05:22 PM, Thomas Huth wrote:
> > On Mon,  6 Jul 2015 14:09:10 +1000
> > Alexey Kardashevskiy <aik@ozlabs.ru> wrote:
> >
> >> This adds a maillist and emails to be used when posting a patch for SLOF.
...
> >> I was asked to describe somewhere how to post new patches so this is the first
> >> attempt. Please fix my english and add/remove personal emails and any other
> >> comments are also welcome. Thanks!
> >
> > I think you should add least add a sentence that the subject should
> > contain "[SLOF PATCH...]", so that people on the list can see at a
> > first glance that the patch is about SLOF.
> 
> Patches for SLOF should be made against https://github.com/aik/SLOF,
> the master branch and posted to slof@ozlabs.org.

Great, thanks for creating the list!

> It is recommended cc: to aik@ozlabs.ru, nikunj@linux.vnet.ibm.com,
> thuth@redhat.com.

Since that list is likely low-traffic only, I think I will have a look
at each mail I get on that list, so you can at least remove me from the
CC: list here.

 Thomas
Nikunj A Dadhania July 6, 2015, 8:29 a.m. UTC | #5
Thomas Huth <thuth@redhat.com> writes:

> On Mon, 6 Jul 2015 18:02:00 +1000
> Alexey Kardashevskiy <aik@ozlabs.ru> wrote:
>
>> On 07/06/2015 05:22 PM, Thomas Huth wrote:
>> > On Mon,  6 Jul 2015 14:09:10 +1000
>> > Alexey Kardashevskiy <aik@ozlabs.ru> wrote:
>> >
>> >> This adds a maillist and emails to be used when posting a patch for SLOF.
> ...
>> >> I was asked to describe somewhere how to post new patches so this is the first
>> >> attempt. Please fix my english and add/remove personal emails and any other
>> >> comments are also welcome. Thanks!
>> >
>> > I think you should add least add a sentence that the subject should
>> > contain "[SLOF PATCH...]", so that people on the list can see at a
>> > first glance that the patch is about SLOF.
>> 
>> Patches for SLOF should be made against https://github.com/aik/SLOF,
>> the master branch and posted to slof@ozlabs.org.

slof@lists.ozlabs.org ?

>
> Great, thanks for creating the list!
>
>> It is recommended cc: to aik@ozlabs.ru, nikunj@linux.vnet.ibm.com,
>> thuth@redhat.com.
>
> Since that list is likely low-traffic only, I think I will have a look
> at each mail I get on that list, so you can at least remove me from the
> CC: list here.

Ditto, no need to CC me as well.

Regards
Nikunj
diff mbox

Patch

diff --git a/README b/README
index 58e9294..b81fd08 100644
--- a/README
+++ b/README
@@ -11,6 +11,7 @@  Index
 2.2 Overview of the source code
 2.4 Extending the Forth engine
 3.0 Limitations
+4.0 Submitting patches
 
 
 1.0 Introduction to Slimline Open Firmware
@@ -236,6 +237,15 @@  To add primitives:
  On a JS21 all memory configurations should work.
 
 
+4.0 Submitting patches
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+
+Patches for SLOF should be made against https://github.com/aik/SLOF,
+the master branch and posted to linuxppc-dev@ozlabs.org. It is
+recommended also cc: to aik@ozlabs.ru, nikunj@linux.vnet.ibm.com,
+thuth@redhat.com.
+
+
 Documentation
 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++