[PATCH] filter-branch.sh: remove temporary directory on failure

From: Brandon Casey <casey@nrlssc.navy.mil>

One of the first things filter-branch does is to create a temporary directory. This directory is eventually removed by the script during normal operation, but is not removed if the script encounters an error.

Set a trap to remove it when the script terminates for any reason.

Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
---

Even though this directory may be useful for debugging when you encounter a problem, I wonder if the normal "error" will be _user_ error. In which case the user will adjust the command line parameters and try to rerun. Currently the user would then receive the message

     "$tempdir already exists, please remove it"

Then, rm -rf the tempdir, then rerun.

Is this necessary? Maybe anyone with the know-how to debug would also have the know-how to comment out the 'trap' in the script (or direct someone asking for help on the mailing list to do so).

If we really want to leave this temporary directory around on failure, maybe we can at least remove it for some simple errors like:

    line 198: die "Namespace $orig_namespace not empty"
    line 217: die "Which ref do you want to rewrite?"
    etc.

by moving the 'trap - 0' higher up.

-brandon

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