Re: [PATCH] git-rev-list: Don't die on a bad tag

Frank Sorenson [off-list ref] writes:

The current cogito tree contains an invalid tag
(junio-gpg-pub => 0918385dbd9656cab0d1d81ba7453d49bbc16250), and
git-repack-script (and probably others) will die when git-rev-list
tries to dump the objects.  This patch changes it to complain, but
continue.

Signed-off-by: Frank Sorenson <redacted>

I think this patch is wrong.

The check you are disabling is to see if the repository contains
an object that a tag refers to, and the user wanted to slurp the
tag, along with whatever thing it refers to, so barfing and
dying is the only right thing to do.

The repository does not have the object the tag points at.  IOW,
the repository is not fsck clean.

If it bothers you either you could drop that tag from cogito
repository (because there would not be anything that is signed
by _my_ public key anyway), or borrow the object from git.git
repository.