Skip to content

fix(deps): setup clones master, the branch these repos actually have - #99

Merged
srpatcha merged 2 commits into
embeddedos-org:masterfrom
Kartikey1306:fix/setup-clones-master-not-main
Sep 1, 2026
Merged

fix(deps): setup clones master, the branch these repos actually have#99
srpatcha merged 2 commits into
embeddedos-org:masterfrom
Kartikey1306:fix/setup-clones-master-not-main

Conversation

@Kartikey1306

Copy link
Copy Markdown
Contributor

Stacked on #97 and the version-ordering PR. Review the last commit.

ebuild setup — the first command a new developer runs — clones eos and eBoot at the branch named in DEFAULT_CONFIG. Both entries name main. Neither repository has a main:

$ git ls-remote --heads https://github.com/embeddedos-org/eos.git main
$ git ls-remote --heads https://github.com/embeddedos-org/eBoot.git main
$          # both empty; both default to master

So setup fails for every new developer, before any build can run.

efirmware, in the same mapping, already names master — only the two that matter for a first setup were wrong.

tests/unit/test_sources_are_importable.py::test_default_repo_branch_exists_upstream asserts each configured branch exists upstream and has been failing on exactly this, which the suite could not report while collection was broken (#97).

🤖 Generated with Claude Code

`ebuild setup` clones eos and eBoot at the branch named in DEFAULT_CONFIG.
Both entries named "main". Neither repository has a main -- both default
to master -- so the clone failed for every new developer, before any
build could run:

    $ git ls-remote --heads .../eos.git main   # no output

efirmware, in the same mapping, already names master; only the two that
matter for a first `setup` were wrong.

tests/unit/test_sources_are_importable.py asserts each configured branch
exists upstream and has been failing on this, which the suite could not
report while collection was broken.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@srpatcha
srpatcha merged commit 562d28d into embeddedos-org:master Sep 1, 2026
19 of 28 checks passed
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.

2 participants