Closed Bug 742180 Opened 11 years ago Closed 11 years ago

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

Categories

(Developer Services :: Mercurial: bzexport, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: sfink)

Details

Attachments

(1 file)

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.
Actually, the "r?|" part went away. bzexport didn't want to tell people that you'd requested review from that "|" guy.
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)
Attachment #614908 - Flags: review?(josh) → review+
Assignee: nobody → sphink
http://hg.mozilla.org/users/tmielczarek_mozilla.com/bzexport/rev/a8ebbe6e3ddc
Status: NEW → RESOLVED
Closed: 11 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.