Page MenuHomePhabricator
Feed All Stories

Thu, May 30

epriestley created T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.
Thu, May 30, 7:09 PM · Transactions
epriestley added a comment to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.

This likely affects other similar dialogs. The fix is to duct-tape something in Quicksand to something in the "click an area to click a link" behavior, most likely.

Thu, May 30, 7:07 PM · Quicksand
epriestley added a comment to D20561: Add a "View Task" button to HTML mail from Maniphest.

Oh, sure, that's pretty reasonable.

Thu, May 30, 7:07 PM
D20561: Add a "View Task" button to HTML mail from Maniphest is now accepted and ready to land.

Shouldn't DifferentialTransactionEditor->buildMailBody() get an update to use this new API?

Thu, May 30, 7:06 PM
epriestley created T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.
Thu, May 30, 7:05 PM · Quicksand
D20560: Clean up "phabricator.timezone" configuration instructions a little bit is now accepted and ready to land.
Thu, May 30, 7:03 PM
epriestley updated the summary of D20559: Render timezone names more readably, with spaces rather than underscores ("America/Los Angeles", not "America/Los_Angeles")..
Thu, May 30, 7:03 PM
D20559: Render timezone names more readably, with spaces rather than underscores ("America/Los Angeles", not "America/Los_Angeles"). is now accepted and ready to land.

¯\_(ツ)_/¯

Thu, May 30, 7:00 PM
amckinley updated the test plan for D20559: Render timezone names more readably, with spaces rather than underscores ("America/Los Angeles", not "America/Los_Angeles")..
Thu, May 30, 7:00 PM
asherkin added a comment to T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled.

The changelog for PHP 7.3.6 mentions "Fixed possible crashes, because of inconsistent PCRE cache and opcache SHM reset." which might be relevant, but no bug link so who knows.

Thu, May 30, 6:52 PM · Infrastructure
epriestley requested review of D20561: Add a "View Task" button to HTML mail from Maniphest.
Thu, May 30, 5:16 PM
epriestley created T13301: Browser "Find..." feature isn't very useful in Safari on Workboards.
Thu, May 30, 3:29 PM · Workboards
epriestley updated the summary of D20559: Render timezone names more readably, with spaces rather than underscores ("America/Los Angeles", not "America/Los_Angeles")..
Thu, May 30, 3:20 PM
epriestley added a comment to D20560: Clean up "phabricator.timezone" configuration instructions a little bit.

Thu, May 30, 3:19 PM
epriestley requested review of D20560: Clean up "phabricator.timezone" configuration instructions a little bit.
Thu, May 30, 3:18 PM
epriestley requested review of D20559: Render timezone names more readably, with spaces rather than underscores ("America/Los Angeles", not "America/Los_Angeles")..
Thu, May 30, 3:09 PM

Tue, May 28

epriestley edited the content of Changelog.
Tue, May 28, 5:29 PM
epriestley committed rPHU0d206344cfd3: (stable) Promote 2019 Week 21 (authored by epriestley).
(stable) Promote 2019 Week 21
Tue, May 28, 5:28 PM
Herald added 1 auditor(s) for rARCbbbd1502f078: (stable) Promote 2019 Week 21: Cowboy Commits.
(stable) Promote 2019 Week 21
Tue, May 28, 5:28 PM
Herald added 1 auditor(s) for rPd9b41d3a0f87: (stable) Promote 2019 Week 21: Cowboy Commits.
(stable) Promote 2019 Week 21
Tue, May 28, 5:27 PM
epriestley created an object: 2019 Week 21 (Very Late May).
Tue, May 28, 5:27 PM
epriestley triaged T13300: When a "Draft" revision is closed into "Published" (not "Abandoned"), it should immediately promote as Low priority.
Tue, May 28, 5:22 PM · Diffusion, Differential
epriestley created T13300: When a "Draft" revision is closed into "Published" (not "Abandoned"), it should immediately promote.
Tue, May 28, 5:22 PM · Diffusion, Differential
amckinley awarded D20558: Test for "CAN_INTERACT" on comment edits in a way that survives objects which only implement "CAN_VIEW" a Manufacturing Defect? token.
Tue, May 28, 5:20 PM
epriestley committed rP53b9acfb7d58: Test for "CAN_INTERACT" on comment edits in a way that survives objects which… (authored by epriestley).
Test for "CAN_INTERACT" on comment edits in a way that survives objects which…
Tue, May 28, 5:14 PM
epriestley added a revision to T13289: Plans: 2019 Week 21-23 Bonus Content: D20558: Test for "CAN_INTERACT" on comment edits in a way that survives objects which only implement "CAN_VIEW".
Tue, May 28, 5:14 PM · Plans

Mon, May 27

epriestley committed rPHU86ee6e90797c: If "vi" or "vim" exit with an error code, test for error-on-interactive-mistake… (authored by epriestley).
If "vi" or "vim" exit with an error code, test for error-on-interactive-mistake…
Mon, May 27, 11:49 PM
epriestley closed D20556: If "vi" or "vim" exit with an error code, test for error-on-interactive-mistake behavior.
Mon, May 27, 11:49 PM
joshuaspence added a comment to D20557: Allow linter to be passed into `ArcanistLinterTestCase::executeTestsInDirectory`.

This is the last of my linter improvement diffs for a while.

Mon, May 27, 9:57 PM
joshuaspence requested review of D20557: Allow linter to be passed into `ArcanistLinterTestCase::executeTestsInDirectory`.
Mon, May 27, 9:51 PM
epriestley moved T13166: Herald "block" and "send an email" actions are mutually exclusive, which isn't intuitive from Far Future to Next on the Herald board.
Mon, May 27, 3:36 PM · Diffusion, Herald
epriestley added a comment to T13166: Herald "block" and "send an email" actions are mutually exclusive, which isn't intuitive.

A sort of broad issue here is that Herald sometimes knows (or could know, or could guess, or maybe could speculate) that a rule won't do what you expect, but it doesn't tell you.

Mon, May 27, 3:36 PM · Diffusion, Herald
epriestley triaged T13299: Herald only triggers for commits once, during the initial publish phase as Normal priority.
Mon, May 27, 3:34 PM · Diffusion, Herald
epriestley closed T13283: When Herald acts, include the transaction group as part of the "state" it acts upon as Resolved.

I think this is about as good as we're going to get, and we've only taken a very small step toward the precipice of a self-aware Herald that hates humankind.

Mon, May 27, 2:56 PM · Herald
epriestley triaged T13298: Improve Herald profiling information available in transcripts as Normal priority.
Mon, May 27, 2:54 PM · Herald
epriestley closed T12257: Add an "author email" field to the "Differential Diff" Herald rule as Wontfix.

KDE appears to be moving to GitLab (see: https://gitlab.com/gitlab-org/gitlab-ce/issues/53206) and we haven't seen this request from other installs, so I'm just going to close this out.

Mon, May 27, 2:38 PM · KDE, Herald, Differential
epriestley closed T8804: Replacing a Herald rule with a Doorkeeper extension as Wontfix.

I think the modern answer here is "use Webhooks". They may not do everything you want if you're writing a chat bot (notably, they intentionally do not currently provide a human-readable text representation of transactions) but there generally suitable for publishing changes to Phabricator objects into a remote system and will produce a program with generally reasonable behaviors and no weird demons lurking under the surface.

Mon, May 27, 2:30 PM · Restricted Project, Herald, Doorkeeper
Herald updated subscribers of T7961: In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules.
Mon, May 27, 2:15 PM · Applications, Herald
Herald updated subscribers of T8952: Hide "Herald added subscribers..." stories from feed completely.
Mon, May 27, 2:15 PM · Feed, Herald

Sun, May 26

joshuaspence added inline comments to D20555: Add a hook method that is called before `ArcanistLinter::getLinterConfigurationOptions`.
Sun, May 26, 10:19 PM

Sat, May 25

amckinley accepted D20556: If "vi" or "vim" exit with an error code, test for error-on-interactive-mistake behavior.
Sat, May 25, 3:41 PM

Fri, May 24

epriestley added a comment to T13297: "vi" and "vim" may exit with errors even though edits succeed.

Here's a more concise test:

Fri, May 24, 9:10 PM · Arcanist
avivey awarded D20556: If "vi" or "vim" exit with an error code, test for error-on-interactive-mistake behavior a Baby Tequila token.
Fri, May 24, 9:04 PM
avivey added a comment to T13297: "vi" and "vim" may exit with errors even though edits succeed.

Running the thing on my ubuntu exists with 0...

$ vi --version
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Apr 10 2018 21:31:58)
Included patches: 1-1453
.....
.....
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H   -Wdate-time  -g -O2 -fdebug-prefix-map=/build/vim-NQEcoP/vim-8.0.1453=. -fstack-protector-strong -Wformat -Werror=format-security -DTINY_VIMRC -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc   -Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -o vim    -lSM -lICE -lXpm -lXt -lX11 -lXdmcp -lSM -lICE  -lm -ltinfo  -lselinux -lacl -lattr -ldl
Fri, May 24, 9:04 PM · Arcanist
epriestley requested review of D20556: If "vi" or "vim" exit with an error code, test for error-on-interactive-mistake behavior.
Fri, May 24, 8:35 PM
epriestley triaged T13297: "vi" and "vim" may exit with errors even though edits succeed as Low priority.
Fri, May 24, 8:15 PM · Arcanist
Herald updated subscribers of T12402: Report on team velocity.
Fri, May 24, 5:59 PM · Feature Request
Herald updated subscribers of T6004: When moving a wiki page with subpages prompt to also move subpages.
Fri, May 24, 1:54 PM · Phriction
trevorsummerssmith awarded T6004: When moving a wiki page with subpages prompt to also move subpages a Like token.
Fri, May 24, 1:54 PM · Phriction
thuffir added a comment to T4752: Daemons may not have permission to remove files from local disk storage engine.

setfacl -Rdm u:phab-phd:rw /var/local/phabricator/filestore
setfacl -Rm u:phab-phd:rw /var/local/phabricator/filestore

Fri, May 24, 9:21 AM · Files
joshuaspence requested review of D20555: Add a hook method that is called before `ArcanistLinter::getLinterConfigurationOptions`.
Fri, May 24, 4:45 AM
epriestley committed rPce6fc5be9092: Fix a looping workflow when trying to submit a partially-effectless transaction… (authored by epriestley).
Fix a looping workflow when trying to submit a partially-effectless transaction…
Fri, May 24, 2:16 AM
epriestley closed D20552: Fix a looping workflow when trying to submit a partially-effectless transaction group.
Fri, May 24, 2:16 AM
epriestley committed rP719dd6d3f442: Remove the "search_documentfield" table (authored by epriestley).
Remove the "search_documentfield" table
Fri, May 24, 2:11 AM
epriestley closed D20549: Remove the "search_documentfield" table.
Fri, May 24, 2:11 AM
Herald updated subscribers of T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud).
Fri, May 24, 2:11 AM
epriestley committed rPaacc62463d61: Prevent editing and deleting comments in locked conversations (authored by epriestley).
Prevent editing and deleting comments in locked conversations
Fri, May 24, 2:05 AM
epriestley closed D20551: Prevent editing and deleting comments in locked conversations.
Fri, May 24, 2:05 AM
epriestley committed rPf838ad182753: Fix two straggling pagination issues in Drydock (authored by epriestley).
Fix two straggling pagination issues in Drydock
Fri, May 24, 1:50 AM
epriestley closed D20553: Fix two straggling pagination issues in Drydock.
Fri, May 24, 1:50 AM
epriestley committed rPHU1422024de70d: Disable PHP config option "pcre.jit" to work around PCRE segfault after "fork()" (authored by epriestley).
Disable PHP config option "pcre.jit" to work around PCRE segfault after "fork()"
Fri, May 24, 1:39 AM
epriestley closed D20554: Disable PHP config option "pcre.jit" to work around PCRE segfault after "fork()".
Fri, May 24, 1:39 AM
joshuaspence updated the diff for D20523: Add a hook method that is called after `ArcanistLinter::setEngine`.

Rebase

Fri, May 24, 12:22 AM

Thu, May 23

amckinley accepted D20554: Disable PHP config option "pcre.jit" to work around PCRE segfault after "fork()".
Thu, May 23, 11:58 PM
epriestley lowered the priority of T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled from Normal to Low.
Thu, May 23, 11:54 PM · Infrastructure
epriestley added a comment to T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled.

I built my PHP from source in T13232 so this is presumably not a Homebrew issue.

Thu, May 23, 11:49 PM · Infrastructure
epriestley updated the task description for T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled.
Thu, May 23, 11:42 PM · Infrastructure
epriestley requested review of D20554: Disable PHP config option "pcre.jit" to work around PCRE segfault after "fork()".
Thu, May 23, 11:36 PM
epriestley added a revision to T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled: D20554: Disable PHP config option "pcre.jit" to work around PCRE segfault after "fork()".
Thu, May 23, 11:36 PM · Infrastructure
epriestley triaged T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled as Normal priority.
Thu, May 23, 11:29 PM · Infrastructure
joshuaspence updated the diff for D20526: Make it easier to parse external linter versions.

Minor

Thu, May 23, 10:20 PM
joshuaspence updated the summary of D20526: Make it easier to parse external linter versions.
Thu, May 23, 10:10 PM
amckinley accepted D20551: Prevent editing and deleting comments in locked conversations.
Thu, May 23, 8:16 PM
amckinley accepted D20552: Fix a looping workflow when trying to submit a partially-effectless transaction group.
Thu, May 23, 8:11 PM
amckinley accepted D20553: Fix two straggling pagination issues in Drydock.
Thu, May 23, 8:10 PM
epriestley updated the task description for T13289: Plans: 2019 Week 21-23 Bonus Content.
Thu, May 23, 7:03 PM · Plans
epriestley requested review of D20553: Fix two straggling pagination issues in Drydock.
Thu, May 23, 6:56 PM
epriestley added a revision to T13289: Plans: 2019 Week 21-23 Bonus Content: D20553: Fix two straggling pagination issues in Drydock.
Thu, May 23, 6:55 PM · Plans
epriestley requested review of D20552: Fix a looping workflow when trying to submit a partially-effectless transaction group.
Thu, May 23, 6:46 PM
epriestley added a revision to T13289: Plans: 2019 Week 21-23 Bonus Content: D20552: Fix a looping workflow when trying to submit a partially-effectless transaction group.
Thu, May 23, 6:45 PM · Plans
epriestley triaged T13295: Clean up magic / passthrough request parameters as Low priority.
Thu, May 23, 6:34 PM · Transactions
epriestley requested review of D20551: Prevent editing and deleting comments in locked conversations.
Thu, May 23, 6:03 PM
epriestley added a revision to T13289: Plans: 2019 Week 21-23 Bonus Content: D20551: Prevent editing and deleting comments in locked conversations.
Thu, May 23, 6:02 PM · Plans
epriestley planned changes to D20550: Remove misleading reference to "search_documentfield" in documentation.

I'll just hold on to it for now in case something weird comes up, not a big deal if this doesn't land at the earliest possible moment.

Thu, May 23, 5:57 PM
amckinley added a comment to D20550: Remove misleading reference to "search_documentfield" in documentation.

This looks good, except as you said, we can't land it until after we remove conpherence_index as well. Should I just approve it now and let it hang around in your unlanded revisions?

Thu, May 23, 5:28 PM
epriestley added a comment to D20549: Remove the "search_documentfield" table.

Yeah, the end goal is to get those two MyISAM lines out of quickstart.sql. This gets one of 'em but is probably less than half the work. 🙈

Thu, May 23, 5:26 PM
amckinley added a comment to D20549: Remove the "search_documentfield" table.

If we're trying to drop the MyISAM dependency, don't forget to update the quickstart when we're finished: https://secure.phabricator.com/source/phabricator/browse/master/resources/sql/quickstart.sql$9727

Thu, May 23, 5:25 PM
amckinley accepted D20549: Remove the "search_documentfield" table.
Thu, May 23, 5:18 PM
epriestley requested review of D20550: Remove misleading reference to "search_documentfield" in documentation.
Thu, May 23, 5:01 PM
epriestley added a revision to T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud): D20550: Remove misleading reference to "search_documentfield" in documentation.
Thu, May 23, 5:00 PM
epriestley requested review of D20549: Remove the "search_documentfield" table.
Thu, May 23, 4:56 PM
epriestley added a revision to T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud): D20549: Remove the "search_documentfield" table.
Thu, May 23, 4:55 PM
epriestley updated the task description for T13289: Plans: 2019 Week 21-23 Bonus Content.
Thu, May 23, 4:42 PM · Plans
epriestley updated the task description for T13289: Plans: 2019 Week 21-23 Bonus Content.
Thu, May 23, 4:11 PM · Plans
epriestley updated the task description for T13289: Plans: 2019 Week 21-23 Bonus Content.
Thu, May 23, 3:51 PM · Plans
epriestley moved T6276: Cannot Create Named Link to URI with Brackets from Parser to Next on the Remarkup board.
Thu, May 23, 2:49 PM · Remarkup
epriestley moved T12191: Remarkup autocomplete can remain open after keyboard form submission from Backlog to Next on the Remarkup board.
Thu, May 23, 2:48 PM · Remarkup, PHUI
epriestley removed a project from T10667: Improve ambiguity when resolving commits that appear in multiple repositories: Remarkup.
Thu, May 23, 2:48 PM · Diffusion (v3)
epriestley moved T7967: impossible to link string that contain ']' with remarkup syntax. from Backlog to Next on the Remarkup board.
Thu, May 23, 2:47 PM · Remarkup
epriestley edited projects for T10152: Allow uri.allowed-protocols to be configured in the Phacility cluster, added: Phacility; removed Customer Impact, Remarkup, Feature Request.
Thu, May 23, 2:46 PM · Phacility