Bug 13314

Summary: Fixes to .mailmap for the 3.18 release
Product: Koha Reporter: Tomás Cohen Arazi <tomascohen>
Component: Developer documentationAssignee: Tomás Cohen Arazi <tomascohen>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: katrin.fischer, m.de.rooy, magnus, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13314: fixes to the .mailmap file
Bug 13314: Fridolin and Simith fixes to the .mailmap file
Bug 13314: Holger Meissner needs a mapping
Bug 13314: Fridolin and Simith fixes to the .mailmap file
Bug 13314: Holger Meissner needs a mapping
Bug 13314: [QA Follow-up]: Add another line for Fridolin to .mailmap
[PASSED QA] Bug 13314: Fridolin and Simith fixes to the .mailmap file
[PASSED QA] Bug 13314: Holger Meissner needs a mapping
[PASSED QA] Bug 13314: [QA Follow-up]: Add another line for Fridolin to .mailmap
Bug 13314: attribute Oslo Public Library
Bug 13314: OPL followup - Petter Goksoyr Asen
Bug 13314: OPL followup - Benjamin Rokseth
Bug 13314: attribute Oslo Public Library
[PASSED QA] Bug 13314: attribute Oslo Public Library
Bug 13314: (QA followup) fix for Maxime Beaulieu
Bug 13314: (QA followup) fix for Charles Farmer
Bug 13314: (QA followup) fix for inLibro
Bug 13314: (QA followup) fixes for CINECA
Bug 13314: Follow-up for m.de.rooy in .mailmap
[PASSED QA] Bug 13314: Follow-up for m.de.rooy in .mailmap

Description Tomás Cohen Arazi 2014-11-20 17:50:56 UTC
If you run:

git log --pretty=short v3.16.00..HEAD | git shortlog -s | sort -k3 -

you will notice 'simith' and two different Fridolin's. There's a need for a patch that adds the right mappings to .mailmap.
Comment 1 Tomás Cohen Arazi 2014-11-20 17:54:50 UTC Comment hidden (obsolete)
Comment 2 Tomás Cohen Arazi 2014-11-20 18:29:27 UTC Comment hidden (obsolete)
Comment 3 Tomás Cohen Arazi 2014-11-20 18:29:35 UTC Comment hidden (obsolete)
Comment 4 Marcel de Rooy 2014-11-23 11:28:30 UTC Comment hidden (obsolete)
Comment 5 Marcel de Rooy 2014-11-23 11:28:39 UTC Comment hidden (obsolete)
Comment 6 Marcel de Rooy 2014-11-23 11:28:45 UTC Comment hidden (obsolete)
Comment 7 Marcel de Rooy 2014-11-23 11:31:32 UTC
I am not sure if commit dd16887ee71e45fc28e21180bbd799d49d1326df did something to mailmap that somewhere else creates a problem? The git statistics on http://git.koha-community.org/stats/koha-master/authors.html do not seem to be updated anymore? Could be something else too?

The patches here are fine for git shortlog though.
Comment 8 Marcel de Rooy 2014-11-23 11:36:32 UTC
(In reply to M. de Rooy from comment #7)
> I am not sure if commit dd16887ee71e45fc28e21180bbd799d49d1326df did
> something to mailmap that somewhere else creates a problem? 

The second patch of this report restores the sort order (by moving the gonzaleze entry) of the third section. Maybe that is enough to resolve the problem mentioned?
Comment 9 Katrin Fischer 2014-11-23 12:08:12 UTC Comment hidden (obsolete)
Comment 10 Katrin Fischer 2014-11-23 12:08:16 UTC Comment hidden (obsolete)
Comment 11 Katrin Fischer 2014-11-23 12:08:18 UTC Comment hidden (obsolete)
Comment 12 Tomás Cohen Arazi 2014-11-23 12:41:12 UTC
> Tested by generating the release notes. I noticed that
> the 'company' list often has a domain name instead
> of a company name, maybe something for another patch.

I fix that on the release-tools scripts. That's why i asked for company names on the list.
Comment 13 Tomás Cohen Arazi 2014-11-23 13:23:22 UTC
Patches pushed to master.
Comment 14 Tomás Cohen Arazi 2014-11-24 13:11:15 UTC Comment hidden (obsolete)
Comment 15 Tomás Cohen Arazi 2014-11-24 13:33:59 UTC Comment hidden (obsolete)
Comment 16 Tomás Cohen Arazi 2014-11-24 13:57:37 UTC Comment hidden (obsolete)
Comment 17 Magnus Enger 2014-11-24 14:07:32 UTC Comment hidden (obsolete)
Comment 18 Katrin Fischer 2014-11-24 20:47:54 UTC Comment hidden (obsolete)
Comment 19 Tomás Cohen Arazi 2014-11-24 23:37:39 UTC Comment hidden (obsolete)
Comment 20 Tomás Cohen Arazi 2014-11-24 23:49:21 UTC Comment hidden (obsolete)
Comment 21 Tomás Cohen Arazi 2014-11-25 17:47:25 UTC Comment hidden (obsolete)
Comment 22 Tomás Cohen Arazi 2014-11-25 17:52:26 UTC Comment hidden (obsolete)
Comment 23 Marcel de Rooy 2014-11-26 12:46:01 UTC Comment hidden (obsolete)
Comment 24 Marcel de Rooy 2014-11-26 12:46:21 UTC
Another one for you, Tomas :)
Comment 25 Tomás Cohen Arazi 2014-11-26 14:13:45 UTC Comment hidden (obsolete)