bzexport gets wrong patch description when description contains "?|"

RESOLVED FIXED

Status

RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: bzbarsky, Assigned: sfink)

Tracking

Details

Attachments

(1 attachment)

I have a patch whose hg commit message is:

  Bug 742165.  isArrayBuffer should be true for |ArrayBuffer?| arguments.  r=khuey

When I uploaded it using vanilla "hg bzexport -r :khuey", I ended up with an attachment with this description "isArrayBuffer should be true for |ArrayBuffe arguments."

Note that the "?|" part went away.
(Assignee)

Comment 1

7 years ago
Actually, the "r?|" part went away. bzexport didn't want to tell people that you'd requested review from that "|" guy.
(Assignee)

Comment 2

7 years ago
Created attachment 614908 [details] [diff] [review]
reviewer parsing is too loose

Require a leading alphanumeric, at least. I probably ought to figure out what the valid set of characters is for the reviewer.
Attachment #614908 - Flags: review?(josh)

Updated

7 years ago
Attachment #614908 - Flags: review?(josh) → review+

Updated

7 years ago
Assignee: nobody → sphink
(Assignee)

Comment 3

7 years ago
http://hg.mozilla.org/users/tmielczarek_mozilla.com/bzexport/rev/a8ebbe6e3ddc
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Product: Other Applications → Developer Services
You need to log in before you can comment on or make changes to this bug.