1469966 - [bzexport] TypeError assert with hg 4.3 from ui.edit action parameter

Status: ASSIGNED

(Developer Services :: Mercurial: bzexport, enhancement)

Description

[Steve Fink [:sfink] [:s:]]

Sorry, I know I should be trying to use phabricator for review requests, but v-c-t is the last place I'm using mq (mostly because I want to keep it working for other people), and I don't wnat to figure out how mq and phabricator interact because I just don't care -- I'm hoping the remaining mq users will finally give it up during the phabricator switchover.

Anyway, njn just set up a new Linux box and it had hg 4.3, so it seems worthwhile to support it for now.

Comment 1

[Steve Fink [:sfink] [:s:]]

Attachment: [bzexport] hg 4.3 compatibility wrt ui.edit action parameter

I imagine there's a cleaner way to do this.

Comment 2

[Gregory Szorc [:gps]]

Comment on attachment 8986550 [details] [diff] [review]
[bzexport] hg 4.3 compatibility wrt ui.edit action parameter

Review of attachment 8986550 [details] [diff] [review]:
-----------------------------------------------------------------

There may be a cleaner way to do it. But bzexport feels like dead code walking with the forced transition to Phabricator. So in my head the barrier to changing it is low.

I'll land this for you.