Skip to content

Run "git submodule sync" during autogen.sh#5172

Merged
graydon merged 1 commit intostellar:masterfrom
graydon:submodule-sync
Mar 16, 2026
Merged

Run "git submodule sync" during autogen.sh#5172
graydon merged 1 commit intostellar:masterfrom
graydon:submodule-sync

Conversation

@graydon
Copy link
Contributor

@graydon graydon commented Mar 11, 2026

This is a minor thing but it bit me recently:

If you change one of the submodule URLs by editing .gitmodules, it doesn't immediately reflect in git's internal notion of submodule URLs -- git keeps a list of submodule URLs separate from .gitmodules and the two need to be "sync'ed" in order to reflect changes in .gitmodules. Goodness knows why, this just seems like a pointless git quirk, but it means that git keeps pulling from the wrong repo until you run "sync".

Copilot AI review requested due to automatic review settings March 11, 2026 01:18
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repo bootstrap process to avoid stale submodule URL configuration by running git submodule sync during autogen.sh, before initializing/updating submodules.

Changes:

  • Run git submodule sync when submodules are not skipped.
  • Keep existing submodule init/update and per-submodule autogen.sh invocation flow unchanged.

leighmcculloch
leighmcculloch previously approved these changes Mar 11, 2026
@graydon graydon enabled auto-merge March 16, 2026 20:56
@leighmcculloch
Copy link
Member

The quickstart build failing does not block this pull request, but fyi it is being fixed here:

@graydon graydon added this pull request to the merge queue Mar 16, 2026
Merged via the queue into stellar:master with commit db97040 Mar 16, 2026
35 of 37 checks passed
@graydon graydon deleted the submodule-sync branch March 16, 2026 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants