From: Jacob Keller Date: Mon, 02 Mar 2026 22:09:18 GMT Subject: Re: [PATCH v4] submodule: fetch missing objects from default remote Message-ID: In-Reply-To: <20260301025327.3845292-1-nasser.grainawi@oss.qualcomm.com> On Sat, Feb 28, 2026 at 6:53 PM Nasser Grainawi wrote: > > When be76c21282 (fetch: ensure submodule objects fetched, 2018-12-06) > added support for fetching a missing submodule object by id, it > hardcoded the remote name as "origin" and deferred anything more > complicated for a later patch. Implement the NEEDSWORK item to remove > the hardcoded assumption by adding and using a submodule helper subcmd > 'get-default-remote'. Fixing this lets 'git fetch --recurse-submodules' > succeed when the fetched commit(s) in the superproject trigger a > submodule fetch, and that submodule's default remote name is not > "origin". > > Add non-"origin" remote tests to t5526-fetch-submodules.sh and > t5572-pull-submodule.sh demonstrating this works as expected and add > dedicated tests for get-default-remote. > > Signed-off-by: Nasser Grainawi > --- > I removed Jacob Keller's Reviewed-By since there are more significant > edits to the tests in v4 that they haven't reviewed. > > v4 includes fixes for the issues Junio patched in seen and I confirmed > it merges cleanly to seen (with v3 reverted) as well as next and passes > tests. > v4 looks good, thanks! Reviewed-by: Jacob Keller