From: Elijah Newren <newren@gmail.com>
Date: Wed, 04 Sep 2019 20:32:47 GMT
Subject: Re: [PATCH v5 2/4] t3427: accelerate this test by using fast-export and fast-import
Message-ID: <CABPp-BFi9puCJ8H08ZD_VSM2pygt3GoL52QNuaVLkN_130=n0w@mail.gmail.com>
In-Reply-To: <xmqqftldroae.fsf@gitster-ct.c.googlers.com>

On Tue, Sep 3, 2019 at 3:46 PM Junio C Hamano <gitster@pobox.com> wrote:
>
> Junio C Hamano <gitster@pobox.com> writes:
>
> > Elijah Newren <newren@gmail.com> writes:
> >
> >> +extract_files_subtree () {
> >> +    git fast-export --no-data HEAD -- files_subtree/ |
> >> +            sed -e "s%\([0-9a-f]\{40\} \)files_subtree/%\1%" |
> >> +            git fast-import --force --quiet
> >> +}
>
> This change has obvious interactions with Dscho's d51b771d ("t3427:
> move the `filter-branch` invocation into the `setup` case",
> 2019-07-31) that is still in flight, but in a good way.  There only
> needs a single callsite for the above helper function after that
> step.
>
> I think I'll discard this step from the "move us closer to deprecate
> filter-branch" topic, and ask you and Dscho to work together to have
> it or its moral equivalent included as part of js/rebase-r-strategy
> topic.

Sounds good.  I'll resubmit it separately as a patch on top of his topic.