HomePhorge

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: {F6240967}

Reviewers: amckinley

Reviewed By: amckinley

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

Details

Provenance
epriestleyAuthored on Feb 25 2019, 6:18 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP767afd1780fd: Support an "authorPHIDs" constraint for "transaction.search"
Branches
Unknown
Tags
Unknown

Event Timeline