HomePhorge

Use "resolve()", not "execute()", for PhutilExecPassthru callsites in…

Description

Use "resolve()", not "execute()", for PhutilExecPassthru callsites in Phabricator

Summary: Ref T13660. Clean up callsites to "PhutilExecPassthru->execute()" to prepare to deprecate it.

Test Plan:

  • Grepped for "PhutilExecPassthru" and looked for callsites.
  • Ran GIT_SSH=.../ssh-connect git ls-remote origin to execute the "ssh-connect" code.
  • The two passthru future methods have no callers and could possibly be removed, but I'm just letting sleeping dogs lie for now.

Reviewers: cspeckmim

Reviewed By: cspeckmim

Maniphest Tasks: T13660

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

Details

Provenance
epriestleyAuthored on Jul 20 2021, 9:26 PM
themackabuPushed on Tue, Mar 25, 8:07 PM
Parents
rP2f1acf8b10cb: Suppress PHP 8 deprecation warning in startup
Branches
Unknown
Tags
Unknown

Event Timeline