Hi Lucas, I'm bothering you with technicalities again unfortunately:
1. I'm not sure if it's customary to add @author tags to other modules except the main kvm.py one, but in case it is, here's what I think should be there (reflects the state of kvm_runtest_2 until before the merge with Autotest): - For kvm_vm.py, kvm_utils.py, kvm_tests.py: Uri Lublin ([email protected]) Dror Russo ([email protected]) Michael Goldish ([email protected]) - For make_html_report.py: Dror Russo ([email protected]) (may already be there) - For scan_results.py, calc_md5sum_1m.py, fix_cdkeys.py: Uri Lublin ([email protected]) - For kvm_install.py: Uri Lublin ([email protected]) Dror Russo ([email protected]) Michael Goldish ([email protected]) David Huff ([email protected]) Mike Burns ([email protected]) - For kvm_guest_wizard.py: Alexey Eromenko ([email protected]) Uri Lublin ([email protected]) Dror Russo ([email protected]) Michael Goldish ([email protected]) - For kvm_config.py, kvm_preprocessing.py, stepeditor.py, stepmaker.py, ppm_utils.py: Michael Goldish ([email protected]) I hope I didn't forget anything (or anyone). I also hope new authors will be added as more people contribute to the project. 2. Now that many files were modified due to the merge I'm not sure the above list is still valid -- you should probably be given credit, but I'm not sure exactly what files you modified (all of them?). 3. Should we use @credits tags as well, or something similar? Thanks, Michael ----- Original Message ----- From: "Lucas Meneghel Rodrigues" <[email protected]> To: [email protected] Cc: "KVM mailing list" <[email protected]>, "Uri Lublin" <[email protected]> Sent: Monday, June 1, 2009 6:46:34 PM (GMT+0200) Auto-Detected Subject: Re: [PATCH] Fixing kvm test authorship issue Ok, patch applied. Thanks for pointing this out, Uri. On Mon, 2009-06-01 at 12:41 -0300, Lucas Meneghel Rodrigues wrote: > When preparing the kvm test for inclusion on autotest, I did > forget to add the name of Dror Russo ([email protected]) as > one of the rightful authors of the test. This patch includes > his name on the docstrings of the test. > --- > client/tests/kvm/control | 1 + > client/tests/kvm/kvm.py | 1 + > client/tests/kvm/make_html_report.py | 1 + > 3 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/client/tests/kvm/control b/client/tests/kvm/control > index f4b2113..eb38686 100644 > --- a/client/tests/kvm/control > +++ b/client/tests/kvm/control > @@ -1,5 +1,6 @@ > AUTHOR = """ > [email protected] (Uri Lublin) > [email protected] (Dror Russo) > [email protected] (Michael Goldish) > [email protected] (David Huff) > [email protected] (Alexey Eromenko) > diff --git a/client/tests/kvm/kvm.py b/client/tests/kvm/kvm.py > index 91ec89a..1b9013c 100644 > --- a/client/tests/kvm/kvm.py > +++ b/client/tests/kvm/kvm.py > @@ -17,6 +17,7 @@ class kvm(test.test): > > @copyright: Red Hat 2008-2009 > @author: Uri Lublin ([email protected]) > + @author: Dror Russo ([email protected]) > @author: Michael Goldish ([email protected]) > @author: David Huff ([email protected]) > @author: Alexey Eromenko ([email protected]) > diff --git a/client/tests/kvm/make_html_report.py > b/client/tests/kvm/make_html_report.py > index 988b2f3..6aed39e 100755 > --- a/client/tests/kvm/make_html_report.py > +++ b/client/tests/kvm/make_html_report.py > @@ -4,6 +4,7 @@ Script used to parse the test results and generate an HTML > report. > > @copyright: (c)2005-2007 Matt Kruse (javascripttoolbox.com) > @copyright: Red Hat 2008-2009 > +...@author: Dror Russo ([email protected]) > """ > > import os, sys, re, getopt, time, datetime, commands -- Lucas Meneghel Rodrigues Software Engineer (QE) Red Hat - Emerging Technologies -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
