Re: [PATCH v2 9/9] ci: include the built-in `git add -i` in the `linux-gcc` job
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Jan 1, 2020, 22:10 UTC
- Message-ID
- <nycvar.QRO.7.76.6.2001012309390.46@tvgsbejvaqbjf.bet>
- In-Reply-To
- <600bb1f7-b54a-3bf7-40ae-af656768a752@gmail.com>
Hi Stolee,
On Thu, 26 Dec 2019, Derrick Stolee wrote:
Show 27 quoted lines
> On 12/25/2019 6:57 AM, Johannes Schindelin via GitGitGadget wrote: > > From: Johannes Schindelin <johannes.schindelin@gmx.de> > > > > This job runs the test suite twice, once in regular mode, and once with > > a whole slew of `GIT_TEST_*` variables set. > > > > Now that the built-in version of `git add --interactive` is > > feature-complete, let's also throw `GIT_TEST_ADD_I_USE_BUILTIN` into > > that fray. > > > > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> > > --- > > ci/run-build-and-tests.sh | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/ci/run-build-and-tests.sh b/ci/run-build-and-tests.sh > > index ff0ef7f08e..4df54c4efe 100755 > > --- a/ci/run-build-and-tests.sh > > +++ b/ci/run-build-and-tests.sh > > @@ -20,6 +20,7 @@ linux-gcc) > > export GIT_TEST_OE_DELTA_SIZE=5 > > export GIT_TEST_COMMIT_GRAPH=1 > > export GIT_TEST_MULTI_PACK_INDEX=1 > > + export GIT_TEST_ADD_I_USE_BUILTIN=1 > > make test > > I see that I need to add this to the test-coverage builds.
Thank you for catching this!
It makes me wonder whether the test-coverage builds should use some `sed` invocation on the `ci/run-build-and-tests.sh` script, though, so that you do not have to edit the Azure Pipelines definition manually all the time?
Ciao, Dscho