Page MenuHomePhabricator
Feed Advanced Search

Apr 13 2020

epriestley updated the task description for T13488: Upgrading: Early 2020 Changes to Arcanist.
Apr 13 2020, 1:09 PM · Arcanist, Installing & Upgrading
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 13 2020, 1:08 PM · Arcanist
epriestley updated the task description for T13488: Upgrading: Early 2020 Changes to Arcanist.
Apr 13 2020, 1:06 PM · Arcanist, Installing & Upgrading
epriestley added a comment to T13490: Upgrade all "classic" Arcanist workflows to Toolsets.

I think I'm going to remove these workflows outright:

Apr 13 2020, 12:52 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 13 2020, 12:25 PM · Arcanist
epriestley closed T7616: Arcanist to be able to access site requiring client side SSL certificate as Wontfix.

We currently have no interest in this from customers, so I don't plan to pursue it. If customer interest arises, there's a plausible pathway forward.

Apr 13 2020, 11:49 AM · Arcanist, libphutil
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 13 2020, 11:31 AM · Arcanist

Apr 12 2020

epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 12 2020, 11:15 PM · Arcanist
epriestley updated the task description for T13488: Upgrading: Early 2020 Changes to Arcanist.
Apr 12 2020, 11:12 PM · Arcanist, Installing & Upgrading
epriestley updated the task description for T13488: Upgrading: Early 2020 Changes to Arcanist.
Apr 12 2020, 11:09 PM · Arcanist, Installing & Upgrading
epriestley updated the task description for T13488: Upgrading: Early 2020 Changes to Arcanist.
Apr 12 2020, 10:02 PM · Arcanist, Installing & Upgrading
epriestley updated the task description for T13488: Upgrading: Early 2020 Changes to Arcanist.
Apr 12 2020, 9:40 PM · Arcanist, Installing & Upgrading
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 12 2020, 9:15 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 12 2020, 7:34 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 12 2020, 6:04 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 12 2020, 5:38 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 12 2020, 3:05 PM · Arcanist

Apr 11 2020

epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 11 2020, 11:57 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 11 2020, 10:52 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 11 2020, 10:38 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 11 2020, 10:32 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 11 2020, 10:29 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 11 2020, 8:18 PM · Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 11 2020, 5:40 PM · Arcanist
epriestley updated the task description for T13488: Upgrading: Early 2020 Changes to Arcanist.
Apr 11 2020, 5:17 PM · Arcanist, Installing & Upgrading
epriestley added a comment to T13490: Upgrade all "classic" Arcanist workflows to Toolsets.

A general issue with config is that I'd like to modularize config sources, so you can read config from a service or something if you really want, or define a custom configuration source for other Toolset commands like phage. Previously, sources were hard-coded.

Apr 11 2020, 3:26 PM · Arcanist
epriestley added a comment to T13490: Upgrade all "classic" Arcanist workflows to Toolsets.

T11968 was a very deep hole, but I think I've now dug my way out of it. With those changes, these workflows are now modern:

Apr 11 2020, 2:59 PM · Arcanist
epriestley awarded T11968: Decide the fate of FutureGraph a Hungry Hippo token.
Apr 11 2020, 2:41 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley closed T11968: Decide the fate of FutureGraph as Resolved.

"FutureGraph" is dead. Long live "HardpointEngine".

Apr 11 2020, 2:41 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley closed T11968: Decide the fate of FutureGraph, a subtask of T11954: Improve performance of Arcanist/Conduit/Diffusion, primarily through better caching, as Resolved.
Apr 11 2020, 2:41 PM · Performance, Diffusion, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 11 2020, 1:44 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 11 2020, 1:40 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist

Apr 10 2020

epriestley triaged T13508: In Chrome, the menu bar hover animation can cycle very rapidly if you put your cursor exactly on the bottom as Wishlist priority.
Apr 10 2020, 2:55 PM · Design
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 10 2020, 2:51 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley abandoned D3357: GitHub-style punchcards.

No current plans to pursue this upstream.

Apr 10 2020, 2:32 PM
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 10 2020, 1:06 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a comment to T11968: Decide the fate of FutureGraph.

Generators can't "return" until PHP 7:

Apr 10 2020, 11:43 AM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 10 2020, 11:39 AM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist

Apr 9 2020

epriestley added a revision to T13507: Compress Conduit client requests: Unknown Object (Differential Revision).
Apr 9 2020, 8:27 PM · Performance, Conduit
epriestley added a comment to T13507: Compress Conduit client requests.

PHP has a setting called post_max_size. If a POST request is larger than this size, PHP does not populate $_POST or $_FILES.

Apr 9 2020, 8:22 PM · Performance, Conduit

Apr 8 2020

epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 8 2020, 7:37 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 8 2020, 7:37 PM · Arcanist
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 8 2020, 7:29 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 8 2020, 7:29 PM · Arcanist
epriestley added a revision to T13507: Compress Conduit client requests: Unknown Object (Differential Revision).
Apr 8 2020, 4:21 PM · Performance, Conduit
epriestley claimed T13507: Compress Conduit client requests.
Apr 8 2020, 3:58 PM · Performance, Conduit
epriestley triaged T13507: Compress Conduit client requests as Normal priority.
Apr 8 2020, 3:58 PM · Performance, Conduit
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 8 2020, 3:55 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 8 2020, 3:47 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 8 2020, 3:35 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a comment to T11968: Decide the fate of FutureGraph.

XHPAST currently can't build an AST for $result = yield ..., even though this is a valid construct. This is probably a straightforward fix.

Apr 8 2020, 1:27 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist

Apr 7 2020

epriestley closed T9753: XHPAST no longer parses heredocs and can't build on OS X (Bison 2.3) as Resolved by committing Restricted Diffusion Commit.
Apr 7 2020, 9:29 PM · XHPAST
epriestley closed T4334: Support PHP5.4+ syntax in XHPAST as Resolved.

Much of this is supported now; T13492 has some general followup.

Apr 7 2020, 9:27 PM · XHPAST
epriestley added a revision to T13492: Upgrade Support Infrastructure to Minimum PHP Version 5.5: Unknown Object (Differential Revision).
Apr 7 2020, 9:24 PM · Infrastructure
epriestley added a comment to T13492: Upgrade Support Infrastructure to Minimum PHP Version 5.5.

XHPAST currently can not parse $x = yield $y;.

Apr 7 2020, 9:21 PM · Infrastructure
epriestley added a revision to T13492: Upgrade Support Infrastructure to Minimum PHP Version 5.5: Unknown Object (Differential Revision).
Apr 7 2020, 9:12 PM · Infrastructure
epriestley added a revision to T13492: Upgrade Support Infrastructure to Minimum PHP Version 5.5: Unknown Object (Differential Revision).
Apr 7 2020, 9:11 PM · Infrastructure
epriestley added a revision to T13492: Upgrade Support Infrastructure to Minimum PHP Version 5.5: Unknown Object (Differential Revision).
Apr 7 2020, 9:08 PM · Infrastructure
epriestley added a revision to T13492: Upgrade Support Infrastructure to Minimum PHP Version 5.5: Unknown Object (Differential Revision).
Apr 7 2020, 9:04 PM · Infrastructure
epriestley renamed T9753: XHPAST no longer parses heredocs and can't build on OS X (Bison 2.3) from XHPAST no longer parses heredocs and can't build on OS X (Bison 2.3) to XHPAST no longer parses heredocs and can't build on OS X (Bison 2.3).
Apr 7 2020, 9:03 PM · XHPAST
epriestley added a revision to T9753: XHPAST no longer parses heredocs and can't build on OS X (Bison 2.3): Unknown Object (Differential Revision).
Apr 7 2020, 9:02 PM · XHPAST
epriestley added a comment to T13492: Upgrade Support Infrastructure to Minimum PHP Version 5.5.

This is entangled with T9753.

Apr 7 2020, 2:08 PM · Infrastructure
epriestley added a comment to T13492: Upgrade Support Infrastructure to Minimum PHP Version 5.5.

XHPAST currently can not parse $x = yield $y;. I suspect this was added to the PHP grammar after XHPAST diverged. I'm hopeful the fix is straightforward.

Apr 7 2020, 1:53 PM · Infrastructure
epriestley moved T6925: XHPAST is out of date on Windows from Backlog to Windows on the XHPAST board.
Apr 7 2020, 1:33 PM · Windows, XHPAST
epriestley moved T8653: xhpast generation does not work properly under MinGW if the repo has CRLF line endings from Backlog to Windows on the XHPAST board.
Apr 7 2020, 1:33 PM · XHPAST
epriestley moved T4725: `arc liberate` should throw when encountering (and eventually support) namespaces and traits from Backlog to Future on the XHPAST board.
Apr 7 2020, 1:32 PM · XHPAST, Arcanist
epriestley closed T7301: XHPAST application is not optimized for mobile as Wontfix.

No current plans to pursue this.

Apr 7 2020, 1:32 PM · XHPAST, Mobile
epriestley moved T10971: php lint: catch `"${var}"` vs `"{$var}"` from Backlog to String Variables on the XHPAST board.
Apr 7 2020, 1:32 PM · XHPAST, Lint, Feature Request
epriestley moved T11081: XHPAST doesn't handle complex embedded variables in strings using the same quoting style from Backlog to String Variables on the XHPAST board.
Apr 7 2020, 1:32 PM · XHPAST, Bug Report
epriestley moved T8049: Improve parsing of variables embedded in strings in XHPAST from Backlog to String Variables on the XHPAST board.
Apr 7 2020, 1:32 PM · XHPAST
epriestley triaged T9286: XHPAST fails to recognize inline HTML starting with "<?php" as Wishlist priority.
Apr 7 2020, 1:31 PM · XHPAST
epriestley moved T9286: XHPAST fails to recognize inline HTML starting with "<?php" from Backlog to Future on the XHPAST board.
Apr 7 2020, 1:31 PM · XHPAST
epriestley renamed T8325: XHPASTLinter does not scope parameters to closures correctly from Treat functions with parameters defined in other functions in XHPASTLinter correctly to XHPASTLinter does not scope parameters to closures correctly.
Apr 7 2020, 1:31 PM · XHPAST, Lint
epriestley moved T8325: XHPASTLinter does not scope parameters to closures correctly from Backlog to Future on the XHPAST board.
Apr 7 2020, 1:30 PM · XHPAST, Lint
epriestley closed T9062: Generalize PHPAST application as Wontfix.

No plans to pursue this.

Apr 7 2020, 1:29 PM · XHPAST
epriestley closed T8534: Need a way to retrieve fully qualified class name as Wontfix.

No longer clear what this accomplishes.

Apr 7 2020, 1:29 PM · XHPAST
epriestley closed T6195: Replace XHPAST with a pure PHP implementation as Wontfix.

I don't currently plan to pursue this.

Apr 7 2020, 1:28 PM · XHPAST
epriestley closed T10257: xhpast does not build on ARM hardware as Wontfix.

This is very old and we have no outstanding customer requests for ARM support.

Apr 7 2020, 1:27 PM · XHPAST, Bug Report

Apr 6 2020

epriestley closed T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given" as Resolved.

I deployed D21062, and the three examples in the original report look good now. I checked about a dozen other pages and couldn't find anything else broken; let me know if I missed anything. Thanks for the report!

Apr 6 2020, 7:02 PM · Diviner
epriestley added a revision to T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given": Unknown Object (Differential Revision).
Apr 6 2020, 6:50 PM · Diviner
epriestley added a comment to T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given".

The second issue is that the API still returns a string when there is exactly one @attribute of a given type. This is a bit goofy; PhutilExecChannel is another example case.

Apr 6 2020, 6:47 PM · Diviner
epriestley added a comment to T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given".

I deployed D21061. First two example pages are good now, third one is hitting a different error. New trace is:

Apr 6 2020, 6:45 PM · Diviner
aklapper awarded T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given" a Like token.
Apr 6 2020, 6:43 PM · Diviner
epriestley added a revision to T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given": Unknown Object (Differential Revision).
Apr 6 2020, 6:21 PM · Diviner
epriestley added a comment to T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given".

Docblocks may have multiple copies of the same @attribute directive, like this:

Apr 6 2020, 6:19 PM · Diviner
epriestley claimed T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given".
Apr 6 2020, 6:11 PM · Diviner
epriestley added a comment to T13505: Some Diviner pages on secure.phabricator.com show "RuntimeException: strlen() expects parameter 1 to be string, array given".

Here's the full stack trace:

Apr 6 2020, 6:11 PM · Diviner
epriestley added a revision to T13490: Upgrade all "classic" Arcanist workflows to Toolsets: Unknown Object (Differential Revision).
Apr 6 2020, 6:08 PM · Arcanist
epriestley closed T13499: Make "undefined index" PHP errors throw a RuntimeException as Resolved.

One long-standing warning in a unit test cropped up (D21055) but if this does have far-reaching implications, they don't seem terribly obvious/common.

Apr 6 2020, 2:26 PM · Infrastructure
epriestley closed T13504: "proc_open()" may fail to return a resource on Windows if the command isn't executable as Resolved.

Character encoding issues aren't stable yet (see also T13500) but I think everything else here is.

Apr 6 2020, 2:25 PM · Windows

Apr 5 2020

epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 5 2020, 12:50 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist

Apr 3 2020

epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 3 2020, 7:03 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a revision to T11968: Decide the fate of FutureGraph: Unknown Object (Differential Revision).
Apr 3 2020, 6:56 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist
epriestley added a comment to T11968: Decide the fate of FutureGraph.

XHPAST currently can't build an AST for $result = yield ..., even though this is a valid construct. This is probably a straightforward fix.

Apr 3 2020, 2:38 PM · Diffusion, Performance, Conduit, Infrastructure, Restricted Project, Arcanist

Apr 2 2020

epriestley added a comment to T13500: During startup, guarantee "(string)1.23" is "1.23", not "1,23".

See also T13209.

Apr 2 2020, 11:32 PM · Arcanist
epriestley added a revision to T13209: How To Properly Escape Commands on Windows (A Dark Tragedy): Restricted Differential Revision.
Apr 2 2020, 8:44 PM · Arcanist, Windows
epriestley added a revision to T13504: "proc_open()" may fail to return a resource on Windows if the command isn't executable: Restricted Differential Revision.
Apr 2 2020, 8:44 PM · Windows
epriestley added a revision to T13209: How To Properly Escape Commands on Windows (A Dark Tragedy): Unknown Object (Differential Revision).
Apr 2 2020, 7:40 PM · Arcanist, Windows
epriestley added a revision to T13504: "proc_open()" may fail to return a resource on Windows if the command isn't executable: Unknown Object (Differential Revision).
Apr 2 2020, 7:40 PM · Windows
epriestley added a comment to T13209: How To Properly Escape Commands on Windows (A Dark Tragedy).

See also:

Apr 2 2020, 7:35 PM · Arcanist, Windows