On 07/28/2017 08:50 AM, Paolo Bonzini wrote:
On 28/07/2017 08:24, Fam Zheng wrote:
On Fri, 07/28 02:36, Philippe Mathieu-Daudé wrote:
diff --git a/MAINTAINERS b/MAINTAINERS
index 224890643d..7854792765 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1537,6 +1537,12 @@ Checkpatch
S: Odd Fixes
F: scripts/checkpatch.pl
+Coccinelle scripts
+R: Philippe Mathieu-Daudé <f4...@amsat.org>
+S: Orphan
"Orphan" sounds like we don't care much anymore but I suppose this is not the
case. But I don't have any better idea either.
M: Mail patches to: FullName <address@domain>
R: Designated reviewer: FullName <address@domain>
M: is mail of maintainer
R: is mail of reviewer
S: Status, one of the following:
Maintained: Someone actually looks after it.
Odd Fixes: It has a maintainer but they don't have time
to do much other than throw the odd patch in.
-> Orphan: No current maintainer
"Orphan" sounds the best fit... Do you want to raise it to "Odd Fixes"?
This implies having a maintainer... I don't feel confident enough to Ack
cocci scripts but an incorrect cocci script will not break QEMU so maybe
I cat take M: for Odd Fixes, and lowering it back to Orphan without
maintainer is 1 commit easy :)
Just don't add it to the file. It doesn't have a specific owner.
Paolo you mean don't add the "S: Status" to the MAINTAINERS file, or the
"F: *cocci*" entries?
Regards,
Phil.
Paolo
Fam
+F: scripts/cocci-macro-file.h
+F: scripts/coccinelle/
+
Migration
M: Juan Quintela <quint...@redhat.com>
M: Dr. David Alan Gilbert <dgilb...@redhat.com>
@@ -1970,4 +1976,3 @@ Build system architecture
M: Daniel P. Berrange <berra...@redhat.com>
S: Odd Fixes
F: docs/devel/build-system.txt