R1:f61e825905c1
R1:f61e825905c1
Make the Diffusion warning about "svnlook" and PATH more clear
Summary:
See <https://discourse.phabricator-community.org/t/display-error-on-the-status-page-for-svn-repos/2443> for discussion.
The UI currently shows a misleading warning that looks like "found svnlook; can't find svnlook".
It actually means "found svnlook, but when Subversion wipes PATH before executing commit hooks, we will no longer be able to find it".
Test Plan: {F6240967}
Reviewers: amckinley
Reviewed By: amckinley
Differential…
Summary:
See <https://discourse.phabricator-community.org/t/display-error-on-the-status-page-for-svn-repos/2443> for discussion.
The UI currently shows a misleading warning that looks like "found svnlook; can't find svnlook".
It actually means "found svnlook, but when Subversion wipes PATH before executing commit hooks, we will no longer be able to find it".
Test Plan: {F6240967}
Reviewers: amckinley
Reviewed By: amckinley
Differential…
Repository: R1 hydra
Commit Date: Feb 25 2019