Closed Bug 1051804 Opened 7 years ago Closed 7 years ago

Change user to emailuser in hg annotate web interface

Categories

(Developer Services :: Mercurial: hg.mozilla.org, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: nONoNonO, Assigned: emorley)

References

Details

Attachments

(1 file)

The current hgweb interface shows my contributions as o. My username is o.e.ekker@gmail.com, which gets abbreviated to o. If you display emailuser instead of user, o.e.ekker would be shown.
See also http://bz.selenic.com/show_bug.cgi?id=2276 for more info.
An example of current annotate can be seen here:
http://hg.mozilla.org/releases/l10n/mozilla-aurora/nl/annotate/087b0af1cc16/chat/conversations.properties#l42
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 7 → All
Hardware: x86 → All
Unfortunately it would seem that the 'emailuser' filter requires Hg v2.2 or higher:
http://www.selenic.com/hg/rev/e5788269741a
Also search for "templates/filters: extracting the user portion of an email address (BC)" on http://mercurial.selenic.com/wiki/WhatsNew

As far as I can tell we're on 2.0.2 on hg.mozilla.org at present (bug 623505), and bug 945383 is where further upgrade discussions have taken place.
Blocks: 945383
Assignee: nobody → emorley
Status: NEW → ASSIGNED
Bug 945383 comment 4 says we're running 2.5.4, so this should work fine :-)
No longer blocks: 945383
Comment on attachment 8471469 [details] [diff] [review]
Use |emailuser| rather than |user| for the annotate author

For legacy reasons, the 'user' filter incorrectly turns email addresses of form
'a.b.surname@domain.org' into 'a'. Using 'emailuser' instead gives the entire
part before the @, ie 'a.b.surname' - and so this patch means that on the hgweb
annotate pages the author username shown for each line is less ambiguous.

Requires Hg v2.2 or higher; hg.m.o is running 2.5.4.

See also: http://bz.selenic.com/show_bug.cgi?id=2276
Attachment #8471469 - Flags: review?(ted)
Attachment #8471469 - Flags: review?(ted) → review+
Comment on attachment 8471469 [details] [diff] [review]
Use |emailuser| rather than |user| for the annotate author

Thank you :-)

https://hg.mozilla.org/hgcustom/hg_templates/rev/781a6d82af8c
Attachment #8471469 - Flags: checked-in+
Depends on: 1052570
In production; seems to work :-)
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Thank you for this quick change! I'm sure glad you don;t have change advisory boards and releases and thing like that for changes like this
Status: RESOLVED → VERIFIED
Product: Release Engineering → Developer Services
See Also: → 1262360
You need to log in before you can comment on or make changes to this bug.