Hi,
On Aug 23, 2014 12:26 PM, Jeff King <peff@xxxxxxxx> 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
> packed-refs.
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 <peff@xxxxxxxx>
> ---
> 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.
Best,
Gábor��.n��������+%������w��{.n��������n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�