threads / patch / 15497

patchgit-gui: I18n fix sentence parts into full sentences for translation again.

Subject: [PATCH] git-gui: I18n fix sentence parts into full sentences for translation again.

## tl;dr

4 messages between Sep 12, 2008 and Sep 12, 2008. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Christian Stimming· Sep 12, 2008, 09:17 UTC · lore

For translations, it is almost always impossible to correctly translate parts of sentences in almost any other language. Hence, messages like this must be re-organized into full sentences.

Signed-off-by: Christian Stimming <stimming@tuhh.de>
---
Patch against origin/master of git-gui.git at repo.or.cz.
 lib/mergetool.tcl |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)
Show changes to lib/mergetool.tcl +6 −6
diff --git a/lib/mergetool.tcl b/lib/mergetool.tcl
index 79c58bc..d10cadd 100644
--- a/lib/mergetool.tcl
+++ b/lib/mergetool.tcl
@@ -5,18 +5,18 @@ proc merge_resolve_one {stage} {
 	global current_diff_path
 
 	switch -- $stage {
-		1 { set target [mc "the base version"] }
-		2 { set target [mc "this branch"] }
-		3 { set target [mc "the other branch"] }
+		1 { set targetquestion [mc "Force resolution to the base version?"] }
+		2 { set targetquestion [mc "Force resolution to this branch?"] }
+		3 { set targetquestion [mc "Force resolution to the other branch?"] }
 	}
 
-	set op_question [mc "Force resolution to %s?
-Note that the diff shows only conflicting changes.
+    set op_question [strcat $targetquestion "\n" \
+[mc "Note that the diff shows only conflicting changes.
 
 %s will be overwritten.
 
 This operation can be undone only by restarting the merge." \
-		$target [short_path $current_diff_path]]
+	 	$targetquestion [short_path $current_diff_path]]]
 
 	if {[ask_popup $op_question] eq {yes}} {
 		merge_load_stages $current_diff_path [list merge_force_stage $stage]
-- 
1.6.0.rc1.34.g0fe8c
Alexander Gavrilov· Sep 12, 2008, 10:23 UTC · re: Christian Stimming · lore

Re: [PATCH] git-gui: I18n fix sentence parts into full sentences for translation again.

On Fri, Sep 12, 2008 at 1:17 PM, Christian Stimming <stimming@tuhh.de> wrote:
>-       set op_question [mc "Force resolution to %s?
>-Note that the diff shows only conflicting changes.
>+    set op_question [strcat $targetquestion "\n" \
>+[mc "Note that the diff shows only conflicting changes.
...
> -               $target [short_path $current_diff_path]]
> +               $targetquestion [short_path $current_diff_path]]]
You should remove this $target completely -- there is no %s to match it anymore.
Alexander.
Christian Stimming· Sep 12, 2008, 10:32 UTC · re: Alexander Gavrilov · lore

Re: [PATCH] git-gui: I18n fix sentence parts into full sentences for translation again.

Am Freitag, 12. September 2008 12:23 schrieb Alexander Gavrilov:
> On Fri, Sep 12, 2008 at 1:17 PM, Christian Stimming <stimming@tuhh.de> 
wrote:
Show 12 quoted lines
> >-       set op_question [mc "Force resolution to %s?
> >-Note that the diff shows only conflicting changes.
> >+    set op_question [strcat $targetquestion "\n" \
> >+[mc "Note that the diff shows only conflicting changes.
>
> ...
>
> > -               $target [short_path $current_diff_path]]
> > +               $targetquestion [short_path $current_diff_path]]]
>
> You should remove this $target completely -- there is no %s to match it
> anymore.
Oh - right. Thanks for pointing this out. Please modify the patch accordingly.
Christian
Shawn O. Pearce· Sep 12, 2008, 15:39 UTC · re: Christian Stimming · lore

Re: [PATCH] git-gui: I18n fix sentence parts into full sentences for translation again.

Christian Stimming <stimming@tuhh.de> wrote:
Show 17 quoted lines
> Am Freitag, 12. September 2008 12:23 schrieb Alexander Gavrilov:
> > On Fri, Sep 12, 2008 at 1:17 PM, Christian Stimming <stimming@tuhh.de> 
> wrote:
> > >-       set op_question [mc "Force resolution to %s?
> > >-Note that the diff shows only conflicting changes.
> > >+    set op_question [strcat $targetquestion "\n" \
> > >+[mc "Note that the diff shows only conflicting changes.
> >
> > ...
> >
> > > -               $target [short_path $current_diff_path]]
> > > +               $targetquestion [short_path $current_diff_path]]]
> >
> > You should remove this $target completely -- there is no %s to match it
> > anymore.
> 
> Oh - right. Thanks for pointing this out. Please modify the patch accordingly.
Fixed and applied.  Thanks, both.
-- 
Shawn.

← back to recent threads