Am 11.05.2015 um 21:08 schrieb Jim Fehlig: > The generated AUTHORS file contains many duplicates. If an author > has N commits, there will be N entries for the author in AUTHORS. > Check if an author already exists in the list before appending. > > Reported-by: Andreas Färber <afaerber(a)suse.de&gt; > Signed-off-by: Jim Fehlig <jfehlig(a)suse.com&gt; > --- > setup.py | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/setup.py b/setup.py > index 2571742..7a1964b 100755 > --- a/setup.py > +++ b/setup.py > @@ -176,7 +176,9 @@ class my_sdist(sdist): > f = os.popen("git log --pretty=format:'%aN <%aE>'") > authors = [] > for line in f: > - authors.append(" " + line.strip()) > + line = " " + line.strip() > + if line not in authors: > + authors.append(line) > > authors.sort(key=str.lower) > > Reviewed-by: Andreas Färber <afaerber(a)suse.de&gt;

On 11.05.2015 22:05, Jim Fehlig wrote: > Andreas Färber wrote: >> Am 11.05.2015 um 21:08 schrieb Jim Fehlig: >> >>> The generated AUTHORS file contains many duplicates. If an author >>> has N commits, there will be N entries for the author in AUTHORS. >>> Check if an author already exists in the list before appending. >>> >>> >> Reported-by: Andreas Färber <afaerber(a)suse.de&gt; >> >> >>> Signed-off-by: Jim Fehlig <jfehlig(a)suse.com&gt; >>> --- >>> setup.py | 4 +++- >>> 1 file changed, 3 insertions(+), 1 deletion(-) >>> >>> diff --git a/setup.py b/setup.py >>> index 2571742..7a1964b 100755 >>> --- a/setup.py >>> +++ b/setup.py >>> @@ -176,7 +176,9 @@ class my_sdist(sdist): >>> f = os.popen("git log --pretty=format:'%aN <%aE>'") >>> authors = [] >>> for line in f: >>> - authors.append(" " + line.strip()) >>> + line = " " + line.strip() >>> + if line not in authors: >>> + authors.append(line) >>> >>> authors.sort(key=str.lower) >>> >>> >> Reviewed-by: Andreas Färber <afaerber(a)suse.de&gt; >> > Thanks. I just noticed a .mailmap would remove three other duplicates, > where the author's name has a different spelling or the author's mail > address is different. I'll let other libvirt maintainers comment on > whether that is desirable. > > Regards, > Jim > Yeah, I think we should go with .mailmap (copied from libvirt.git).