handle all mailmap with name and email address

[Bhuvan Arumugam]

The parse_mailmap() method handle following format:
    <email address hidden> <email address hidden>

It can't handle following formats:
  Foo <email address hidden> Bar <email address hidden>
  Foo ZZ <email address hidden> Bar YY <email address hidden>

All 3 formats are valid mailmap formats, as per this document.
  http://man.github.com/git/git-shortlog.html

Please fix it.

[Bhuvan Arumugam (bhuvan)]

Posted a patch for review in openstack-common project. Once it's merged, i'll propagate it to other projects.
  https://review.openstack.org/7150

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/7174

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/7175

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/7181

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/7174
Committed: http://github.com/openstack/openstack-common/commit/2e1371a13205cd494dc4b9ade79ae20b607256b1
Submitter: Jenkins
Branch: master

commit 2e1371a13205cd494dc4b9ade79ae20b607256b1
Author: Bhuvan Arumugam <email address hidden>
Date: Sun May 6 15:13:44 2012 -0700

    Tests to cover various mailmap formats.

    Bug: 994957

    Mailmap might include entries in different format:
       Full Name <email address hidden> Full Name <email address hidden>
       Firstname <email address hidden> Firstname <email address hidden>
       <email address hidden> <email address hidden>

    This commit adds a test case to parse these entries.

    * tests/unit/test_setup.py
      Remove unused imports.
      Import os and setup methods to verify.

      SetupTest: Define unique test case name.
      setUp(): New method to define temporary mailmap file.
      tearDown(): New method to remove the temporary mailmap file.
      test_mailmap_with_fullname(): New method to test mailmap entry
      with fullname.
      test_mailmap_with_firstname(): New method to test mailmap entry
      with first name only.
      test_mailmap_with_noname(): New method to test mailmap entry with
      just email addresses.

    Change-Id: Icc018d4b2692ce7a7de757f02d1dd49241c30a6c

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/7175
Committed: http://github.com/openstack/keystone/commit/fd7df12e3dba97760c1eb2a3d8bd782bbf633a24
Submitter: Jenkins
Branch: master

commit fd7df12e3dba97760c1eb2a3d8bd782bbf633a24
Author: Bhuvan Arumugam <email address hidden>
Date: Sun May 6 15:35:58 2012 -0700

    Truly handle mailmap entries for all combinations.

    Bug: 994957

    Mailmap might contain mapping with full name. This commit fixes
    it to handle these combinations:
      Foo ZZ <email address hidden> Bar YY <email address hidden>
      Foo <email address hidden> Bar <email address hidden>
      <email address hidden> <email address hidden>

    This is inline with mailmap convention described here:
      http://man.github.com/git/git-shortlog.html

    * keystone/openstack/common/setup.py
      parse_mailmap(): Pull out canonical and alias email addresses.

    * keystone/tests/test_setup.py
      New test script to cover various combinations in mailmap.

    Change-Id: I471172aa012f37e499e86ffeb74c30ebcdb89b00

[Sean Dague (sdague)]

Suggested importance: Low

There still needs to be a sync from openstack-common using update.py (as specified at http://wiki.openstack.org/CommonLibrary#Incubation)

This should be low hanging fruit to complete