Why doesn't git blame --ignore-rev/--ignore-revs-file work for me?

Check if Git 2.29 (Q4 2020) has fixed the isue: "git blame --ignore-rev/--ignore-revs-file"^(man) failed to validate their input are valid revision, and failed to take into account that the user may want to give an annotated tag instead of a commit, which has been corrected.

See commit 610e2b9, commit f58931c (24 Sep 2020) by Junio C Hamano (gitster).
^{(Merged by Junio C Hamano -- gitster -- in commit 230ff3e, 04 Oct 2020)}

blame: validate and peel the object names on the ignore list

The command reads list of object names to place on the ignore list either from the command line or from a file, but they are not checked with their object type (those read from the file are not even checked for object existence).

Extend the oidset_parse_file() API and allow it to take a callback that can be used to die (e.g. when an inappropriate input is read) or modify the object name read (e.g. when a tag pointing at a commit is read, and the caller wants a commit object name), and use it in the code that handles ignore list.

With Git 2.30 (Q1 2021), "git blame --ignore-revs-file=<file>^(man)" learned to ignore a non-existent object name in the input, instead of complaining.

See commit c714d05 (10 Nov 2020) by René Scharfe (rscharfe).
^{(Merged by Junio C Hamano -- gitster -- in commit b4e245a, 18 Nov 2020)}

blame: silently ignore invalid ignore file objects

^{Reported-by: Jean-Yves Avenard
Signed-off-by: René Scharfe
Reviewed-by: Barret Rhoden}

Since 610e2b9240 ("blame: validate and peel the object names on the ignore list", 2020-09-24, Git v2.29.0-rc0 -- merge listed in batch #19) git blame^(man) reports checks if objects specified with --ignore-rev and in files loaded with --ignore-revs-file and config option blame.ignoreRevsFile are actual objects and dies if they aren't. The intent is to report typos to the user.

This also breaks the ability to use a single ignore file for multiple repositories.
Typos are presumably less likely in files than on the command line, so alerting is less useful here.
Restore that feature by skipping non-commits without dying.

TL;DR: the ignore-revs feature is designed for refactoring commits, not commits where lines are changed entirely.

Hi Wayne,

Thanks for trying it out. Sorry your first experience with it wasn't good. I think the docs could be improved to communicate the intent of the feature better.

git blame --ignore-rev is trying to find a line that is somewhat similar to "awesome". But "awesome" and "two" have nothing in common so it gives up and attributes "awesome" to the actual commit that added it.

There is a feature to identify this scenario, but it needs to be explicitly enabled:

If the blame.markUnblamableLines config option is set, then those lines touched by an ignored commit that we could not attribute to another revision are marked with a *.

Using this option, with your example script I see a * to indicate the problem:

^6bce3bb (One 2021-03-28 15:08:08 +0100 1) one*b75aaf2 (Two 2021-03-28 15:08:08 +0100 2) awesome5d3b18c7 (One 2021-03-28 15:08:08 +0100 3) sauce

If you try git blame --ignore-rev on a commit that you genuinely want to ignore (for example reformatting your code) then it should work well.

For example, if I change your script to replace two with awesome TWO then I see this:

^5307c74 (One 2021-03-28 15:14:52 +0100 1) one^5307c74 (One 2021-03-28 15:14:52 +0100 2) awesome TWO9d3fcc01 (One 2021-03-28 15:14:52 +0100 3) sauce