Fix a couple bad fprintf() method calls in Phage
ClosedPublic
Actions

Authored by epriestley on Feb 19 2017, 2:54 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Apr 19, 6:17 PM

	Unknown Object (File)
	Wed, Apr 17, 7:45 AM

	Unknown Object (File)
	Sun, Apr 14, 9:52 AM

	Unknown Object (File)
	Thu, Apr 11, 9:30 AM

	Unknown Object (File)
	Sun, Apr 7, 10:30 AM

	Unknown Object (File)
	Sat, Apr 6, 12:42 AM

	Unknown Object (File)
	Fri, Apr 5, 9:53 AM

	Unknown Object (File)
	Fri, Apr 5, 9:15 AM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T2794: Implement Phage (like Hypershell)

Commits

rPHU2e26268fb854: Fix a couple bad fprintf() method calls in Phage

Summary

Ref T2794. These currently don't do the right thing if you run a command that prints "%" chacters.

Test Plan

Ran some command which outout "%" characters, no more warnings.

Diff Detail

Repository

rPHU libphutil

Branch

phage1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 15702
Build 20741: Run Core Tests
Build 20740: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 19 2017, 2:54 PM

Harbormaster completed remote builds in B15702: Diff 41805.Feb 19 2017, 2:54 PM

chad accepted this revision.Feb 19 2017, 5:27 PM

This revision is now accepted and ready to land.Feb 19 2017, 5:27 PM

Closed by commit rPHU2e26268fb854: Fix a couple bad fprintf() method calls in Phage (authored by epriestley, committed by epriestley). · Explain WhyFeb 19 2017, 6:54 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

phage/

action/

PhageAgentAction.php

4 lines

Diff 41805

View Options

src/phage/action/PhageAgentAction.php

Fix a couple bad fprintf() method calls in PhageClosedPublicActions