On Aug 23, 2014 12:26 PM, Jeff King <p...@peff.net> wrote:
> Since dd0b72c (bash prompt: use bash builtins to check stash
> state, 2011-04-01), git-prompt checks whether we have a
> stash by looking for $GIT_DIR/refs/stash. Generally external
> programs should never do this, because they would miss
Not sure whether the prompt script is external program or not, but doesn't
matter, this is the right thing to do.
> That commit claims that packed-refs does not pack
> refs/stash, but that is not quite true. It does pack the
> ref, but due to a bug, fails to prune the ref. When we fix
> that bug, we would want to be doing the right thing here.
> Signed-off-by: Jeff King <p...@peff.net>
> I know we are pretty sensitive to forks in the prompt code (after all,
> that was the point of dd0b72c). This patch is essentially a reversion of
> this hunk of dd0b72c, and is definitely safe.
I'm not sure, but if I remember correctly (don't have the means to check it at
the moment, sorry) in that commit I also added a 'git pack-ref' invocation to
the relevant test(s?) to guard us against breakages due to changes in 'git
pack-refs'. If that is so, then I think those invocations should be removed as
well, as they'll become useless.