Git development
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Jeff King <peff@peff.net>
Cc: "Patrick Steinhardt" <ps@pks.im>,
	git@vger.kernel.org, "Todd Zullinger" <tmz@pobox.com>,
	"SZEDER Gábor" <szeder.dev@gmail.com>,
	"Kaartic Sivaraam" <kaartic.sivaraam@gmail.com>
Subject: Re: [PATCH v4 0/8] ci: some housekeeping and modernizations
Date: Thu, 15 Oct 2026 11:06:10 -0700	[thread overview]
Message-ID: <xmqq4iemon3h.fsf@gitster.g> (raw)
In-Reply-To: <20261015070605.GA2556608@coredump.intra.peff.net> (Jeff King's message of "Thu, 15 Oct 2026 03:06:05 -0400")

Jeff King <peff@peff.net> writes:

> On Thu, Oct 15, 2026 at 08:57:09AM +0200, Patrick Steinhardt wrote:
>
>> Changes in v4:
>>   - Make the test in t5004 work both with SHA1 and SHA256 by forcing the
>>     repo to SHA1.
>>   - Link to v3: https://patch.msgid.link/20261014-pks-ci-housekeeping-v3-0-04c24e1f480d@pks.im
>
> Thanks, the updated patch 1 looks good to me. I think all of my other
> comments on v3 were things that were either non-issues, or things we
> decided to address outside of this series.
>
> So v4 looks good to me.

Yeah, I am especially pleased to see your "what would happen after
SHA-256 becomes the default?" question and updates in response to
it.

Thanks for writing, and reviewing.  Let me mark it for 'next'.

      reply	other threads:[~2026-10-15 18:06 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-08 10:01 [PATCH 0/8] ci: some housekeeping and modernizations Patrick Steinhardt
2026-10-08 10:01 ` [PATCH 1/8] t5004: skip SHA-1-only test in SHA-256 repository Patrick Steinhardt
2026-10-08 18:05   ` Junio C Hamano
2026-10-09  5:46     ` Patrick Steinhardt
2026-10-08 10:01 ` [PATCH 2/8] ci: fix "fedora-breaking-changes-meson" job Patrick Steinhardt
2026-10-08 10:01 ` [PATCH 3/8] ci: drop unused "linux-clang" logic Patrick Steinhardt
2026-10-08 10:01 ` [PATCH 4/8] ci: switch away from unsupported i386/ubuntu image Patrick Steinhardt
2026-10-08 15:03   ` Todd Zullinger
2026-10-08 18:12     ` Junio C Hamano
2026-10-09  5:46       ` Patrick Steinhardt
2026-10-08 10:01 ` [PATCH 5/8] ci: rename linux-TEST-vars job Patrick Steinhardt
2026-10-08 10:01 ` [PATCH 6/8] ci: switch away from EOL'd Ubuntu version in linux-exotic Patrick Steinhardt
2026-10-08 10:01 ` [PATCH 7/8] ci: drop now-dead Python 2 coverage Patrick Steinhardt
2026-10-08 10:01 ` [PATCH 8/8] ci: drop redundant linux-reftable job Patrick Steinhardt
2026-10-08 18:18   ` Junio C Hamano
2026-10-09  5:55     ` Patrick Steinhardt
2026-10-09 20:22       ` Junio C Hamano
2026-10-09 11:31 ` [PATCH v2 0/8] ci: some housekeeping and modernizations Patrick Steinhardt
2026-10-09 11:31   ` [PATCH v2 1/8] t5004: skip SHA-1-only test in SHA-256 repository Patrick Steinhardt
2026-10-09 11:31   ` [PATCH v2 2/8] ci: fix "fedora-breaking-changes-meson" job Patrick Steinhardt
2026-10-09 11:32   ` [PATCH v2 3/8] ci: drop unused "linux-clang" logic Patrick Steinhardt
2026-10-09 11:32   ` [PATCH v2 4/8] ci: switch away from unsupported i386/ubuntu image Patrick Steinhardt
2026-10-09 11:32   ` [PATCH v2 5/8] ci: rename linux-TEST-vars job Patrick Steinhardt
2026-10-09 15:31     ` SZEDER Gábor
2026-10-13  7:21       ` Patrick Steinhardt
2026-10-13 14:47         ` Junio C Hamano
2026-10-14  7:03           ` Patrick Steinhardt
2026-10-09 11:32   ` [PATCH v2 6/8] ci: switch away from EOL'd Ubuntu version in linux-exotic Patrick Steinhardt
2026-10-09 11:32   ` [PATCH v2 7/8] ci: drop now-dead Python 2 coverage Patrick Steinhardt
2026-10-09 11:32   ` [PATCH v2 8/8] ci: improve reftable test coverage Patrick Steinhardt
2026-10-14  8:30 ` [PATCH v3 0/8] ci: some housekeeping and modernizations Patrick Steinhardt
2026-10-14  8:30   ` [PATCH v3 1/8] t5004: skip SHA-1-only test in SHA-256 repository Patrick Steinhardt
2026-10-14  9:07     ` Jeff King
2026-10-14 10:34       ` Patrick Steinhardt
2026-10-14 15:04         ` Kaartic Sivaraam
2026-10-15  5:43           ` Patrick Steinhardt
2026-10-14  8:30   ` [PATCH v3 2/8] ci: fix "fedora-breaking-changes-meson" job Patrick Steinhardt
2026-10-14  9:10     ` Jeff King
2026-10-14 10:35       ` Patrick Steinhardt
2026-10-14 11:07         ` Jeff King
2026-10-14 11:18           ` Patrick Steinhardt
2026-10-14  8:30   ` [PATCH v3 3/8] ci: drop unused "linux-clang" logic Patrick Steinhardt
2026-10-14  8:30   ` [PATCH v3 4/8] ci: switch away from unsupported i386/ubuntu image Patrick Steinhardt
2026-10-14  9:17     ` Jeff King
2026-10-14 10:35       ` Patrick Steinhardt
2026-10-14 11:08         ` Jeff King
2026-10-14  8:30   ` [PATCH v3 5/8] ci: rename linux-TEST-vars job Patrick Steinhardt
2026-10-14  9:17     ` Jeff King
2026-10-14  8:30   ` [PATCH v3 6/8] ci: switch away from EOL'd Ubuntu version in linux-feature-testbed Patrick Steinhardt
2026-10-14  9:18     ` Jeff King
2026-10-14  8:30   ` [PATCH v3 7/8] ci: drop now-dead Python 2 coverage Patrick Steinhardt
2026-10-14  8:30   ` [PATCH v3 8/8] ci: improve reftable test coverage Patrick Steinhardt
2026-10-14  9:21     ` Jeff King
2026-10-14 10:35       ` Patrick Steinhardt
2026-10-14 11:13         ` Jeff King
2026-10-14 11:18           ` Patrick Steinhardt
2026-10-14 13:47             ` Jeff King
2026-10-14  9:22   ` [PATCH v3 0/8] ci: some housekeeping and modernizations Jeff King
2026-10-14 14:28   ` Junio C Hamano
2026-10-15  5:43     ` Patrick Steinhardt
2026-10-15 17:12       ` Junio C Hamano
2026-10-15  6:57 ` [PATCH v4 " Patrick Steinhardt
2026-10-15  6:57   ` [PATCH v4 1/8] t5004: fix test creating an archive with large objects under SHA256 Patrick Steinhardt
2026-10-15  6:57   ` [PATCH v4 2/8] ci: fix "fedora-breaking-changes-meson" job Patrick Steinhardt
2026-10-15  6:57   ` [PATCH v4 3/8] ci: drop unused "linux-clang" logic Patrick Steinhardt
2026-10-15  8:36     ` Kaartic Sivaraam
2026-10-15  9:31       ` Patrick Steinhardt
2026-10-15  6:57   ` [PATCH v4 4/8] ci: switch away from unsupported i386/ubuntu image Patrick Steinhardt
2026-10-15  6:57   ` [PATCH v4 5/8] ci: rename linux-TEST-vars job Patrick Steinhardt
2026-10-15  6:57   ` [PATCH v4 6/8] ci: switch away from EOL'd Ubuntu version in linux-feature-testbed Patrick Steinhardt
2026-10-15  6:57   ` [PATCH v4 7/8] ci: drop now-dead Python 2 coverage Patrick Steinhardt
2026-10-15  6:57   ` [PATCH v4 8/8] ci: improve reftable test coverage Patrick Steinhardt
2026-10-15  7:06   ` [PATCH v4 0/8] ci: some housekeeping and modernizations Jeff King
2026-10-15 18:06     ` Junio C Hamano [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqq4iemon3h.fsf@gitster.g \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=kaartic.sivaraam@gmail.com \
    --cc=peff@peff.net \
    --cc=ps@pks.im \
    --cc=szeder.dev@gmail.com \
    --cc=tmz@pobox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox