Closed
Bug 730200
Opened 12 years ago
Closed 12 years ago
bzexport crashes on patches with no description
Categories
(Developer Services :: Mercurial: bzexport, defect)
Developer Services
Mercurial: bzexport
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sfink, Assigned: sfink)
Details
Attachments
(1 file)
1.08 KB,
patch
|
jdm
:
review+
|
Details | Diff | Splinter Review |
It's a bad sign when nearly every time I run bzexport, I run into another problem. I should figure out how to set up a test suite. (I have one for part of the functionality, but it's too ugly to land.)
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #600312 -
Flags: review?(josh)
Comment 2•12 years ago
|
||
Comment on attachment 600312 [details] [diff] [review] imported patch empty-desc Review of attachment 600312 [details] [diff] [review]: ----------------------------------------------------------------- ::: bzexport.py @@ +952,5 @@ > if update_patch: > # Add "Bug nnnn - " to the beginning of the description > ph = mq.patchheader(q.join(rev), q.plainmode) > msg = ph.message > + if len(msg) == 0: Make this |if not msg|.
Attachment #600312 -
Flags: review?(josh) → review+
Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Other Applications → Developer Services
You need to log in
before you can comment on or make changes to this bug.
Description
•