# [PATCH] filter-branch: Fix error message for --prune-empty --commit-filter

4 messages from 2010-02-08 to 2010-02-12. Participants: Jacob Helwig, Michael J Gruber.
Thread: https://gitlist.dev/t/22565

## Jacob Helwig, 2010-02-08 02:36

Subject: [PATCH] filter-branch: Fix error message for --prune-empty --commit-filter
Message-ID: <1265596587-9949-1-git-send-email-jacob.helwig@gmail.com>
URL: https://gitlist.dev/e/1265596587-9949-1-git-send-email-jacob.helwig%40gmail.com

```
Running filter-branch with --prune-empty and --commit-filter, no longer
incorrectly reports that you had tried to run '--filter-commit' in
combination with '--prune-empty'.
---

This problem with filter-branch's error message was originally brought up by
hipe in the IRC channel.

 git-filter-branch.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/git-filter-branch.sh b/git-filter-branch.sh
index e95845c..88fb0f0 100755
--- a/git-filter-branch.sh
+++ b/git-filter-branch.sh
@@ -207,7 +207,7 @@ t,)
 ,*)
 	;;
 *)
-	die "Cannot set --prune-empty and --filter-commit at the same time"
+	die "Cannot set --prune-empty and --commit-filter at the same time"
 esac
 
 case "$force" in
-- 
1.7.0.rc1.50.g8424

```

## Jacob Helwig, 2010-02-08 02:38

Subject: [PATCH] filter-branch: Fix error message for --prune-empty --commit-filter
Message-ID: <1265596681-10561-1-git-send-email-jacob.helwig@gmail.com>
URL: https://gitlist.dev/e/1265596681-10561-1-git-send-email-jacob.helwig%40gmail.com
In-Reply-To: <1265596587-9949-1-git-send-email-jacob.helwig@gmail.com>

```
Running filter-branch with --prune-empty and --commit-filter, no longer
incorrectly reports that you had tried to run '--filter-commit' in
combination with '--prune-empty'.

Signed-off-by: Jacob Helwig <jacob.helwig@gmail.com>
---

Sorry for the re-send.  Forgot the SOB...

 git-filter-branch.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/git-filter-branch.sh b/git-filter-branch.sh
index e95845c..88fb0f0 100755
--- a/git-filter-branch.sh
+++ b/git-filter-branch.sh
@@ -207,7 +207,7 @@ t,)
 ,*)
 	;;
 *)
-	die "Cannot set --prune-empty and --filter-commit at the same time"
+	die "Cannot set --prune-empty and --commit-filter at the same time"
 esac
 
 case "$force" in
-- 
1.7.0.rc1.50.g8424

```

## Michael J Gruber, 2010-02-08 10:10

Subject: Re: [PATCH] filter-branch: Fix error message for --prune-empty --commit-filter
Message-ID: <4B6FE325.5000909@drmicha.warpmail.net>
URL: https://gitlist.dev/e/4B6FE325.5000909%40drmicha.warpmail.net
In-Reply-To: <1265596681-10561-1-git-send-email-jacob.helwig@gmail.com>

```
Jacob Helwig venit, vidit, dixit 08.02.2010 03:38:
> Running filter-branch with --prune-empty and --commit-filter, no longer
> incorrectly reports that you had tried to run '--filter-commit' in
> combination with '--prune-empty'.

In order to understand this sentence, I had to look twice at the patch text.

What's going on is that that typo was there from the beginning, right?
(Induced from the name of a script variable, I see.) How about:

Running filter-branch with --prune-empty and --commit-filter reports
"Cannot set --prune-empty and --filter-commit at the same time".

Change it to use the correct name "--commit-filter" of the option.


> Signed-off-by: Jacob Helwig <jacob.helwig@gmail.com>
> ---
> 
> Sorry for the re-send.  Forgot the SOB...
> 
>  git-filter-branch.sh |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/git-filter-branch.sh b/git-filter-branch.sh
> index e95845c..88fb0f0 100755
> --- a/git-filter-branch.sh
> +++ b/git-filter-branch.sh
> @@ -207,7 +207,7 @@ t,)
>  ,*)
>  	;;
>  *)
> -	die "Cannot set --prune-empty and --filter-commit at the same time"
> +	die "Cannot set --prune-empty and --commit-filter at the same time"
>  esac
>  
>  case "$force" in

```

## Jacob Helwig, 2010-02-12 02:46

Subject: [PATCH] filter-branch: Fix error message for --prune-empty --commit-filter
Message-ID: <1265942782-16257-1-git-send-email-jacob.helwig@gmail.com>
URL: https://gitlist.dev/e/1265942782-16257-1-git-send-email-jacob.helwig%40gmail.com
In-Reply-To: <4B6FE325.5000909@drmicha.warpmail.net>

```
Running filter-branch with --prune-empty and --commit-filter reports:
  "Cannot set --prune-empty and --filter-commit at the same time".

Change it to use the correct option name: --commit-filter

Signed-off-by: Jacob Helwig <jacob.helwig@gmail.com>
---

Sorry for taking so long to respond to this.

I wasn't quite happy with the suggested commit message, but hopefully this will
make everyone happy.

 git-filter-branch.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/git-filter-branch.sh b/git-filter-branch.sh
index e95845c..88fb0f0 100755
--- a/git-filter-branch.sh
+++ b/git-filter-branch.sh
@@ -207,7 +207,7 @@ t,)
 ,*)
 	;;
 *)
-	die "Cannot set --prune-empty and --filter-commit at the same time"
+	die "Cannot set --prune-empty and --commit-filter at the same time"
 esac
 
 case "$force" in
-- 
1.6.6.1

```