[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] scripts/add_maintainers.pl adding empty Cc: lines



Just had a chat with Lars on IRC, which might be of common
interest (and Lars asked me to post it to xen-devel):

(17:00:16) juergen_gross: lars_kurth: any idea why ./scripts/add_maintainers.pl would add a "Cc:" without a mail address to a patch? Happened e.g. in my series "[PATCH v2 0/9] xen: scheduler cleanups" (cover-letter, patches 1, 2, 7 and 9) (17:01:58) lars_kurth: juergen_gross: oh, an actual bug! Let me look at the code
(17:02:19) lars_kurth: juergen_gross:  is it missing some e-mails?
(17:02:34) juergen_gross: git send-email seems to remove those empty Cc: lines (17:02:53) juergen_gross: I'm not aware of a mail address missing. Let me double check (17:06:56) juergen_gross: lars_kurth: hmm, shouldn't the MAINTAINERS entry "L: DornerWorks Xen-Devel <xen-devel@xxxxxxxxxxxxxxx>" result in a Cc:?
(17:08:17) lars_kurth: Let me have a look
(17:13:16) juergen_gross: lars_kurth: at least the related file is touched exactly by the affected patches (and not by any not affected patch) (17:13:36) lars_kurth: Looking at the series the most likely cause of this is the L: entry - need to look at the code (17:15:21) lars_kurth: juergen_gross: it's also an odd one because it changes MAINTAINERS and renames a lot of files, which may be the cause for the empty spaces (17:15:52) juergen_gross: lars_kurth: in Linux MAINTAINERS all "L:" entries just have a mail address as first word after the "L:" (not "bla bla <addr@domain>")
(17:16:11) lars_kurth: Ah yes: let me look at that code
(17:21:29) lars_kurth: juergen_gross: I think that is in fact the issue
(17:27:16) lars_kurth: juergen_gross: I can't fix this with some debugging. Could you copy this conversation into a mail on xen-devel@ such that I remember
(17:27:43) lars_kurth: uergen_gross: with=without
(17:29:36) lars_kurth: juergen_gross: I think what happens is that get_maintainer.pl and add_maintainer.pl process these lines differently, but add_maintainer.pl also checks against output created from get_maintainer.pl (17:44:58) juergen_gross: lars_kurth: what about doing it the easy way? With a modifed MAINTAINERS file (using "L: xen-devel@xxxxxxxxxxxxxxx") everything is fine. I can send a patch in case you agree. (17:46:41) lars_kurth: juergen_gross: let's do that first, but I still would like to fix the underlying issue at some point - asking for you to send the IRC log, as I cleared my history by mistake (when I was typing a reply I slipped from shift to ctrl, which did it)


Juergen

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.