<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">
<div class="">I was busy on other things this week, sorry for the late reply</div>
- Agreed for the commend on the third patch.
<div class="">- For the second patch, only one string is localized in crecord.py, localizing the string could be done in a future series, I suggest that we keep it as is for now</div>
<div class="">- I will send a V2</div>
<div class=""><br class="">
</div>
<div class="">Thanks,</div>
<div class=""><br class="">
</div>
<div class="">Laurent</div>
<div class=""><br class="">
<div>
<blockquote type="cite" class="">
<div class="">On Jun 10, 2015, at 7:37 AM, Augie Fackler <<a href="mailto:raf@durin42.com" class="">raf@durin42.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">On
 Mon, Jun 08, 2015 at 10:50:58AM -0400, Augie Fackler wrote:</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
On Fri, Jun 05, 2015 at 02:03:33PM -0700, Laurent Charignon wrote:<br class="">
<blockquote type="cite" class=""># HG changeset patch<br class="">
# User Laurent Charignon <<a href="mailto:lcharignon@fb.com" class="">lcharignon@fb.com</a>><br class="">
# Date 1433536874 25200<br class="">
#      Fri Jun 05 13:41:14 2015 -0700<br class="">
# Node ID 55d45b5b1cb25d431f4771cd42e456b76c904b36<br class="">
# Parent  82711472302682face294f925835780ab9f710a8<br class="">
crecord: exit edition of hunk with non-zero status does not interrupt session<br class="">
<br class="">
Before this patch, when editing a hunk, if the editor was exiting with a non-<br class="">
zero status the crecord session would be aborted. This patch makes crecord<br class="">
handle the failure and goes back to the state before the failed edition.<br class="">
We use the new error reporting mechanism introduced in the previous patch to<br class="">
notify the user of the issue.<br class="">
<br class="">
diff --git a/mercurial/crecord.py b/mercurial/crecord.py<br class="">
--- a/mercurial/crecord.py<br class="">
+++ b/mercurial/crecord.py<br class="">
@@ -1468,9 +1468,11 @@<br class="">
                f.close()<br class="">
                # start the editor and wait for it to complete<br class="">
                editor = self.ui.geteditor()<br class="">
-                self.ui.system("%s \"%s\"" % (editor, patchfn),<br class="">
-                          environ={'hguser': self.ui.username()},<br class="">
-                          onerr=util.Abort, errprefix=_("edit failed"))<br class="">
+                ret = self.ui.system("%s \"%s\"" % (editor, patchfn),<br class="">
+                          environ={'hguser': self.ui.username()})<br class="">
+                if ret != 0:<br class="">
+                    self.errorstr = "Editor exited with status %d" % ret<br class="">
</blockquote>
<br class="">
This looks suspiciously like something that should get localized?<br class="">
</blockquote>
<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Friendly
 ping on this series - I liked where it was going, and once we</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">work
 out my confusion on patches 2 and 3 it'll be ready to</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">land.
 Thanks!</span></div>
</blockquote>
</div>
<br class="">
</div>
</body>
</html>