Page MenuHomePhabricator

Fix callsites which called libphutil_console_wrap like it were _format
ClosedPublic

Authored by alexmv on Sep 3 2015, 6:43 PM.
Tags
None
Referenced Files
F18250087: D14053.id.diff
Thu, Aug 21, 3:27 PM
F18235152: D14053.diff
Wed, Aug 20, 7:09 PM
F18193434: D14053.id33978.diff
Sun, Aug 17, 5:26 AM
F18085165: D14053.id.diff
Tue, Aug 5, 7:01 PM
F18083143: D14053.id33979.diff
Tue, Aug 5, 9:03 AM
F18081243: D14053.id33978.diff
Mon, Aug 4, 11:10 PM
F18060779: D14053.id.diff
Mon, Aug 4, 11:56 AM
F18005351: D14053.diff
Sat, Aug 2, 12:59 PM
Subscribers

Details

Summary

7e2df9a attempted to pht() some strings; unfortunately, it assumed
that some things that were calls to phutil_console_wrap() were
actually calls to phutil_console_format(). This produces errors of
the form:

[2015-07-17 21:17:28] ERROR 2: str_repeat() expects parameter 2 to be long, string given at [/usr/local/libphutil/src/console/format.php:162]
#0 str_repeat(string, string) called at [<phutil>/src/console/format.php:162]
#1 phutil_console_wrap(string, string, string) called at [<arcanist>/scripts/arcanist.php:620]
#2 arcanist_load_libraries(array, boolean, string, ArcanistWorkingCopyIdentity) called at [<arcanist>/scripts/arcanist.php:154]
%s: %s

Provide an additional call to phutil_console_format() when necessary,
or simply append the relevant characters if possible.

Test Plan

Caused a library load error

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

alexmv retitled this revision from to Fix callsites which called libphutil_console_wrap like it were _format.
alexmv updated this object.
alexmv edited the test plan for this revision. (Show Details)
alexmv added a reviewer: epriestley.

I believe Dropbox has now signed the Corporate CLA, which should suffice to cover this submission.

This revision is now accepted and ready to land.Sep 3 2015, 7:01 PM
This revision was automatically updated to reflect the committed changes.