Page MenuHomePhabricator

Make "bin/files" parsing of working set arguments more consistent
ClosedPublic

Authored by epriestley on Jun 24 2019, 10:13 PM.
Tags
None
Referenced Files
F13196145: D20614.diff
Sun, May 12, 11:00 PM
F13184151: D20614.diff
Fri, May 10, 3:49 PM
Unknown Object (File)
Sat, May 4, 5:42 PM
Unknown Object (File)
Fri, May 3, 3:05 AM
Unknown Object (File)
Wed, May 1, 4:58 AM
Unknown Object (File)
Mon, Apr 29, 2:36 PM
Unknown Object (File)
Sun, Apr 28, 7:16 AM
Unknown Object (File)
Fri, Apr 26, 8:15 PM
Subscribers
None

Details

Summary

Fixes T13326. In D20571, I slightly generalized construction of an iterator over a set of files, but missed some code in other "bin/files ..." commands which was also affected.

Today, basically all of these workflows define their own "--all" and "names" flags. Pull these definitions up and implement them more consistently.

Test Plan

Ran multiple different bin/files commands with different combinations of arguments, saw consistent handling of iterator construction.

Diff Detail

Repository
rP Phabricator
Branch
fileparse1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23049
Build 31641: Run Core Tests
Build 31640: arc lint + arc unit