HomePhabricator

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

Description

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:

Screen Shot 2019-02-25 at 6.16.46 AM.png (60×952 px, 27 KB)

Reviewers: amckinley

Reviewed By: amckinley

Differential Revision: https://secure.phabricator.com/D20210

Details

Provenance
epriestleyAuthored on Feb 25 2019, 2:18 PM
epriestleyPushed on Feb 25 2019, 6:48 PM
Reviewer
amckinley
Differential Revision
D20210: Make the Diffusion warning about "svnlook" and PATH more clear
Parents
rP767afd1780fd: Support an "authorPHIDs" constraint for "transaction.search"
Branches
Unknown
Tags
Unknown
Build Status
Buildable 22116
Build 30214: Run Core Tests