[PATCH] stash: don't translate literal commands

Subsystems: the rest

STALE1903d

3 messages, 2 authors, 2021-05-18 · open the first message on its own page

[PATCH] stash: don't translate literal commands

From: Alex Henrie <hidden>
Date: 2021-05-16 21:58:29

Signed-off-by: Alex Henrie <redacted>
---
 builtin/stash.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/builtin/stash.c b/builtin/stash.c
index d68ed784d2..075bcf6ee5 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -26,7 +26,7 @@ static const char * const git_stash_usage[] = {
 	N_("git stash drop [-q|--quiet] [<stash>]"),
 	N_("git stash ( pop | apply ) [--index] [-q|--quiet] [<stash>]"),
 	N_("git stash branch <branchname> [<stash>]"),
-	N_("git stash clear"),
+	"git stash clear",
 	N_("git stash [push [-p|--patch] [-k|--[no-]keep-index] [-q|--quiet]\n"
 	   "          [-u|--include-untracked] [-a|--all] [-m|--message <message>]\n"
 	   "          [--pathspec-from-file=<file> [--pathspec-file-nul]]\n"
@@ -67,7 +67,7 @@ static const char * const git_stash_branch_usage[] = {
 };
 
 static const char * const git_stash_clear_usage[] = {
-	N_("git stash clear"),
+	"git stash clear",
 	NULL
 };
 
-- 
2.31.1

Re: [PATCH] stash: don't translate literal commands

From: Derrick Stolee <hidden>
Date: 2021-05-17 16:51:29

On 5/16/2021 5:57 PM, Alex Henrie wrote:
quoted hunk
Signed-off-by: Alex Henrie <redacted>
---
 builtin/stash.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/builtin/stash.c b/builtin/stash.c
index d68ed784d2..075bcf6ee5 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -26,7 +26,7 @@ static const char * const git_stash_usage[] = {
 	N_("git stash drop [-q|--quiet] [<stash>]"),
 	N_("git stash ( pop | apply ) [--index] [-q|--quiet] [<stash>]"),
 	N_("git stash branch <branchname> [<stash>]"),
-	N_("git stash clear"),
+	"git stash clear",
I immediately saw the commands above and wondered why you are
not removing them from being translated. My guess is that the
"<stash>" strings are the bits that are interesting to
translate. Perhaps add some description in your commit message
as to why you are drawing the line here?
 static const char * const git_stash_clear_usage[] = {
-	N_("git stash clear"),
+	"git stash clear",
 	NULL
 };
Are there other instances in builtin/*.c? If not, then it
would be nice to say so. Otherwise, it would be good to fix
them, while you are here.

Thanks,
-Stolee

Re: [PATCH] stash: don't translate literal commands

From: Alex Henrie <hidden>
Date: 2021-05-18 06:08:28

On Mon, May 17, 2021 at 10:48 AM Derrick Stolee [off-list ref] wrote:
On 5/16/2021 5:57 PM, Alex Henrie wrote:
quoted
 static const char * const git_stash_clear_usage[] = {
-     N_("git stash clear"),
+     "git stash clear",
      NULL
 };
Are there other instances in builtin/*.c? If not, then it
would be nice to say so. Otherwise, it would be good to fix
them, while you are here.
I reviewed all of the translatable strings and only found this problem
in graph.c and builtin/stash.c.

-Alex
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help