Re: [PATCH v2 0/8] commit-graph: segfault & other fixes for broken graphs

2019-03-15 Thread Ævar Arnfjörð Bjarmason
On Fri, Mar 15 2019, Eric Sunshine wrote: > On Thu, Mar 14, 2019 at 5:47 PM Ævar Arnfjörð Bjarmason > wrote: >> I fixed a test to avoid the pattern a0a630192d >> (t/check-non-portable-shell: detect "FOO=bar shell_func", 2018-07-13) >> tests for. The new pattern is more obvious. >> >> As an asid

Re: [PATCH v2 0/8] commit-graph: segfault & other fixes for broken graphs

2019-03-15 Thread Eric Sunshine
On Thu, Mar 14, 2019 at 5:47 PM Ævar Arnfjörð Bjarmason wrote: > I fixed a test to avoid the pattern a0a630192d > (t/check-non-portable-shell: detect "FOO=bar shell_func", 2018-07-13) > tests for. The new pattern is more obvious. > > As an aside I don't get how that doesn't work as intended from t

[PATCH v2 0/8] commit-graph: segfault & other fixes for broken graphs

2019-03-14 Thread Ævar Arnfjörð Bjarmason
See the v1 cover letter for details: https://public-inbox.org/git/20190221223753.20070-1-ava...@gmail.com/ I'd forgotten this after 2.21 was released. This addresses all the comments on v1 and rebases it. A range-diff is below. I also improved 7/8's commit message a bit. I fixed a test to avoid