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


Reply via email to