martinvonz added a comment.

  In, @indygreg wrote:
  > Now I'm second guessing accepting/committing this...
  Some thoughts:
  - This is probably not very well tested code (because it's an error path)
  - As long as we grep for TypeError and add binascii.Error in all those 
places, we're probably fine
  - Extensions need to be fixed too
  I think I'd prefer to define our own node.bin() with the old contract (IOW to 
wrap it)

  rHG Mercurial


To: durin42, #hg-reviewers, indygreg
Cc: martinvonz, indygreg, mercurial-devel
Mercurial-devel mailing list

Reply via email to