vcs: remove confusing and unnecessary local variable This removes the "username" local variable, substituting the equivalent "user.username". The function also has another, unrelated use of the same local variable name; this OTHER "username" variable is untouched. The relevant code is duplicated between Hg and Git... and thus also this change.