Thread (44 messages) 44 messages, 3 authors, 2021-09-17

Re: [PATCH v2 4/7] p5326: create missing 'perf-tag' tag

From: Taylor Blau <hidden>
Date: 2021-09-17 04:14:45

On Thu, Sep 16, 2021 at 06:36:54PM -0400, Jeff King wrote:
On Tue, Sep 14, 2021 at 06:06:09PM -0400, Taylor Blau wrote:
quoted
Some of the tests in test_full_bitmap rely on having a tag named
perf-tag in place. We could create it in test_full_bitmap(), but we want
to have it in place before the repack starts.
I wondered how p5326 ever could have worked without this.

[...]

Knowing the history of the midx-bitmap series, I'm almost certain what
happened is that it got rebased, and you probably _did_ see a textual
conflict, which you resolved correctly, moving the perf-tag test into
lib-bitmap.sh But you missed the semantic conflict that p5326 also
needed to add in the setup step.
Sounds about right :-).

Thanks,
Taylor
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help