All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Đoàn Trần Công Danh" <congdanhqx@gmail.com>
To: Atharva Raykar <raykar.ath@gmail.com>
Cc: git@vger.kernel.org, Emily Shaffer <emilyshaffer@google.com>,
	Jonathan Nieder <jrnieder@gmail.com>,
	Junio C Hamano <gitster@pobox.com>,
	Christian Couder <christian.couder@gmail.com>,
	Shourya Shukla <periperidip@gmail.com>,
	Kaartic Sivaraam <kaartic.sivaraam@gmail.com>,
	Eric Sunshine <sunshine@sunshineco.com>,
	Prathamesh Chavan <pc44800@gmail.com>,
	Rafael Silva <rafaeloliveira.cs@gmail.com>
Subject: Re: [GSoC] [PATCH v2 2/4] submodule: prefix die messages with 'fatal'
Date: Fri, 9 Jul 2021 21:52:29 +0700	[thread overview]
Message-ID: <YOhirWqj7ajsqlYw@danh.dev> (raw)
In-Reply-To: <20210708095533.26226-3-raykar.ath@gmail.com>

On 2021-07-08 15:25:31+0530, Atharva Raykar <raykar.ath@gmail.com> wrote:
> The standard `die()` function that is used in C code prefixes all the
> messages passed to it with 'fatal: '. This does not happen with the
> `die` used in 'git-submodule.sh'.
> 
> Let's prefix each of the shell die messages with 'fatal: ' so that when
> they are converted to C code, the error messages stay the same as before
> the conversion.

That sounds good.

> --- a/git-submodule.sh
> +++ b/git-submodule.sh
> @@ -147,7 +147,7 @@ cmd_add()
>  
>  	if ! git submodule--helper config --check-writeable >/dev/null 2>&1
>  	then
> -		 die "$(eval_gettext "please make sure that the .gitmodules file is in the working tree")"
> +		 die "$(eval_gettext "fatal: please make sure that the .gitmodules file is in the working tree")"

Except that, "fatal: " isn't subjected to translation. And this will
create new translatable item for translator. Perhaps:

-		 die "$(eval_gettext "please make sure that the .gitmodules file is in the working tree")"
+		 die "fatal: $(eval_gettext "please make sure that the .gitmodules file is in the working tree")"

-- Danh

>  	fi
>  
>  	if test -n "$reference_path"
> @@ -176,7 +176,7 @@ cmd_add()
>  	case "$repo" in
>  	./*|../*)
>  		test -z "$wt_prefix" ||
> -		die "$(gettext "Relative path can only be used from the toplevel of the working tree")"
> +		die "$(gettext "fatal: Relative path can only be used from the toplevel of the working tree")"
>  
>  		# dereference source url relative to parent's url
>  		realrepo=$(git submodule--helper resolve-relative-url "$repo") || exit
> @@ -186,7 +186,7 @@ cmd_add()
>  		realrepo=$repo
>  		;;
>  	*)
> -		die "$(eval_gettext "repo URL: '\$repo' must be absolute or begin with ./|../")"
> +		die "$(eval_gettext "fatal: repo URL: '\$repo' must be absolute or begin with ./|../")"
>  	;;
>  	esac
>  
> @@ -205,17 +205,17 @@ cmd_add()
>  	if test -z "$force"
>  	then
>  		git ls-files --error-unmatch "$sm_path" > /dev/null 2>&1 &&
> -		die "$(eval_gettext "'\$sm_path' already exists in the index")"
> +		die "$(eval_gettext "fatal: '\$sm_path' already exists in the index")"
>  	else
>  		git ls-files -s "$sm_path" | sane_grep -v "^160000" > /dev/null 2>&1 &&
> -		die "$(eval_gettext "'\$sm_path' already exists in the index and is not a submodule")"
> +		die "$(eval_gettext "fatal: '\$sm_path' already exists in the index and is not a submodule")"
>  	fi
>  
>  	if test -d "$sm_path" &&
>  		test -z $(git -C "$sm_path" rev-parse --show-cdup 2>/dev/null)
>  	then
>  	    git -C "$sm_path" rev-parse --verify -q HEAD >/dev/null ||
> -	    die "$(eval_gettext "'\$sm_path' does not have a commit checked out")"
> +	    die "$(eval_gettext "fatal: '\$sm_path' does not have a commit checked out")"
>  	fi
>  
>  	if test -z "$force"
> @@ -238,7 +238,7 @@ cmd_add()
>  
>  	if ! git submodule--helper check-name "$sm_name"
>  	then
> -		die "$(eval_gettext "'$sm_name' is not a valid submodule name")"
> +		die "$(eval_gettext "fatal: '$sm_name' is not a valid submodule name")"
>  	fi
>  
>  	# perhaps the path exists and is already a git repo, else clone it
> @@ -281,7 +281,7 @@ or you are unsure what this means choose another name with the '--name' option."
>  	git config submodule."$sm_name".url "$realrepo"
>  
>  	git add --no-warn-embedded-repo $force "$sm_path" ||
> -	die "$(eval_gettext "Failed to add submodule '\$sm_path'")"
> +	die "$(eval_gettext "fatal: Failed to add submodule '\$sm_path'")"
>  
>  	git submodule--helper config submodule."$sm_name".path "$sm_path" &&
>  	git submodule--helper config submodule."$sm_name".url "$repo" &&
> @@ -290,7 +290,7 @@ or you are unsure what this means choose another name with the '--name' option."
>  		git submodule--helper config submodule."$sm_name".branch "$branch"
>  	fi &&
>  	git add --force .gitmodules ||
> -	die "$(eval_gettext "Failed to register submodule '\$sm_path'")"
> +	die "$(eval_gettext "fatal: Failed to register submodule '\$sm_path'")"
>  
>  	# NEEDSWORK: In a multi-working-tree world, this needs to be
>  	# set in the per-worktree config.
> @@ -565,7 +565,7 @@ cmd_update()
>  		else
>  			subsha1=$(sanitize_submodule_env; cd "$sm_path" &&
>  				git rev-parse --verify HEAD) ||
> -			die "$(eval_gettext "Unable to find current revision in submodule path '\$displaypath'")"
> +			die "$(eval_gettext "fatal: Unable to find current revision in submodule path '\$displaypath'")"
>  		fi
>  
>  		if test -n "$remote"
> @@ -575,12 +575,12 @@ cmd_update()
>  			then
>  				# Fetch remote before determining tracking $sha1
>  				fetch_in_submodule "$sm_path" $depth ||
> -				die "$(eval_gettext "Unable to fetch in submodule path '\$sm_path'")"
> +				die "$(eval_gettext "fatal: Unable to fetch in submodule path '\$sm_path'")"
>  			fi
>  			remote_name=$(sanitize_submodule_env; cd "$sm_path" && git submodule--helper print-default-remote)
>  			sha1=$(sanitize_submodule_env; cd "$sm_path" &&
>  				git rev-parse --verify "${remote_name}/${branch}") ||
> -			die "$(eval_gettext "Unable to find current \${remote_name}/\${branch} revision in submodule path '\$sm_path'")"
> +			die "$(eval_gettext "fatal: Unable to find current \${remote_name}/\${branch} revision in submodule path '\$sm_path'")"
>  		fi
>  
>  		if test "$subsha1" != "$sha1" || test -n "$force"
> @@ -604,36 +604,36 @@ cmd_update()
>  				# not be reachable from any of the refs
>  				is_tip_reachable "$sm_path" "$sha1" ||
>  				fetch_in_submodule "$sm_path" "$depth" "$sha1" ||
> -				die "$(eval_gettext "Fetched in submodule path '\$displaypath', but it did not contain \$sha1. Direct fetching of that commit failed.")"
> +				die "$(eval_gettext "fatal: Fetched in submodule path '\$displaypath', but it did not contain \$sha1. Direct fetching of that commit failed.")"
>  			fi
>  
>  			must_die_on_failure=
>  			case "$update_module" in
>  			checkout)
>  				command="git checkout $subforce -q"
> -				die_msg="$(eval_gettext "Unable to checkout '\$sha1' in submodule path '\$displaypath'")"
> +				die_msg="$(eval_gettext "fatal: Unable to checkout '\$sha1' in submodule path '\$displaypath'")"
>  				say_msg="$(eval_gettext "Submodule path '\$displaypath': checked out '\$sha1'")"
>  				;;
>  			rebase)
>  				command="git rebase ${GIT_QUIET:+--quiet}"
> -				die_msg="$(eval_gettext "Unable to rebase '\$sha1' in submodule path '\$displaypath'")"
> +				die_msg="$(eval_gettext "fatal: Unable to rebase '\$sha1' in submodule path '\$displaypath'")"
>  				say_msg="$(eval_gettext "Submodule path '\$displaypath': rebased into '\$sha1'")"
>  				must_die_on_failure=yes
>  				;;
>  			merge)
>  				command="git merge ${GIT_QUIET:+--quiet}"
> -				die_msg="$(eval_gettext "Unable to merge '\$sha1' in submodule path '\$displaypath'")"
> +				die_msg="$(eval_gettext "fatal: Unable to merge '\$sha1' in submodule path '\$displaypath'")"
>  				say_msg="$(eval_gettext "Submodule path '\$displaypath': merged in '\$sha1'")"
>  				must_die_on_failure=yes
>  				;;
>  			!*)
>  				command="${update_module#!}"
> -				die_msg="$(eval_gettext "Execution of '\$command \$sha1' failed in submodule path '\$displaypath'")"
> +				die_msg="$(eval_gettext "fatal: Execution of '\$command \$sha1' failed in submodule path '\$displaypath'")"
>  				say_msg="$(eval_gettext "Submodule path '\$displaypath': '\$command \$sha1'")"
>  				must_die_on_failure=yes
>  				;;
>  			*)
> -				die "$(eval_gettext "Invalid update mode '$update_module' for submodule path '$path'")"
> +				die "$(eval_gettext "fatal: Invalid update mode '$update_module' for submodule path '$path'")"
>  			esac
>  
>  			if (sanitize_submodule_env; cd "$sm_path" && $command "$sha1")
> @@ -660,7 +660,7 @@ cmd_update()
>  			res=$?
>  			if test $res -gt 0
>  			then
> -				die_msg="$(eval_gettext "Failed to recurse into submodule path '\$displaypath'")"
> +				die_msg="$(eval_gettext "fatal: Failed to recurse into submodule path '\$displaypath'")"
>  				if test $res -ne 2
>  				then
>  					err="${err};$die_msg"
> diff --git a/t/t7400-submodule-basic.sh b/t/t7400-submodule-basic.sh
> index 7aa7fefdfa..cb1b8e35db 100755
> --- a/t/t7400-submodule-basic.sh
> +++ b/t/t7400-submodule-basic.sh
> @@ -51,7 +51,7 @@ test_expect_success 'submodule update aborts on missing gitmodules url' '
>  
>  test_expect_success 'add aborts on repository with no commits' '
>  	cat >expect <<-\EOF &&
> -	'"'repo-no-commits'"' does not have a commit checked out
> +	fatal: '"'repo-no-commits'"' does not have a commit checked out
>  	EOF
>  	git init repo-no-commits &&
>  	test_must_fail git submodule add ../a ./repo-no-commits 2>actual &&
> @@ -199,7 +199,7 @@ test_expect_success 'submodule add to .gitignored path with --force' '
>  test_expect_success 'submodule add to path with tracked content fails' '
>  	(
>  		cd addtest &&
> -		echo "'\''dir-tracked'\'' already exists in the index" >expect &&
> +		echo "fatal: '\''dir-tracked'\'' already exists in the index" >expect &&
>  		mkdir dir-tracked &&
>  		test_commit foo dir-tracked/bar &&
>  		test_must_fail git submodule add "$submodurl" dir-tracked >actual 2>&1 &&
> diff --git a/t/t7406-submodule-update.sh b/t/t7406-submodule-update.sh
> index f4f61fe554..11cccbb333 100755
> --- a/t/t7406-submodule-update.sh
> +++ b/t/t7406-submodule-update.sh
> @@ -448,7 +448,7 @@ test_expect_success 'fsck detects command in .gitmodules' '
>  '
>  
>  cat << EOF >expect
> -Execution of 'false $submodulesha1' failed in submodule path 'submodule'
> +fatal: Execution of 'false $submodulesha1' failed in submodule path 'submodule'
>  EOF
>  
>  test_expect_success 'submodule update - command in .git/config catches failure' '
> @@ -465,7 +465,7 @@ test_expect_success 'submodule update - command in .git/config catches failure'
>  '
>  
>  cat << EOF >expect
> -Execution of 'false $submodulesha1' failed in submodule path '../submodule'
> +fatal: Execution of 'false $submodulesha1' failed in submodule path '../submodule'
>  EOF
>  
>  test_expect_success 'submodule update - command in .git/config catches failure -- subdirectory' '
> @@ -484,7 +484,7 @@ test_expect_success 'submodule update - command in .git/config catches failure -
>  
>  test_expect_success 'submodule update - command run for initial population of submodule' '
>  	cat >expect <<-EOF &&
> -	Execution of '\''false $submodulesha1'\'' failed in submodule path '\''submodule'\''
> +	fatal: Execution of '\''false $submodulesha1'\'' failed in submodule path '\''submodule'\''
>  	EOF
>  	rm -rf super/submodule &&
>  	test_must_fail git -C super submodule update 2>actual &&
> @@ -493,8 +493,8 @@ test_expect_success 'submodule update - command run for initial population of su
>  '
>  
>  cat << EOF >expect
> -Execution of 'false $submodulesha1' failed in submodule path '../super/submodule'
> -Failed to recurse into submodule path '../super'
> +fatal: Execution of 'false $submodulesha1' failed in submodule path '../super/submodule'
> +fatal: Failed to recurse into submodule path '../super'
>  EOF
>  
>  test_expect_success 'recursive submodule update - command in .git/config catches failure -- subdirectory' '
> -- 
> 2.32.0
> 

-- 
Danh

  parent reply	other threads:[~2021-07-09 14:52 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-06 18:19 [GSoC] [PATCH 0/3] submodule add: partial conversion to C Atharva Raykar
2021-07-06 18:19 ` [GSoC] [PATCH 1/3] t7400: test failure to add submodule in tracked path Atharva Raykar
2021-07-06 18:19 ` [GSoC] [PATCH 2/3] submodule--helper: refactor module_clone() Atharva Raykar
2021-07-06 18:19 ` [GSoC] [PATCH 3/3] submodule--helper: introduce add-clone subcommand Atharva Raykar
2021-07-07 19:57   ` Junio C Hamano
2021-07-08  6:45     ` Atharva Raykar
2021-07-08  9:55 ` [GSoC] [PATCH v2 0/4] submodule add: partial conversion to C Atharva Raykar
2021-07-08  9:55   ` [GSoC] [PATCH v2 1/4] t7400: test failure to add submodule in tracked path Atharva Raykar
2021-07-08  9:55   ` [GSoC] [PATCH v2 2/4] submodule: prefix die messages with 'fatal' Atharva Raykar
2021-07-08 15:17     ` Junio C Hamano
2021-07-09 14:52     ` Đoàn Trần Công Danh [this message]
2021-07-10  7:52       ` Atharva Raykar
2021-07-10 12:04         ` Kaartic Sivaraam
2021-07-08  9:55   ` [GSoC] [PATCH v2 3/4] submodule--helper: refactor module_clone() Atharva Raykar
2021-07-08  9:55   ` [GSoC] [PATCH v2 4/4] submodule--helper: introduce add-clone subcommand Atharva Raykar
2021-07-10  7:47   ` [GSoC] [PATCH v3 0/4] submodule add: partial conversion to C Atharva Raykar
2021-07-10  7:47     ` [GSoC] [PATCH v3 1/4] t7400: test failure to add submodule in tracked path Atharva Raykar
2021-07-10  7:47     ` [GSoC] [PATCH v3 2/4] submodule: prefix die messages with 'fatal' Atharva Raykar
2021-07-10  7:48     ` [GSoC] [PATCH v3 3/4] submodule--helper: refactor module_clone() Atharva Raykar
2021-07-10  7:48     ` [GSoC] [PATCH v3 4/4] submodule--helper: introduce add-clone subcommand Atharva Raykar
2021-07-23 11:12       ` [PATCH] submodule: drop unused sm_name parameter from show_fetch_remotes() Jeff King
2021-07-23 17:12         ` Atharva Raykar
2021-07-26 19:03           ` Junio C Hamano
2021-08-05 19:28       ` [PATCH] submodule--helper: fix incorrect newlines in an error message Kaartic Sivaraam
2021-08-06  6:29         ` Atharva Raykar
2021-08-06 19:07           ` Kaartic Sivaraam
2021-09-18 19:31         ` [PATCH v2 0/1] submodule: corret an incorrectly formatted " Kaartic Sivaraam
2021-09-18 19:31           ` [PATCH v2 1/1] submodule--helper: fix incorrect newlines in an " Kaartic Sivaraam
2021-09-20 18:09             ` Junio C Hamano
2021-09-21 16:52               ` Atharva Raykar
2021-09-21 16:47             ` Atharva Raykar
2021-10-23 12:57               ` [PATCH v3 0/1] submodule: correct an incorrectly formatted " Kaartic Sivaraam
2021-10-23 12:57                 ` [PATCH v3 1/1] submodule--helper: fix incorrect newlines in an " Kaartic Sivaraam
2021-10-24  6:05                 ` [PATCH v3 0/1] submodule: correct an incorrectly formatted " Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YOhirWqj7ajsqlYw@danh.dev \
    --to=congdanhqx@gmail.com \
    --cc=christian.couder@gmail.com \
    --cc=emilyshaffer@google.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=jrnieder@gmail.com \
    --cc=kaartic.sivaraam@gmail.com \
    --cc=pc44800@gmail.com \
    --cc=periperidip@gmail.com \
    --cc=rafaeloliveira.cs@gmail.com \
    --cc=raykar.ath@gmail.com \
    --cc=sunshine@sunshineco.com \
    --subject='Re: [GSoC] [PATCH v2 2/4] submodule: prefix die messages with '\''fatal'\''' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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.