Re: [PATCH v4] submodule: fetch missing objects from default remote
- From
Jacob Keller <jacob.keller@gmail.com>
- Date
- Mar 2, 2026, 22:09 UTC
- Message-ID
- <CA+P7+xqC10D9a3zp7JO_yoK9Vjpg+xua7yH=fGXjApUYisTf4w@mail.gmail.com>
- In-Reply-To
- <20260301025327.3845292-1-nasser.grainawi@oss.qualcomm.com>
On Sat, Feb 28, 2026 at 6:53 PM Nasser Grainawi <nasser.grainawi@oss.qualcomm.com> wrote:
Show 24 quoted lines
> > 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 <nasser.grainawi@oss.qualcomm.com> > --- > 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 <jacob.keller@gmail.com>