Add dependency checks for CVS.

Review Request #12547 — Created Aug. 18, 2022 and submitted — Latest diff uploaded

Information

RBTools
release-4.x

Reviewers

This updates CVSClient to add formal support for dependency checks.
CVSClient.check_dependencies() now determines if cvs is installed.

CVSClient.get_local_path() now calls this instead of checking for
dependencies itself, emitting a warning if dependencies haven't already
been checked via setup() or has_dependencies().

Unit tests have been updated to include new dependency-related tests. If
cvs isn't installed, tests that depend on it will be automatically
skipped.

Unit tests pass on Python 3.7-3.11.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Add dependency checks for CVS.
This updates `CVSClient` to add formal support for dependency checks. `CVSClient.check_dependencies()` now determines if `cvs` is installed. `CVSClient.get_local_path()` now calls this instead of checking for dependencies itself, emitting a warning if dependencies haven't already been checked via `setup()` or `has_dependencies()`. Unit tests have been updated to include new dependency-related tests. If `cvs` isn't installed, tests that depend on it will be automatically skipped.
697eb111cb1b26c139e3073852974729d38ba343 Christian Hammond
rbtools/clients/cvs.py
rbtools/clients/tests/test_cvs.py
Loading...