HomePhabricator

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

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

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:

Reviewers: amckinley

Reviewed By: amckinley

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

Details

Committed
epriestleyFeb 25 2019, 6:48 PM
Pushed
epriestleyFeb 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