[PATCH 1/1] domain_conf.c: skip checking ZPCI address is incomplete if not present

Daniel Henrique Barboza danielhb413 at gmail.com
Mon Jun 29 10:47:54 UTC 2020



On 6/29/20 5:44 AM, Andrea Bolognani wrote:
> On Sun, 2020-06-28 at 09:25 -0300, Daniel Henrique Barboza wrote:
>> On 6/27/20 10:32 AM, Andrea Bolognani wrote:
>>> In the future, please don't include CC tags in your commits: removing
>>> them results in extra work when picking up a patch, and it's also
>>> generally not considered very polite to CC individual developers.
>>> Everyone is subscribed to the list anyway :)
>>
>> I screwed up with the CC: tags in the commit msg. I usually do it via
>> "--cc" in git-sendpatch in these cases.
>>
>> As for not being polite, I CC'ed the people that was related with the
>> commit that I was fixing (signed-off/reviewed-by). Perhaps I should
>> have CC'ed just the author instead .... assuming that CC'ing the author
>> of the commit I'm amending is OK here in Libvirt, of course. At least
>> in the QEMU mailing list it's not just OK, but encouraged to CC the
>> author of the commit you're fixing to make the person aware.
> 
> QEMU and libvirt are different projects, which follow different
> conventions in many areas such as coding style, merge workflow, and
> mailing list usage.
> 
> For libvirt specifically, our documentation[1] states
> 
>    As a rule, patches should be sent to the mailing list only: all
>    developers are subscribed to libvir-list and read it regularly, so
>    **please don't CC individual developers** unless they've explicitly
>    asked you to.


Got it. I suppose this rule will stay the same after we move to Gitlab,
which makes me a bit nervous. Today I can ask people to put me in the CC
if they're fixing/reverting a commit I've authored, but I don't know if
this is possible at all with pull requests.


Thanks,

DHB


> 
> 
> [1] https://libvirt.org/submitting-patches.html
> 




More information about the libvir-list mailing list