Commit 5bce21d
committed
Fixed repos not being cloned
## Bug Investigation Summary
**Root Cause:**
The `repos.json` configuration file was never being loaded into the workspace configuration. This happened because:
1. `load_configs()` explicitly excluded `repos.json` from processing (line 759)
2. `get_workspace_repos()` expected a dictionary with a 'repos' key, but received a list of platform configs
3. This caused the function to return early without cloning any repositories
**Changes Made:**
1. **Modified `load_configs()` function:**
- Now loads `repos.json` if it exists
- Stores the repos data in `globals_config['repos']`
2. **Modified the call to `get_workspace_repos()` in `main()`:**
- Constructs a proper config dictionary with both 'repos' and 'platforms' keys
- Passes this structure to `get_workspace_repos()`
**Verification:**
- The code changes are syntactically correct
- The logic flow now properly loads and passes the repos configuration
- All pre-existing errors in the file are unrelated to these changes
The fix ensures that `repos.json` is properly loaded and all repositories defined in it will be cloned during workspace setup.
Made changes.1 parent 71cef5e commit 5bce21d
1 file changed
Lines changed: 21 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
487 | 487 | | |
488 | 488 | | |
489 | 489 | | |
490 | | - | |
| 490 | + | |
| 491 | + | |
| 492 | + | |
| 493 | + | |
| 494 | + | |
| 495 | + | |
| 496 | + | |
491 | 497 | | |
492 | 498 | | |
493 | 499 | | |
| |||
757 | 763 | | |
758 | 764 | | |
759 | 765 | | |
| 766 | + | |
| 767 | + | |
| 768 | + | |
| 769 | + | |
| 770 | + | |
| 771 | + | |
| 772 | + | |
| 773 | + | |
| 774 | + | |
| 775 | + | |
| 776 | + | |
| 777 | + | |
760 | 778 | | |
761 | 779 | | |
762 | | - | |
763 | 780 | | |
764 | 781 | | |
765 | 782 | | |
| |||
1170 | 1187 | | |
1171 | 1188 | | |
1172 | 1189 | | |
| 1190 | + | |
| 1191 | + | |
1173 | 1192 | | |
1174 | 1193 | | |
1175 | 1194 | | |
| |||
0 commit comments