Re: [PATCH v3 4/6] revision: add new parameter to exclude hidden refs
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Nov 7, 2022, 17:07 UTC
- Message-ID
- <221107.86eduevgk3.gmgdl@evledraar.gmail.com>
- In-Reply-To
- <221107.86pmdyx4me.gmgdl@evledraar.gmail.com>
On Mon, Nov 07 2022, Ævar Arnfjörð Bjarmason wrote:
> On Mon, Nov 07 2022, Patrick Steinhardt wrote:
>> +TEST_PASSES_SANITIZE_LEAK=true > > Thanks for adding this! :)
Hrm, I spoke too soon :) This series adds new leaks, so it'll fail with the linux-leaks job. I have the following local monkeypatch on top, which obviously doesn't address the root cause. The t6018 leak is new due to the new tests you added.
diff --git a/t/t6018-rev-list-glob.sh b/t/t6018-rev-list-glob.sh index f92616de12d..54221588dd0 100755 --- a/t/t6018-rev-list-glob.sh +++ b/t/t6018-rev-list-glob.sh @@ -5,7 +5,6 @@ test_description='rev-list/rev-parse --glob' GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME -TEST_PASSES_SANITIZE_LEAK=true . ./test-lib.sh commit () { diff --git a/t/t6021-rev-list-exclude-hidden.sh b/t/t6021-rev-list-exclude-hidden.sh index d08fc2da93d..908b9dba611 100755 --- a/t/t6021-rev-list-exclude-hidden.sh +++ b/t/t6021-rev-list-exclude-hidden.sh @@ -2,7 +2,6 @@ test_description='git rev-list --exclude-hidden test' -TEST_PASSES_SANITIZE_LEAK=true . ./test-lib.sh test_expect_success 'setup' '