-
-
This is actually a bit of a regression from where I think we want to go.
is_supported
was factored out intoP4Wrapper
so that unit tests could override it, allowing forPerforceClient
tests to run without needing Perforce.Can we keep
is_supported
and just add the logging statement?Down the road, I'd like to move toward more abstractions like
P4Wrapper
so that tests can be faster when they only care about processing known results.
Add some logging when check_install fails.
Review Request #5625 — Created March 14, 2014 and submitted
Right now, if the command-line tools aren't available in the path, rbtools will
just skip that backend silently. This change adds some debug logging that
indicates which backends are skipped due to missing dependencies.
Ran unit tests.
Description | From | Last Updated |
---|---|---|
This is actually a bit of a regression from where I think we want to go. is_supported was factored out … |
chipx86 |