HomePhorge

Update "arc liberate" to fix error with PHP 8 and add "--verbose" argument to…

Description

Update "arc liberate" to fix error with PHP 8 and add "--verbose" argument to adjust it

Summary:
Reported at the phorge project (https://we.phorge.it/D25017), running arc liberate fails on PHP 8 due to the log() function using fwrite() incorrectly assuming a format pattern can be used.

This updates to remove most of these status messages are they are largely uninformative and instead we can report progress.

  • Remove the --quiet argument
  • Always display the progress
  • Remove all informational/status log statements

Test Plan:
Tested using both PHP 7.3 and PHP 8:

  1. I ran arc liberate and saw the standard output:
 SCAN  Searching for libraries in the current working directory...
 WORK  Updating library: src/
Done.
 DONE  Updated library.
  1. I ran deleted phabricator/src/.phutil_module_cache and ran arc liberate /src, verifying that progress was displayed while the map was computed.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D21718

Details

Provenance
speckAuthored on Sep 4 2021, 16:59
aviveyPushed on Jul 26 2022, 15:04
aviveyPushed on Jul 26 2022, 15:03
Differential Revision
D25039: merge phab/master -> phorge/master
Parents
rARCf993b1fbda71: Provide "MethodCallFuture" to fix exception semantics in mixed-future contexts
Branches
Unknown
Tags
Unknown
Tasks
T15094: Catch up the master branch to upstream