Page MenuHomePhabricator
Feed All Stories

Jul 23 2020

epriestley requested review of D21425: Streamline handling of Futures and PIDs in daemons.
Jul 23 2020, 5:41 PM
epriestley added a revision to T13555: In "ExecFuture", "proc_get_status()" may be passed NULL, possibly leading to a daemon stall: D21425: Streamline handling of Futures and PIDs in daemons.
Jul 23 2020, 5:40 PM · Daemons
epriestley updated the diff for D21423: Give Futures clearer start/end and exception semantics.
  • Add "ExecFuture->hasPID()" to simplify reading PID information.
Jul 23 2020, 5:17 PM
epriestley added a comment to T13555: In "ExecFuture", "proc_get_status()" may be passed NULL, possibly leading to a daemon stall.

The call sequence here is approximately:

Jul 23 2020, 5:11 PM · Daemons
epriestley requested review of D21424: Manage PIDs more carefully in DaemonHandle.
Jul 23 2020, 5:10 PM
epriestley added a revision to T13555: In "ExecFuture", "proc_get_status()" may be passed NULL, possibly leading to a daemon stall: D21424: Manage PIDs more carefully in DaemonHandle.
Jul 23 2020, 5:09 PM · Daemons
epriestley requested review of D21423: Give Futures clearer start/end and exception semantics.
Jul 23 2020, 5:08 PM
epriestley added a revision to T13555: In "ExecFuture", "proc_get_status()" may be passed NULL, possibly leading to a daemon stall: D21423: Give Futures clearer start/end and exception semantics.
Jul 23 2020, 5:07 PM · Daemons
epriestley triaged T13555: In "ExecFuture", "proc_get_status()" may be passed NULL, possibly leading to a daemon stall as Normal priority.
Jul 23 2020, 2:58 PM · Daemons
epriestley committed rPfcb75d0503c6: Fix an issue where prose diffing may fail after hitting the PCRE backtracking… (authored by epriestley).
Fix an issue where prose diffing may fail after hitting the PCRE backtracking…
Jul 23 2020, 2:46 PM
epriestley closed D21422: Fix an issue where prose diffing may fail after hitting the PCRE backtracking limit.
Jul 23 2020, 2:46 PM
epriestley closed T13554: Undefined offset exception in "PhutilProseDifferenceEngine::stitchPieces()" as Resolved by committing rPfcb75d0503c6: Fix an issue where prose diffing may fail after hitting the PCRE backtracking….
Jul 23 2020, 2:46 PM · Infrastructure
epriestley requested review of D21422: Fix an issue where prose diffing may fail after hitting the PCRE backtracking limit.
Jul 23 2020, 2:41 PM
epriestley added a revision to T13554: Undefined offset exception in "PhutilProseDifferenceEngine::stitchPieces()": D21422: Fix an issue where prose diffing may fail after hitting the PCRE backtracking limit.
Jul 23 2020, 2:40 PM · Infrastructure
epriestley added a comment to T13554: Undefined offset exception in "PhutilProseDifferenceEngine::stitchPieces()".

This is a PCRE backtracking issue. It reproduces on an input corpus of length 3,277 bytes with only "x", " " (space), and "\n" (newline) characters, and disappears if I increase pcre.backtrack_limit.

Jul 23 2020, 2:36 PM · Infrastructure
epriestley triaged T13554: Undefined offset exception in "PhutilProseDifferenceEngine::stitchPieces()" as Low priority.
Jul 23 2020, 1:41 PM · Infrastructure

Jul 22 2020

epriestley added a comment to T13545: Upgrading: Mid 2020 Changes to "arc diff".

Please use Discourse to discuss Phabricator.

Jul 22 2020, 11:49 PM · Arcanist, Guides
jbrownEP added a comment to T13545: Upgrading: Mid 2020 Changes to "arc diff".

This task references more details on "Excuses" and "Prompts", but there isn't any. Is there any way to provide context around lint issues?

Jul 22 2020, 11:46 PM · Arcanist, Guides
epriestley committed rP324445589b5e: (stable) Fix an issue with destruction of Revision and Diff objects with… (authored by epriestley).
(stable) Fix an issue with destruction of Revision and Diff objects with…
Jul 22 2020, 8:20 PM
epriestley committed rP8f9ba4852861: Fix an issue with destruction of Revision and Diff objects with viewstates (authored by epriestley).
Fix an issue with destruction of Revision and Diff objects with viewstates
Jul 22 2020, 8:07 PM
epriestley closed D21421: Fix an issue with destruction of Revision and Diff objects with viewstates.
Jul 22 2020, 8:07 PM
epriestley requested review of D21421: Fix an issue with destruction of Revision and Diff objects with viewstates.
Jul 22 2020, 6:56 PM
epriestley added a comment to D21041: Use a named remote and branches for staging to help git-lfs.

If you believe you've found a bug in Phabricator, please report it on Discourse, including complete reproduction instructions.

Jul 22 2020, 2:07 AM
maroux added inline comments to D21041: Use a named remote and branches for staging to help git-lfs.
Jul 22 2020, 1:03 AM

Jul 21 2020

ipalencar updated ipalencar.
Jul 21 2020, 6:42 PM
epriestley updated the task description for T13547: Upgrading: Mid 2020 Changes to "arc land".
Jul 21 2020, 6:24 PM · Guides, Arcanist
epriestley updated the task description for T13547: Upgrading: Mid 2020 Changes to "arc land".
Jul 21 2020, 6:24 PM · Guides, Arcanist
epriestley triaged T13553: Creating a milestone via the API fails when accessing the project's parent as Normal priority.
Jul 21 2020, 5:54 PM · Transactions, Projects

Jul 20 2020

epriestley removed a member for Phacility Core Access: amckinley.
Jul 20 2020, 5:23 PM
epriestley removed a member for Phacility: amckinley.
Jul 20 2020, 5:23 PM
epriestley defrocked amckinley.
Jul 20 2020, 5:23 PM

Jul 18 2020

epriestley committed rPd32c3f8d7d65: (stable) Likely, fix a warning when rendering modified coverage (authored by epriestley).
(stable) Likely, fix a warning when rendering modified coverage
Jul 18 2020, 2:45 AM
epriestley committed rP0ed5569e9f7b: Likely, fix a warning when rendering modified coverage (authored by epriestley).
Likely, fix a warning when rendering modified coverage
Jul 18 2020, 2:45 AM
epriestley closed D21420: Likely, fix a warning when rendering modified coverage.
Jul 18 2020, 2:45 AM
epriestley requested review of D21420: Likely, fix a warning when rendering modified coverage.
Jul 18 2020, 2:45 AM

Jul 17 2020

epriestley committed rP37ffb71c4d5e: In source views, wrap display tabs in "user-select: all" to improve cursor… (authored by epriestley).
In source views, wrap display tabs in "user-select: all" to improve cursor…
Jul 17 2020, 10:10 PM
epriestley closed D21419: In source views, wrap display tabs in "user-select: all" to improve cursor selection behavior.
Jul 17 2020, 10:10 PM
epriestley requested review of D21419: In source views, wrap display tabs in "user-select: all" to improve cursor selection behavior.
Jul 17 2020, 10:08 PM
epriestley added a revision to T2495: Improve whitespace expansion of tab literals: D21419: In source views, wrap display tabs in "user-select: all" to improve cursor selection behavior.
Jul 17 2020, 10:07 PM · Restricted Project, FreeBSD, Differential, Diffusion

Jul 16 2020

epriestley added a comment to T6769: Confusing UI when harbormaster fails but differential is accepted.

Please use Discourse to discuss Phabricator. (And please don't bump unrelated tasks from five and a half years ago.)

Jul 16 2020, 3:11 PM · Harbormaster
Kwisatz added a comment to T6769: Confusing UI when harbormaster fails but differential is accepted.

Is this really a harbormaster problem though? I would have located it within Differential.

Jul 16 2020, 12:49 PM · Harbormaster

Jul 12 2020

epriestley requested review of D21418: Improve desktop and mobile layouts for new "CommitGridView".
Jul 12 2020, 9:27 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21418: Improve desktop and mobile layouts for new "CommitGridView".
Jul 12 2020, 9:26 PM · Differential
epriestley requested review of D21417: Unify more build, property, auditor, and status information into "CommitGraphView".
Jul 12 2020, 7:47 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21417: Unify more build, property, auditor, and status information into "CommitGraphView".
Jul 12 2020, 7:45 PM · Differential
epriestley requested review of D21416: Clean up some minor commit list CSS.
Jul 12 2020, 6:04 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21416: Clean up some minor commit list CSS.
Jul 12 2020, 6:03 PM · Differential
epriestley requested review of D21415: Remove "PhabricatorAuditListView".
Jul 12 2020, 5:45 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21415: Remove "PhabricatorAuditListView".
Jul 12 2020, 5:44 PM · Differential
epriestley requested review of D21414: Remove "DiffusionHistoryTableView" and "DiffusionHistoryView".
Jul 12 2020, 4:26 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21414: Remove "DiffusionHistoryTableView" and "DiffusionHistoryView".
Jul 12 2020, 4:25 PM · Differential
epriestley requested review of D21413: Move the view of merged changes to "DiffusionCommitGraphView".
Jul 12 2020, 4:17 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21413: Move the view of merged changes to "DiffusionCommitGraphView".
Jul 12 2020, 4:16 PM · Differential
epriestley requested review of D21412: Replace "DiffusionCommitListView" with "DiffusionCommitGraphView".
Jul 12 2020, 3:44 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21412: Replace "DiffusionCommitListView" with "DiffusionCommitGraphView".
Jul 12 2020, 3:41 PM · Differential

Jul 11 2020

epriestley committed rARC2565cc7b4d1d: (stable) Promote 2020 Week 27 (authored by epriestley).
(stable) Promote 2020 Week 27
Jul 11 2020, 4:34 PM
epriestley edited the content of Changelog.
Jul 11 2020, 4:33 PM
epriestley committed rP2710a1e7aa42: (stable) Promote 2020 Week 27 (authored by epriestley).
(stable) Promote 2020 Week 27
Jul 11 2020, 4:32 PM
epriestley created an object: 2020 Week 27 (Mid July).
Jul 11 2020, 4:32 PM

Jul 10 2020

epriestley requested review of D21411: Improve rendering of history graph in "CommitGraphView".
Jul 10 2020, 6:46 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21411: Improve rendering of history graph in "CommitGraphView".
Jul 10 2020, 6:44 PM · Differential
epriestley requested review of D21410: Introduce "DiffusionCommitGraphView", which unifies "HistoryListView" and "HistoryTableView".
Jul 10 2020, 6:22 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21410: Introduce "DiffusionCommitGraphView", which unifies "HistoryListView" and "HistoryTableView".
Jul 10 2020, 6:20 PM · Differential
epriestley requested review of D21409: Remove the "Graph" view as a dedicated repository view.
Jul 10 2020, 5:57 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21409: Remove the "Graph" view as a dedicated repository view.
Jul 10 2020, 5:56 PM · Differential
epriestley requested review of D21408: Remove "Recent Commits" from repository landing page.
Jul 10 2020, 5:49 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21408: Remove "Recent Commits" from repository landing page.
Jul 10 2020, 5:48 PM · Differential
epriestley requested review of D21407: Remove "DiffusionTagTableView".
Jul 10 2020, 4:27 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21407: Remove "DiffusionTagTableView".
Jul 10 2020, 4:25 PM · Differential
epriestley requested review of D21406: Remove the "authored" subheader from commits.
Jul 10 2020, 4:21 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21406: Remove the "authored" subheader from commits.
Jul 10 2020, 4:20 PM · Differential
Krinkle awarded T13069: Make mail stamps routable on the server, include transaction information, and replace mail tags a Orange Medal token.
Jul 10 2020, 4:17 PM · Mail
epriestley requested review of D21405: Modernize "Author" and "Committer" rendering for commits.
Jul 10 2020, 4:17 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21405: Modernize "Author" and "Committer" rendering for commits.
Jul 10 2020, 4:16 PM · Differential
epriestley added a comment to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch.

The broader structural issue here is:

Jul 10 2020, 4:11 PM · Differential
epriestley requested review of D21404: Remove construction of "author" information from "LastModified" payload in Diffusion.
Jul 10 2020, 4:06 PM
epriestley added a revision to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch: D21404: Remove construction of "author" information from "LastModified" payload in Diffusion.
Jul 10 2020, 4:05 PM · Differential
arian.maykon updated arian.maykon.
Jul 10 2020, 12:47 PM

Jul 9 2020

epriestley added a comment to T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch.

A second issue is that some of the "publish" behavior depends on the commit/revision relationship. This can still be sequenced correctly entirely in the publish daemon, but "close related revisions" and "publish the commit" are not entirely independent.

Jul 9 2020, 10:24 PM · Differential
epriestley triaged T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch as Normal priority.
Jul 9 2020, 10:13 PM · Differential
epriestley renamed T13361: Get rid of the "Actions" dropdown in Diffusion from Get rid of the "Actions" dropdown in in Diffusion to Get rid of the "Actions" dropdown in Diffusion.
Jul 9 2020, 9:37 PM · Design, Diffusion
epriestley closed T13369: When a large number of commits are pushed to a repository, import them at lower priority as Resolved.

This appears stable.

Jul 9 2020, 9:36 PM · Daemons, Diffusion
epriestley committed rP1d4d860cb5a0: Allow non-authors to "Request Review" of draft revisions (authored by epriestley).
Allow non-authors to "Request Review" of draft revisions
Jul 9 2020, 9:21 PM
epriestley committed rP6e85b521fe86: Don't raise the "Subscribers Won't Be Notified" draft warning if you aren't… (authored by epriestley).
Don't raise the "Subscribers Won't Be Notified" draft warning if you aren't…
Jul 9 2020, 9:21 PM
epriestley committed rPb21b73b8dd35: Expand Revision transaction API to allow actions to vary more broadly based on… (authored by epriestley).
Expand Revision transaction API to allow actions to vary more broadly based on…
Jul 9 2020, 9:21 PM
epriestley closed D21403: Allow non-authors to "Request Review" of draft revisions.
Jul 9 2020, 9:20 PM
epriestley closed D21401: Expand Revision transaction API to allow actions to vary more broadly based on the viewer and revision state.
Jul 9 2020, 9:20 PM
epriestley closed D21402: Don't raise the "Subscribers Won't Be Notified" draft warning if you aren't adding any non-you subscribers.
Jul 9 2020, 9:20 PM
epriestley requested review of D21403: Allow non-authors to "Request Review" of draft revisions.
Jul 9 2020, 9:11 PM
epriestley requested review of D21402: Don't raise the "Subscribers Won't Be Notified" draft warning if you aren't adding any non-you subscribers.
Jul 9 2020, 9:02 PM
epriestley requested review of D21401: Expand Revision transaction API to allow actions to vary more broadly based on the viewer and revision state.
Jul 9 2020, 7:31 PM
epriestley requested review of D21400: Require rows passed to "loadAllFromArray()" have unique keys.
Jul 9 2020, 5:58 PM
epriestley added a reverting change for D21399: Fix an issue where querying for a large number of projects by slug could paginate incorrectly: D21400: Require rows passed to "loadAllFromArray()" have unique keys.
Jul 9 2020, 5:56 PM
epriestley committed rP852e368cb777: (stable) Fix an issue where querying for a large number of projects by slug… (authored by epriestley).
(stable) Fix an issue where querying for a large number of projects by slug…
Jul 9 2020, 5:50 PM
epriestley committed rP73c4240415e0: Add some additional patterns to the "filter Mercurial --debug output" list (authored by epriestley).
Add some additional patterns to the "filter Mercurial --debug output" list
Jul 9 2020, 5:50 PM
epriestley closed D21398: Add some additional patterns to the "filter Mercurial --debug output" list.
Jul 9 2020, 5:50 PM
epriestley committed rP56838c0e3dc2: Fix an issue where querying for a large number of projects by slug could… (authored by epriestley).
Fix an issue where querying for a large number of projects by slug could…
Jul 9 2020, 5:50 PM
epriestley closed D21399: Fix an issue where querying for a large number of projects by slug could paginate incorrectly.
Jul 9 2020, 5:50 PM
epriestley requested review of D21399: Fix an issue where querying for a large number of projects by slug could paginate incorrectly.
Jul 9 2020, 5:49 PM
epriestley requested review of D21398: Add some additional patterns to the "filter Mercurial --debug output" list.
Jul 9 2020, 5:35 PM