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

	F22636424: D20603.id.diff
	Wed, Sep 30, 7:33 AM

	Unknown Object (File)
	Jul 9 2026, 4:43 PM

	Unknown Object (File)
	Jul 3 2026, 12:02 PM

	Unknown Object (File)
	Jun 25 2026, 10:53 PM

	Unknown Object (File)
	Jun 25 2026, 10:53 PM

	Unknown Object (File)
	Jun 23 2026, 7:30 AM

	Unknown Object (File)
	Jun 22 2026, 11:44 AM

	Unknown Object (File)
	Jun 22 2026, 11:44 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