Paths

Table of Contentst

Differential D21703

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

Authored by epriestley on Jul 21 2021, 4:54 PM.

Tags

None

Referenced Files

	F13529931: D21703.diff
	Sat, Jul 27, 12:11 AM

	Unknown Object (File)
	Sun, Jul 21, 6:27 AM

	Unknown Object (File)
	Fri, Jul 12, 9:14 PM

	Unknown Object (File)
	Thu, Jul 11, 9:53 PM

	Unknown Object (File)
	Thu, Jul 11, 10:37 AM

	Unknown Object (File)
	Wed, Jul 10, 8:43 PM

	Unknown Object (File)
	Mon, Jul 8, 12:44 AM

	Unknown Object (File)
	Thu, Jul 4, 10:28 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()"

Commits

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

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.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jul 21 2021, 4:54 PM

Harbormaster completed remote builds in B25492: Diff 51689.Jul 21 2021, 4:56 PM

epriestley requested review of this revision.Jul 21 2021, 4:56 PM

epriestley mentioned this in D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".Jul 21 2021, 5:11 PM

cspeckmim accepted this revision.Jul 21 2021, 5:18 PM

This revision is now accepted and ready to land.Jul 21 2021, 5:18 PM

Closed by commit rPaf6cc0c934c4: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in… (authored by epriestley). · Explain WhyJul 21 2021, 5:21 PM

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPaf6cc0c934c4: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in….

epriestley mentioned this in rARC8bb7d5889050: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".

Revision Contents
Changeset List

Path

Size

scripts/

ssh/

ssh-connect.php

2 lines

src/

applications/

repository/

storage/

PhabricatorRepository.php

4 lines

Diff 51692

scripts/ssh/ssh-connect.php

Loading...

src/applications/repository/storage/PhabricatorRepository.php

Loading...