Merge lp://staging/~manishsinha/filestore/fix-sphinx-hardcoding-in-usrbin into lp://staging/filestore
Proposed by
Manish Sinha (मनीष सिन्हा)
| Status: | Merged |
|---|---|
| Merged at revision: | 189 |
| Proposed branch: | lp://staging/~manishsinha/filestore/fix-sphinx-hardcoding-in-usrbin |
| Merge into: | lp://staging/filestore |
| Diff against target: |
32 lines (+11/-2) 1 file modified
setup.py (+11/-2) |
| To merge this branch: | bzr merge lp://staging/~manishsinha/filestore/fix-sphinx-hardcoding-in-usrbin |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jason Gerard DeRose | Approve | ||
|
Review via email:
|
|||
Description of the change
Fixed bugs #847044 where sphinx-build was hardcoded at /usr/bin/
Read the $PATH environment variable. Split it by : character. For every entry append
'sphinx-build' and check whether the file exists. If the file exists, check if it
is executable. If yes, then return the path. If none is found then return None.
The method calling this search method has to handle None return by showing a WARNING
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Rock on, thanks!