All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] remove code that's dead post-rebase --preserve-merges removal
@ 2021-10-21 18:37 Ævar Arnfjörð Bjarmason
  2021-10-21 18:37 ` [PATCH 1/2] git-sh-i18n: remove unused eval_ngettext() Ævar Arnfjörð Bjarmason
  2021-10-21 18:37 ` [PATCH 2/2] git-sh-setup: remove messaging supporting --preserve-merges Ævar Arnfjörð Bjarmason
  0 siblings, 2 replies; 3+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-21 18:37 UTC (permalink / raw)
  To: git
  Cc: Junio C Hamano, Johannes Schindelin,
	Ævar Arnfjörð Bjarmason

A small follow-up to the now-landed "git rebase --preserve-merges"
topic.

Ævar Arnfjörð Bjarmason (2):
  git-sh-i18n: remove unused eval_ngettext()
  git-sh-setup: remove messaging supporting --preserve-merges

 git-sh-i18n.sh  | 12 ------------
 git-sh-setup.sh | 12 +-----------
 2 files changed, 1 insertion(+), 23 deletions(-)

-- 
2.33.1.1494.g88b39a443e1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 1/2] git-sh-i18n: remove unused eval_ngettext()
  2021-10-21 18:37 [PATCH 0/2] remove code that's dead post-rebase --preserve-merges removal Ævar Arnfjörð Bjarmason
@ 2021-10-21 18:37 ` Ævar Arnfjörð Bjarmason
  2021-10-21 18:37 ` [PATCH 2/2] git-sh-setup: remove messaging supporting --preserve-merges Ævar Arnfjörð Bjarmason
  1 sibling, 0 replies; 3+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-21 18:37 UTC (permalink / raw)
  To: git
  Cc: Junio C Hamano, Johannes Schindelin,
	Ævar Arnfjörð Bjarmason

The "eval_ngettext()" function has been orphaned since its last user
was removed in a74b35081c5 (rebase: drop support for
`--preserve-merges`, 2021-09-07).

See b8fc9e43a7d (i18n: rebase-interactive: mark here-doc strings for
translation, 2016-06-17) for the commit that added these
eval_ngettext() wrappers.

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
 git-sh-i18n.sh | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/git-sh-i18n.sh b/git-sh-i18n.sh
index e3d9f4836db..a15c0620db6 100644
--- a/git-sh-i18n.sh
+++ b/git-sh-i18n.sh
@@ -51,12 +51,6 @@ gettext_without_eval_gettext)
 		)
 	}
 
-	eval_ngettext () {
-		ngettext "$1" "$2" "$3" | (
-			export PATH $(git sh-i18n--envsubst --variables "$2");
-			git sh-i18n--envsubst "$2"
-		)
-	}
 	;;
 *)
 	gettext () {
@@ -70,12 +64,6 @@ gettext_without_eval_gettext)
 		)
 	}
 
-	eval_ngettext () {
-		(test "$3" = 1 && printf "%s" "$1" || printf "%s" "$2") | (
-			export PATH $(git sh-i18n--envsubst --variables "$2");
-			git sh-i18n--envsubst "$2"
-		)
-	}
 	;;
 esac
 
-- 
2.33.1.1494.g88b39a443e1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 2/2] git-sh-setup: remove messaging supporting --preserve-merges
  2021-10-21 18:37 [PATCH 0/2] remove code that's dead post-rebase --preserve-merges removal Ævar Arnfjörð Bjarmason
  2021-10-21 18:37 ` [PATCH 1/2] git-sh-i18n: remove unused eval_ngettext() Ævar Arnfjörð Bjarmason
@ 2021-10-21 18:37 ` Ævar Arnfjörð Bjarmason
  1 sibling, 0 replies; 3+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-21 18:37 UTC (permalink / raw)
  To: git
  Cc: Junio C Hamano, Johannes Schindelin,
	Ævar Arnfjörð Bjarmason

Remove messages that were last used by the code removed in
a74b35081c5 (rebase: drop support for `--preserve-merges`,
2021-09-07).

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
 git-sh-setup.sh | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/git-sh-setup.sh b/git-sh-setup.sh
index cee053cdc38..960982f9d53 100644
--- a/git-sh-setup.sh
+++ b/git-sh-setup.sh
@@ -217,9 +217,6 @@ require_clean_work_tree () {
 	then
 		action=$1
 		case "$action" in
-		rebase)
-			gettextln "Cannot rebase: You have unstaged changes." >&2
-			;;
 		"rewrite branches")
 			gettextln "Cannot rewrite branches: You have unstaged changes." >&2
 			;;
@@ -235,14 +232,7 @@ require_clean_work_tree () {
 		if test $err = 0
 		then
 			action=$1
-			case "$action" in
-			rebase)
-				gettextln "Cannot rebase: Your index contains uncommitted changes." >&2
-				;;
-			*)
-				eval_gettextln "Cannot \$action: Your index contains uncommitted changes." >&2
-				;;
-			esac
+			eval_gettextln "Cannot \$action: Your index contains uncommitted changes." >&2
 		else
 		    gettextln "Additionally, your index contains uncommitted changes." >&2
 		fi
-- 
2.33.1.1494.g88b39a443e1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-10-21 18:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-21 18:37 [PATCH 0/2] remove code that's dead post-rebase --preserve-merges removal Ævar Arnfjörð Bjarmason
2021-10-21 18:37 ` [PATCH 1/2] git-sh-i18n: remove unused eval_ngettext() Ævar Arnfjörð Bjarmason
2021-10-21 18:37 ` [PATCH 2/2] git-sh-setup: remove messaging supporting --preserve-merges Ævar Arnfjörð Bjarmason

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.