Paths

Table of Contentst

Differential D20603

Remove "PhutilDaemonOverseer->findRunningDaemons()"
ClosedPublic
Actions

Authored by epriestley on Jun 20 2019, 9:25 PM.

Tags

None

Referenced Files

	F15476880: D20603.id49138.diff
	Mon, Apr 7, 9:57 AM

	F15428878: D20603.diff
	Sun, Mar 23, 10:57 PM

	F15402680: D20603.diff
	Mon, Mar 17, 11:43 PM

	F15302791: D20603.diff
	Mar 5 2025, 9:48 PM

	Unknown Object (File)
	Mar 3 2025, 6:46 PM

	Unknown Object (File)
	Feb 25 2025, 2:14 PM

	Unknown Object (File)
	Feb 14 2025, 7:23 PM

	Unknown Object (File)
	Feb 10 2025, 5:22 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13321: "bin/phd" may try to kill processes it does not own if the daemon PID logfiles on disk are out of date

Commits

rPHUac8e57402236: Remove "PhutilDaemonOverseer->findRunningDaemons()"

Summary

Ref T13321. This was obsoleted by "PhutilProcessQuery" in D20600, and the only callsite was removed in D20601.

Test Plan

Grepped for findRunningDaemons().

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 20 2019, 9:25 PM

Harbormaster completed remote builds in B23010: Diff 49138.Jun 20 2019, 9:25 PM

epriestley requested review of this revision.Jun 20 2019, 9:25 PM

amckinley accepted this revision.Jun 20 2019, 10:34 PM

This revision is now accepted and ready to land.Jun 20 2019, 10:34 PM

Closed by commit rPHUac8e57402236: Remove "PhutilDaemonOverseer->findRunningDaemons()" (authored by epriestley). · Explain WhyJun 24 2019, 6:23 PM

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPHUac8e57402236: Remove "PhutilDaemonOverseer->findRunningDaemons()".

Revision Contents
Changeset List

Path

Size

src/

daemon/

PhutilDaemonOverseer.php

63 lines

Diff 49169

View Options

src/daemon/PhutilDaemonOverseer.php

Remove "PhutilDaemonOverseer->findRunningDaemons()"ClosedPublicActions