Re: [PATCH] doc: remove confusing command to send patch

2024-11-30 Thread Thomas Monjalon
10/10/2023 18:26, Thomas Monjalon: > In the contributor guide, it was said that no need to Cc maintainers > for new additions, probably for new directories not having a maintainer. > There is no harm, and it is a good habit, to always Cc maintainers. > > Remove this case as it can mislead to not C

Re: [PATCH] doc: remove confusing command to send patch

2024-10-07 Thread Stephen Hemminger
On Tue, 10 Oct 2023 18:26:35 +0200 Thomas Monjalon wrote: > In the contributor guide, it was said that no need to Cc maintainers > for new additions, probably for new directories not having a maintainer. > There is no harm, and it is a good habit, to always Cc maintainers. > > Remove this case a

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread Thomas Monjalon
11/10/2023 12:22, Ferruh Yigit: > On 10/11/2023 11:20 AM, Thomas Monjalon wrote: > > 11/10/2023 10:41, Ferruh Yigit: > >> On 10/11/2023 9:30 AM, Bruce Richardson wrote: > >>> On Wed, Oct 11, 2023 at 10:03:07AM +0200, Thomas Monjalon wrote: > 11/10/2023 09:30, David Marchand: > > On Tue, Oc

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread Thomas Monjalon
11/10/2023 17:52, Stephen Hemminger: > On Wed, 11 Oct 2023 09:30:23 +0100 > Bruce Richardson wrote: > > > > Or we can send to dev@dpdk.org, Cc maintainers. > > > This is what I do: > > > git send-email --to dev@dpdk.org --cc-cmd devtools/get-maintainer.sh > > > > > +1 for this, mainly on the b

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread Stephen Hemminger
On Wed, 11 Oct 2023 09:30:23 +0100 Bruce Richardson wrote: > > Or we can send to dev@dpdk.org, Cc maintainers. > > This is what I do: > > git send-email --to dev@dpdk.org --cc-cmd devtools/get-maintainer.sh > > > +1 for this, mainly on the basis of it being what I do too! :-) Ditto. Maybe wou

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread Ferruh Yigit
On 10/11/2023 11:20 AM, Thomas Monjalon wrote: > 11/10/2023 10:41, Ferruh Yigit: >> On 10/11/2023 9:30 AM, Bruce Richardson wrote: >>> On Wed, Oct 11, 2023 at 10:03:07AM +0200, Thomas Monjalon wrote: 11/10/2023 09:30, David Marchand: > On Tue, Oct 10, 2023 at 6:26 PM Thomas Monjalon

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread Thomas Monjalon
11/10/2023 10:41, Ferruh Yigit: > On 10/11/2023 9:30 AM, Bruce Richardson wrote: > > On Wed, Oct 11, 2023 at 10:03:07AM +0200, Thomas Monjalon wrote: > >> 11/10/2023 09:30, David Marchand: > >>> On Tue, Oct 10, 2023 at 6:26 PM Thomas Monjalon > >>> wrote: > > In the contributor guide, i

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread Ferruh Yigit
On 10/11/2023 9:30 AM, Bruce Richardson wrote: > On Wed, Oct 11, 2023 at 10:03:07AM +0200, Thomas Monjalon wrote: >> 11/10/2023 09:30, David Marchand: >>> On Tue, Oct 10, 2023 at 6:26 PM Thomas Monjalon wrote: In the contributor guide, it was said that no need to Cc maintainers for

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread Bruce Richardson
On Wed, Oct 11, 2023 at 10:03:07AM +0200, Thomas Monjalon wrote: > 11/10/2023 09:30, David Marchand: > > On Tue, Oct 10, 2023 at 6:26 PM Thomas Monjalon wrote: > > > > > > In the contributor guide, it was said that no need to Cc maintainers > > > for new additions, probably for new directories not

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread Thomas Monjalon
11/10/2023 09:30, David Marchand: > On Tue, Oct 10, 2023 at 6:26 PM Thomas Monjalon wrote: > > > > In the contributor guide, it was said that no need to Cc maintainers > > for new additions, probably for new directories not having a maintainer. > > There is no harm, and it is a good habit, to alwa

Re: [PATCH] doc: remove confusing command to send patch

2023-10-11 Thread David Marchand
On Tue, Oct 10, 2023 at 6:26 PM Thomas Monjalon wrote: > > In the contributor guide, it was said that no need to Cc maintainers > for new additions, probably for new directories not having a maintainer. > There is no harm, and it is a good habit, to always Cc maintainers. > > Remove this case as i

Re: [PATCH] doc: remove confusing command to send patch

2023-10-10 Thread fengchengwen
Acked-by: Chengwen Feng On 2023/10/11 0:26, Thomas Monjalon wrote: > In the contributor guide, it was said that no need to Cc maintainers > for new additions, probably for new directories not having a maintainer. > There is no harm, and it is a good habit, to always Cc maintainers. > > Remove th

[PATCH] doc: remove confusing command to send patch

2023-10-10 Thread Thomas Monjalon
In the contributor guide, it was said that no need to Cc maintainers for new additions, probably for new directories not having a maintainer. There is no harm, and it is a good habit, to always Cc maintainers. Remove this case as it can mislead to not Cc maintainers when needed. Signed-off-by: Th