Hi, As you have seen, we have author information in Kimchi source code to specially honour the writer of the original source files and the succeeding writers with substantial change. Unfortunately, I found the authors in some files were updated without any substantial change. One recent example about this is "src/kimchi/model/debugreports.py", the author was changed to "Aline Manera" and the original author was gone. And this change was introduced by "refactor model: Create a separated model for debug report resource" that got a new file "debugreports.py" split from another huge file without much change. I don't think this split was deserved to change the author in the new file. Then, how do we define what is "substantial change". I would say it is a new feature changing the file with more than 1/3 of the total lines of code. It is a not necessary a hard rule, but it is obvious that code split should not be deserved to have a new author for the file.

_______________________________________________ Kimchi-devel mailing list Kimchi-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/kimchi-devel

On 02/25/2014 05:51 AM, Shu Ming wrote: > One > recent example about this is "src/kimchi/model/debugreports.py", the > author was changed to "Aline Manera" and the original author was gone. Who is the original author of this specific example you mentioned? I think that if the original author is uncomfortable he/she can submit a patch fixing it.

On 02/25/2014 05:51 AM, Shu Ming wrote: > Hi, > > As you have seen, we have author information in Kimchi source code to > specially honour the writer of the original source files and the > succeeding writers with substantial change. Unfortunately, I found the > authors in some files were updated without any substantial change. One > recent example about this is "src/kimchi/model/debugreports.py", the > author was changed to "Aline Manera" and the original author was gone. > And this change was introduced by "refactor model: Create a separated > model for debug report resource" that got a new file "debugreports.py" > split from another huge file without much change. I don't think this > split was deserved to change the author in the new file. Then, how do we > define what is "substantial change". I would say it is a new feature > changing the file with more than 1/3 of the total lines of code. It is a > not necessary a hard rule, but it is obvious that code split should not > be deserved to have a new author for the file. Sorry, Ming, about that. I created the license header on model/__init__.py and used it in all new files I created for refactoring model task. My intention was not get credits from a work I didn't do. And if you see in the former model.py (commit a0a4dc0f949ed664076b5c47184d2b086d6de942) only Adam Litke is listed as the author so, from what you wrote above, I should keep him as the author for *all* files created during refactor model. **But** everyone knows everyone had contributed to model.py as well for any other file in Kimchi source repository. So I am wondering why to have the name listed there became so important right now. You don't need to have your name listed there to get credits or honor for a work you did. git log is there to show all the contributions, the same for github (https://github.com/kimchi-project/kimchi/graphs/contributors) About the issue you mentioned, I was more concerned about getting the refactor model done than check line by line to identify who had written it. We also have the AUTHORS file which list everyone as contributors to Kimchi. > _______________________________________________ > Kimchi-devel mailing list > Kimchi-devel(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/kimchi-devel > _______________________________________________ Kimchi-devel mailing list Kimchi-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/kimchi-devel