diff tests/test-hook.t @ 24826:9b02b678888e stable

mq: avoid silent failure when single patch doesn't apply (issue4604) The error-handling here is quite byzantine. self._apply raises an AbortNoCleanup, but self.apply was swallowing the exception and returns 2. In self.push, we catch all exceptions.. and cleanup. We try to print a message to clean up.. but that relies on having a top-of-stack. Instead, we re-raise the abort in self.apply, and avoid cleanup on AbortNoCleanup in self.push by adding a trivial new except clause. We also modernize the now-visible abort message.
author Matt Mackall <mpm@selenic.com>
date Mon, 20 Apr 2015 18:13:44 -0500
parents 57f1dbc99631
children 2f2de8a7f97f
line wrap: on
line diff