Page MenuHomePhabricator

Make the Diffusion warning about "svnlook" and PATH more clear
ClosedPublic

Authored by epriestley on Feb 25 2019, 2:20 PM.

Details

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

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Feb 25 2019, 2:20 PM
epriestley requested review of this revision.Feb 25 2019, 2:22 PM
amckinley accepted this revision.Feb 25 2019, 3:08 PM
This revision is now accepted and ready to land.Feb 25 2019, 3:08 PM
This revision was automatically updated to reflect the committed changes.