Return STDIN, STDOUT, and STDERR file descriptors from parent process
ClosedPublic
Actions

Authored by epriestley on May 5 2022, 5:27 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Nov 23, 7:19 AM

	Unknown Object (File)
	Tue, Nov 19, 3:44 AM

	Unknown Object (File)
	Fri, Nov 15, 2:54 AM

	Unknown Object (File)
	Thu, Nov 14, 4:03 AM

	Unknown Object (File)
	Wed, Nov 13, 5:37 AM

	Unknown Object (File)
	Mon, Nov 11, 7:54 AM

	Unknown Object (File)
	Thu, Nov 7, 10:56 AM

	Unknown Object (File)
	Thu, Nov 7, 9:06 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13675: Can Arcanist plausibly be compiled into a binary?
Commits: rARCfc5b228db537: Return STDIN, STDOUT, and STDERR file descriptors from parent process

Summary

Ref T13675. When a process daemonizes, it needs to close these actual file descriptors, so the calling context must provide them.

Elsewhere, modify the embedded "arc" to provide them. Then use them in place of reopening the streams.

Test Plan

Ran locally with embedded "arc", will deploy for production hangs.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley requested review of this revision.May 5 2022, 5:27 PM

epriestley created this revision.

Harbormaster completed remote builds in B25699: Diff 51971.May 5 2022, 5:27 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 5 2022, 5:27 PM

Closed by commit rARCfc5b228db537: Return STDIN, STDOUT, and STDERR file descriptors from parent process (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCfc5b228db537: Return STDIN, STDOUT, and STDERR file descriptors from parent process.

Revision Contents
Changeset List

Path

Size

src/

utils/

PhutilSystem.php

17 lines

Diff 51972

View Options

src/utils/PhutilSystem.php

Return STDIN, STDOUT, and STDERR file descriptors from parent processClosedPublicActions