Summary

Add a function for iterating through all exe locations in the PATH.

Review Request #12617 — Created Sept. 21, 2022 and submitted Sept. 26, 2022, 1:30 a.m.

Information

Owner

chipx86

Repository

RBTools

Branch

release-4.x

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

This introduces rbtools.utils.filesystem.iter_exes_in_path(), which
looks for the specified executable in each path in $PATH, yielding
each one that's found. This can be used to scan for multiple instances
of an executable, trying to find a specific version.

is_exe_in_path() now wraps this, returning just the first result.

Testing Done

Unit tests pass.

Tested this extensively with the Diff Tool scanning.

Commits

Summary	ID
Add a function for iterating through all exe locations in the PATH. This introduces `rbtools.utils.filesystem.iter_exes_in_path()`, which looks for the specified executable in each path in `$PATH`, yielding each one that's found. This can be used to scan for multiple instances of an executable, trying to find a specific version. `is_exe_in_path()` now wraps this, returning just the first result.	781a3e658616e19f5f9d7ff0085391bae330f1db

Issues

Description	From	Last Updated
Mind killing this while you're in here?	david	Sept. 22, 2022, 12:40 p.m.
executable -> executables	david	Sept. 22, 2022, 12:40 p.m.
Wrapping here seems a little off.	david	Sept. 22, 2022, 12:40 p.m.

flake8 passed.

JSHint passed.

rbtools/utils/filesystem.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Mind killing this while you're in here?
```
rbtools/utils/filesystem.py (Diff revision 1)
The issue has been resolved. Show all issues
```
executable -> executables
```
rbtools/utils/filesystem.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Wrapping here seems a little off.
```

Change Summary:


Removed unicode_literals.
Fixed some wrapping and typos in a docstring.

Commits:

	Summary	ID
	Add a function for iterating through all exe locations in the PATH. This introduces `rbtools.utils.filesystem.iter_exes_in_path()`, which looks for the specified executable in each path in `$PATH`, yielding each one that's found. This can be used to scan for multiple instances of an executable, trying to find a specific version. `is_exe_in_path()` now wraps this, returning just the first result.	64e934d3d5c3949108649d781632b311bd8306d3
	Add a function for iterating through all exe locations in the PATH. This introduces `rbtools.utils.filesystem.iter_exes_in_path()`, which looks for the specified executable in each path in `$PATH`, yielding each one that's found. This can be used to scan for multiple instances of an executable, trying to find a specific version. `is_exe_in_path()` now wraps this, returning just the first result.	781a3e658616e19f5f9d7ff0085391bae330f1db