Re: [RFC PATCH v2 02/10] MAINTAINERS: Introduce V: entry for tests

2023-12-06 Thread Nikolai Kondrashov
On 12/6/23 18:38, Joe Perches wrote: On Wed, 2023-12-06 at 18:23 +0200, Nikolai Kondrashov wrote: On 12/6/23 10:12, David Gow wrote: I'm pretty happy with this personally, though I definitely think we need the support for tests which aren't just executable scripts (e.g. the docs in patch 6).

Re: [RFC PATCH v2 02/10] MAINTAINERS: Introduce V: entry for tests

2023-12-06 Thread Joe Perches
On Wed, 2023-12-06 at 18:23 +0200, Nikolai Kondrashov wrote: > On 12/6/23 10:12, David Gow wrote: > > I'm pretty happy with this personally, though I definitely think we > > need the support for tests which aren't just executable scripts (e.g. > > the docs in patch 6). > > > > The

Re: [RFC PATCH v2 02/10] MAINTAINERS: Introduce V: entry for tests

2023-12-06 Thread Nikolai Kondrashov
On 12/6/23 10:12, David Gow wrote: I'm pretty happy with this personally, though I definitely think we need the support for tests which aren't just executable scripts (e.g. the docs in patch 6). The get_maintailer.pl bits, and hence the requirement to not include '@', feel a little bit 'off':

Re: [RFC PATCH v2 02/10] MAINTAINERS: Introduce V: entry for tests

2023-12-06 Thread Nikolai Kondrashov
On 12/5/23 20:58, Joe Perches wrote: On Tue, 2023-12-05 at 20:02 +0200, Nikolai Kondrashov wrote: Require the entry values to not contain the '@' character, so they could be distinguished from emails (always) output by get_maintainer.pl. Why is this useful? Why the need to distinguish? This

Re: [RFC PATCH v2 02/10] MAINTAINERS: Introduce V: entry for tests

2023-12-06 Thread David Gow
On Wed, 6 Dec 2023 at 02:45, Nikolai Kondrashov wrote: > > Introduce a new 'V:' ("Verify") entry to MAINTAINERS. The entry accepts > a test suite command which is proposed to be executed for each > contribution to the subsystem. > > Extend scripts/get_maintainer.pl to support retrieving the V:

Re: [RFC PATCH v2 02/10] MAINTAINERS: Introduce V: entry for tests

2023-12-05 Thread Joe Perches
On Tue, 2023-12-05 at 20:02 +0200, Nikolai Kondrashov wrote: > Require the entry values to not contain the '@' character, so they could > be distinguished from emails (always) output by get_maintainer.pl. Why is this useful? Why the need to distinguish?

[RFC PATCH v2 02/10] MAINTAINERS: Introduce V: entry for tests

2023-12-05 Thread Nikolai Kondrashov
Introduce a new 'V:' ("Verify") entry to MAINTAINERS. The entry accepts a test suite command which is proposed to be executed for each contribution to the subsystem. Extend scripts/get_maintainer.pl to support retrieving the V: entries when '--test' option is specified. Require the entry values