public inbox for git@vger.kernel.org 
 help / color / mirror / Atom feed
From: Patrick Steinhardt <ps@pks•im>
To: Junio C Hamano <gitster@pobox•com>
Cc: git@vger•kernel.org
Subject: Re: What's cooking in git.git (Oct 2025, #06; Fri, 17)
Date: Tue, 21 Oct 2025 07:25:35 +0200	[thread overview]
Message-ID: <aPcZTzqCn3VeGwIW@pks.im> (raw)
In-Reply-To: <xmqq7bwpz3pf.fsf@gitster.g>

On Mon, Oct 20, 2025 at 02:02:52PM -0700, Junio C Hamano wrote:
> * ar/run-command-hook (2025-10-17) 10 commits
>  - receive-pack: convert receive hooks to hook API
>  - receive-pack: convert update hooks to new API
>  - hooks: allow callers to capture output
>  - run-command: allow capturing of collated output
>  - reference-transaction: use hook API instead of run-command
>  - hook: allow overriding the ungroup option
>  - transport: convert pre-push to hook API
>  - hook: convert 'post-rewrite' hook in sequencer.c to hook API
>  - hook: provide stdin via callback
>  - run-command: add stdin callback for parallelization
> 
>  Use hook API to replace ad-hoc invocation of hook scripts with the
>  run_command() API.
> 
>  Comments?
>  source: <20251017141544.1538542-1-adrian.ratiu@collabora•com>

I'll make sure tho give the current iteration another review today.

> * cc/fast-import-strip-signed-tags (2025-10-13) 5 commits
>  - fast-import: add '--signed-tags=<mode>' option
>  - fast-export: handle all kinds of tag signatures
>  - t9350: properly count annotated tags
>  - lib-gpg: allow tests with GPGSM or GPGSSH prereq first
>  - doc: git-tag: stop focusing on GPG signed tags
> 
>  "git fast-import" is taught to handle signed tags, just like it
>  recently learned to handle signed commits, in different ways.
> 
>  Will merge to 'next'?
>  source: <20251013084857.1646783-1-christian.couder@gmail•com>

Should be ready from my point of view. v2 already addressed all of
my comments, and v3 clarified the documentation in one more case. So I'm
happy with this version.

> * ps/ci-rust (2025-10-14) 8 commits
>  - rust: support for Windows
>  - ci: verify minimum supported Rust version
>  - ci: check for common Rust mistakes via Clippy
>  - rust/varint: add safety comments
>  - ci: check formatting of our Rust code
>  - ci: deduplicate calls to `apt-get update`
>  - Merge branch 'ps/gitlab-ci-windows-improvements' into ps/ci-rust
>  - Merge branch 'ps/rust-balloon' into ps/ci-rust
> 
>  CI improvements to handle the recent Rust integration better.
> 
>  Will merge to 'next'?
>  source: <20251015-b4-pks-ci-rust-v3-0-13810af33bd5@pks•im>

I think it should be ready. The 3rd version only saw a single minor
fixup relative to the 2nd one.

> * ps/remove-packfile-store-get-packs (2025-10-09) 7 commits
>  - packfile: rename `packfile_store_get_all_packs()`
>  - packfile: introduce macro to iterate through packs
>  - packfile: drop `packfile_store_get_packs()`
>  - builtin/grep: simplify how we preload packs
>  - builtin/gc: convert to use `packfile_store_get_all_packs()`
>  - object-name: convert to use `packfile_store_get_all_packs()`
>  - Merge branch 'tb/incremental-midx-part-3.1' into ps/remove-packfile-store-get-packs
>  (this branch uses tb/incremental-midx-part-3.1.)
> 
>  Two slightly different ways to get at "all the packfiles" in API
>  has been cleaned up.
> 
>  Comments?
>  source: <20251009-pks-packfiles-convert-get-all-v2-0-0d73b87ce711@pks•im>

I think this series should be ready once the below series is merged
down. v2 addressed all of Taylor's feedback and saw another positive
review by Justin.

> * tb/incremental-midx-part-3.1 (2025-10-15) 50 commits
>  - builtin/repack.c: clean up unused `#include`s
>  - repack: move `write_cruft_pack()` out of the builtin
>  - repack: move `write_filtered_pack()` out of the builtin
>  - repack: move `pack_kept_objects` to `struct pack_objects_args`
>  - repack: move `finish_pack_objects_cmd()` out of the builtin
>  - builtin/repack.c: pass `write_pack_opts` to `finish_pack_objects_cmd()`
>  - repack: extract `write_pack_opts_is_local()`
>  - repack: move `find_pack_prefix()` out of the builtin
>  - builtin/repack.c: use `write_pack_opts` within `write_cruft_pack()`
>  - builtin/repack.c: introduce `struct write_pack_opts`
>  - repack: 'write_midx_included_packs' API from the builtin
>  - builtin/repack.c: inline packs within `write_midx_included_packs()`
>  - builtin/repack.c: pass `repack_write_midx_opts` to `midx_included_packs`
>  - builtin/repack.c: inline `remove_redundant_bitmaps()`
>  - builtin/repack.c: reorder `remove_redundant_bitmaps()`
>  - repack: keep track of MIDX pack names using existing_packs
>  - builtin/repack.c: use a string_list for 'midx_pack_names'
>  - builtin/repack.c: extract opts struct for 'write_midx_included_packs()'
>  - builtin/repack.c: remove ref snapshotting from builtin
>  - repack: remove pack_geometry API from the builtin
>  - builtin/repack.c: pass 'packdir' to `pack_geometry_remove_redundant()`
>  - builtin/repack.c: pass 'pack_kept_objects' to `pack_geometry_init()`
>  - builtin/repack.c: rename various pack_geometry functions
>  - builtin/repack.c: remove "repack_promisor_objects()" from the builtin
>  - builtin/repack.c: pass "packtmp" to `repack_promisor_objects()`
>  - repack: remove 'generated_pack' API from the builtin
>  - builtin/repack.c: provide pack locations to `generated_pack_install()`
>  - builtin/repack.c: pass "packtmp" to `generated_pack_populate()`
>  - builtin/repack.c: factor out "generated_pack_install"
>  - builtin/repack.c: rename "struct generated_pack_data"
>  - repack: remove 'existing_packs' API from the builtin
>  - builtin/repack.c: avoid unnecessary numeric casts in existing_packs
>  - builtin/repack.c: pass "packdir" when removing packs
>  - repack: remove 'remove_redundant_pack' from the builtin
>  - builtin/repack.c: rename many 'struct existing_packs' functions
>  - repack: remove 'prepare_pack_objects' from the builtin
>  - repack: move 'delta_base_offset' to 'struct pack_objects_args'
>  - builtin/repack.c: pass both pack_objects args to repack_config
>  - repack: introduce new compilation unit
>  - builtin/repack.c: avoid using `hash_to_hex()` in pack geometry
>  - builtin/repack.c: avoid "the_hash_algo" in `finish_pack_objects_cmd()`
>  - builtin/repack: avoid "the_hash_algo" in `repack_promisor_objects()`
>  - builtin/repack.c: avoid "the_hash_algo" in `write_oid()`
>  - builtin/repack.c: avoid "the_hash_algo" when deleting packs
>  - builtin/repack.c: avoid "the_repository" when repacking promisor objects
>  - builtin/repack.c: avoid "the_repository" when removing packs
>  - builtin/repack.c: avoid "the_repository" when taking a ref snapshot
>  - builtin/repack.c: avoid "the_repository" in existing packs API
>  - builtin/repack.c: avoid "the_repository" in `cmd_repack()`
>  - Merge branch 'ps/packfile-store' into tb/incremental-midx-part-3.1
>  (this branch is used by ps/remove-packfile-store-get-packs.)
> 
>  Clean-up "git repack" machinery to prepare for incremental update
>  of midx files.
> 
>  Will merge to 'next'?
>  source: <cover.1760567210.git.me@ttaylorr•com>

Yeah, I think this one is ready to go.

Thanks!

Patrick

  reply	other threads:[~2025-10-21  5:25 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-20 21:02 What's cooking in git.git (Oct 2025, #06; Fri, 17) Junio C Hamano
2025-10-21  5:25 ` Patrick Steinhardt [this message]
2025-10-21 21:48   ` Junio C Hamano

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=aPcZTzqCn3VeGwIW@pks.im \
    --to=ps@pks$(echo .)im \
    --cc=git@vger$(echo .)kernel.org \
    --cc=gitster@pobox$(echo .)com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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