Re: [PATCH 08/12] remote.c: mark messages for translation

2 messages, 2 authors, 2018-10-29 · open the first message on its own page

Re: [PATCH 08/12] remote.c: mark messages for translation

From: Junio C Hamano <hidden>
Date: 2018-10-29 07:56:11

Nguyễn Thái Ngọc Duy  [off-list ref] writes:
quoted hunk
@@ -995,12 +995,12 @@ static int match_explicit_lhs(struct ref *src,
 		 * way to delete 'other' ref at the remote end.
 		 */
 		if (try_explicit_object_name(rs->src, match) < 0)
-			return error("src refspec %s does not match any.", rs->src);
+			return error(_("src refspec %s does not match any"), rs->src);
 		if (allocated_match)
 			*allocated_match = 1;
 		return 0;
 	default:
-		return error("src refspec %s matches more than one.", rs->src);
+		return error(_("src refspec %s matches more than one"), rs->src);
 	}
 }
These minor changes that are not accompanied by their own
justification mean that the patches in this series cannot blindly be
trusted, which in turn means that I won't have bandwidth to process
this series properly for now.

I also saw die() -> BUG() that was not highlighted in the proposed
log message; the single instance I happened to notice looked
sensible, but I am not sure about the others.

There are other series in flight that touch the same area of code
and in different ways, causing unnecessary conflicts, which does not
help us either X-<.

Re: [PATCH 08/12] remote.c: mark messages for translation

From: Duy Nguyen <hidden>
Date: 2018-10-29 16:16:58

On Mon, Oct 29, 2018 at 8:56 AM Junio C Hamano [off-list ref] wrote:
There are other series in flight that touch the same area of code
and in different ways, causing unnecessary conflicts, which does not
help us either X-<.
I will of course fix all other comments, but I can hold this off if
it's causing too much trouble. I'm in no hurry to get anything merged
fast.
-- 
Duy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help