Summary

Don't initialize repositories until after we've loaded TREES.

Review Request #14567 — Created Aug. 21, 2025 and submitted Sept. 4, 2025, 5:22 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

I'm not sure when this regressed (perhaps a bad merge at some point),
but running commands that use repositories were not loading
configuration from the TREES setting first. This meant that if a given
checkout required configuration from that setting in order to properly
initialize the repository, commands would error out.

This change reorders things so we load data from TREES before requiring
the repository to be present.

Testing Done

Successfully used rbt stamp in a checkout that required TREES.

Commits

Summary	ID
Don't initialize repositories until after we've loaded TREES. I'm not sure when this regressed (perhaps a bad merge at some point), but running commands that use repositories were not loading configuration from the `TREES` setting first. This meant that if a given checkout required configuration from that setting in order to properly initialize the repository, commands would error out. This change reorders things so we load data from TREES before requiring the repository to be present. Testing Done: Successfully used `rbt stamp` in a checkout that required `TREES`.	tnumxksxzuskkqwqwmprqvylykokpnrz

Summary

Don't initialize repositories until after we've loaded TREES.

I'm not sure when this regressed (perhaps a bad merge at some point), but running commands that use repositories were not loading configuration from the `TREES` setting first. This meant that if a given checkout required configuration from that setting in order to properly initialize the repository, commands would error out. This change reorders things so we load data from TREES before requiring the repository to be present. Testing Done: Successfully used `rbt stamp` in a checkout that required `TREES`.

tnumxksxzuskkqwqwmprqvylykokpnrz

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Change Summary:

Fix assignment of Command.repository_info and Command.tool

Commits:

	Summary	ID
	Don't initialize repositories until after we've loaded TREES. I'm not sure when this regressed (perhaps a bad merge at some point), but running commands that use repositories were not loading configuration from the `TREES` setting first. This meant that if a given checkout required configuration from that setting in order to properly initialize the repository, commands would error out. This change reorders things so we load data from TREES before requiring the repository to be present. Testing Done: Successfully used `rbt stamp` in a checkout that required `TREES`.	tnumxksxzuskkqwqwmprqvylykokpnrz
	Don't initialize repositories until after we've loaded TREES. I'm not sure when this regressed (perhaps a bad merge at some point), but running commands that use repositories were not loading configuration from the `TREES` setting first. This meant that if a given checkout required configuration from that setting in order to properly initialize the repository, commands would error out. This change reorders things so we load data from TREES before requiring the repository to be present. Testing Done: Successfully used `rbt stamp` in a checkout that required `TREES`.	tnumxksxzuskkqwqwmprqvylykokpnrz

Diff:

Revision 2 (+40 -38)

Show changes

rbtools/commands/base/commands.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (9b3ff25)