<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 14-03-27 03:52 AM, Alan Coopersmith
      wrote:<br>
    </div>
    <blockquote cite="mid:5333D8A4.9030901@oracle.com" type="cite">
      <blockquote type="cite" style="color: #000000;">(-s serves the
        same function as the "sort -u", and the cut command
        <br>
        removes the commit counts that -s adds.)
        <br>
      </blockquote>
      <br>
      Ah, I'd looked at git shortlog, but missed the -s option and
      instead
      <br>
      went looking for format options to remove commit titles and found
      none.
      <br>
      <br>
      That works for me, and even takes care of the multiple email
      address
      <br>
      merges, though we could still use .mailmap to clean up a couple
      <br>
      inconsistently spelled names.  (Would be nice to have a global
      mailmap
      <br>
      across all Xorg/xcb repos, but I don't want to figure out the git
      <br>
      magic to make that work myself.)
      <br>
      <br>
      <div class="moz-txt-sig"><span class="moz-txt-tag"></span></div>
    </blockquote>
    Do you want output sorted by number of commits (which do not appear)
    or by authors name?<br>
    The latter would require <big><tt>'git shortlog -s | cut -f2- >
        AUTHORS</tt></big>'. Sorting by commit numbers will result in a
    file that is constantly (ok, often) changing. Reading it to find a
    name is more difficult.<br>
    <br>
    Should this be made available to all xorg modules in util/macros? I
    have prototyped it.<br>
    The makefile would look like in participating modules (under no
    obligation):<br>
    <blockquote><tt>MAINTAINERCLEANFILES = ChangeLog INSTALL AUTHORS</tt><tt><br>
      </tt><tt><br>
      </tt><tt>.PHONY: ChangeLog INSTALL AUTHORS</tt><tt><br>
      </tt><tt><br>
      </tt><tt>INSTALL:</tt><tt><br>
      </tt><tt>    $(INSTALL_CMD)</tt><tt><br>
      </tt><tt><br>
      </tt><tt>ChangeLog:</tt><tt><br>
      </tt><tt>    $(CHANGELOG_CMD)</tt><tt><br>
      </tt><tt><br>
      </tt><tt>AUTHORS:</tt><tt><br>
      </tt><tt>    $(AUTHORS_CMD)</tt><tt><br>
      </tt><tt><br>
      </tt><tt>dist-hook: ChangeLog INSTALL AUTHORS</tt><tt><br>
      </tt></blockquote>
    <br>
    <ol>
      <li>In any case I need an rb line for COPYING which I believe is
        correct.<br>
        <br>
      </li>
      <li>If you elect for an AUTHORS file in git, I'll amend patch #4
        with your choice of sorting<br>
        <br>
      </li>
      <li>If you elect for a util-macros implementation, I'll drop patch
        #4 and submit new patches after this series has been dealt with.<br>
      </li>
    </ol>
    Thanks for the review!<br>
    <br>
    <br>
  </body>
</html>