[PATCH/RFC v2] git-p4: add option to preserve user names

Patches from git passed into p4 end up with the committer being identified as the person who ran git-p4.

This patch adds an option --preserve-user. When enabled, git-p4 will modify the changelist after it has been submitted ("p4 change -f") and set the username to the one matching the git author.

If the person running git-p4 does not have sufficient permissions, git-p4 will refuse to run (detected using "p4 protects"). It's possible that complicated permissions setups might confuse git-p4 - it just looks to see if the user has admin or super on the repo. In theory they might have permissions in some parts and not in others.

If there are commits with authors who do not have p4 accounts, then git-p4 will refuse to run unless git-p4.allowMissingP4Users is true, in which case it falls back to the standard behaviour for those commits.

The code has to get the p4 changelist number. The way it does this is by simply doing 'p4 changes -c <client>', which means if someone else is using the same clientspec at the same time, there is a potential race hazard. The alternative is to rewrite the submit logic to submit a properly marshalled template, which felt a bit too intrusive.

I've hoisted the p4 user name cache to a separate class, since it gets used in a couple of different places now.

I've added an option git-p4.skipSubmitModTimeCheck so that I can write a test case without having to jump through hoops with the editor.

Luke Diamand (1):
  git-p4: add option to preserve user names

 contrib/fast-import/git-p4     |  179 +++++++++++++++++++++++++++++++---------
 contrib/fast-import/git-p4.txt |   29 +++++++
 t/t9800-git-p4.sh              |   84 +++++++++++++++++++
 3 files changed, 254 insertions(+), 38 deletions(-)