Page MenuHomePhabricator
Feed Advanced Search

Jul 21 2021

epriestley committed rP706b91adf677: Remove "HarbormasterBuildableTransaction::TYPE_CREATE" (authored by epriestley).
Remove "HarbormasterBuildableTransaction::TYPE_CREATE"
Jul 21 2021, 9:18 PM
epriestley committed rP1a1395579f95: Remove "HarbormasterBuildCommand" (authored by epriestley).
Remove "HarbormasterBuildCommand"
Jul 21 2021, 9:18 PM
epriestley closed D21692: Remove "HarbormasterBuildCommand".
Jul 21 2021, 9:18 PM
epriestley closed D21691: Modularize almost all Harbormaster build message workflows and UI/UX.
Jul 21 2021, 9:18 PM
epriestley committed rP1212dc5fbe8b: Modularize almost all Harbormaster build message workflows and UI/UX (authored by epriestley).
Modularize almost all Harbormaster build message workflows and UI/UX
Jul 21 2021, 9:18 PM
epriestley committed rP8bbee921398d: Modularize individual Harbormaster build messages (authored by epriestley).
Modularize individual Harbormaster build messages
Jul 21 2021, 9:17 PM
epriestley closed D21690: Modularize individual Harbormaster build messages.
Jul 21 2021, 9:17 PM
epriestley closed D21689: Modularize HarbormasterBuildTransaction.
Jul 21 2021, 9:17 PM
epriestley committed rP6dfea0adaddc: Modularize HarbormasterBuildTransaction (authored by epriestley).
Modularize HarbormasterBuildTransaction
Jul 21 2021, 9:17 PM
epriestley committed rPb84c02959a8b: Remove "HarbormasterBuildTransaction::TYPE_CREATE" (authored by epriestley).
Remove "HarbormasterBuildTransaction::TYPE_CREATE"
Jul 21 2021, 9:17 PM
epriestley committed rP012af00731fd: Improve formality of "HarbormasterBuild" states (authored by epriestley).
Improve formality of "HarbormasterBuild" states
Jul 21 2021, 9:17 PM
epriestley committed rPbf83fffca129: Correct the flow of edit authority when sending messages to HarbormasterBuild… (authored by epriestley).
Correct the flow of edit authority when sending messages to HarbormasterBuild…
Jul 21 2021, 9:17 PM
epriestley closed D21688: Remove "HarbormasterBuildTransaction::TYPE_CREATE".
Jul 21 2021, 9:17 PM
epriestley closed D21685: Improve formality of "HarbormasterBuild" states.
Jul 21 2021, 9:17 PM
epriestley committed rPb48d4fabaf9e: Merge the "HarbormasterBuildCommand" table into "HarbormasterBuildMessage" (authored by epriestley).
Merge the "HarbormasterBuildCommand" table into "HarbormasterBuildMessage"
Jul 21 2021, 9:17 PM
epriestley closed D21684: Merge the "HarbormasterBuildCommand" table into "HarbormasterBuildMessage".
Jul 21 2021, 9:17 PM
epriestley committed rP0f93d1ffe450: Rename "HarbormasterBuild" methods to prepare for use of the "BuildMessages"… (authored by epriestley).
Rename "HarbormasterBuild" methods to prepare for use of the "BuildMessages"…
Jul 21 2021, 9:17 PM
epriestley closed D21683: Rename "HarbormasterBuild" methods to prepare for use of the "BuildMessages" table.
Jul 21 2021, 9:17 PM
epriestley accepted D21706: Update other usages of "hg rebase" to use the new extension-enabling function.
Jul 21 2021, 9:06 PM
epriestley committed rARCec68005d758c: Remove "phutil_deprecated()" (authored by epriestley).
Remove "phutil_deprecated()"
Jul 21 2021, 5:21 PM
epriestley closed D21704: Remove "phutil_deprecated()".
Jul 21 2021, 5:21 PM
epriestley closed D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:21 PM
epriestley committed rARC8bb7d5889050: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()" (authored by epriestley).
Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()"
Jul 21 2021, 5:21 PM
epriestley closed T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()" as Resolved by committing rARC8bb7d5889050: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:21 PM · Infrastructure
epriestley committed rPaf6cc0c934c4: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in… (authored by epriestley).
Use "resolve()", not "execute()", for PhutilExecPassthru callsites in…
Jul 21 2021, 5:21 PM
epriestley closed D21703: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in Phabricator.
Jul 21 2021, 5:21 PM
epriestley added inline comments to D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:20 PM
epriestley updated the test plan for D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:12 PM
epriestley requested review of D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:12 PM
epriestley added a revision to T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()": D21705: Deprecate "PhutilExecPassthru->execute()" in favor of "resolve()".
Jul 21 2021, 5:11 PM · Infrastructure
epriestley requested review of D21704: Remove "phutil_deprecated()".
Jul 21 2021, 5:08 PM
epriestley requested review of D21703: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in Phabricator.
Jul 21 2021, 4:56 PM
epriestley added a revision to T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()": D21703: Use "resolve()", not "execute()", for PhutilExecPassthru callsites in Phabricator.
Jul 21 2021, 4:54 PM · Infrastructure
epriestley triaged T13660: Deprecate "PhutilExecPassthru->execute()" in favor of "PhutilExecPassthru->resolve()" as Low priority.
Jul 21 2021, 4:49 PM · Infrastructure
epriestley added a comment to T13659: `arc land` may fail with missing rebase extension.

(After picking up D21697, the approach works properly in my environment where I originally ran into this, but there are a couple of rebase calls still present in ArcanistMercurialLandEngine that don't use the new API yet.)

Jul 21 2021, 4:44 PM · Arcanist, Mercurial
epriestley added a comment to D21697: Refactor how Mercurial runs commands that require extensions.

I just use TextMate, which is a fairly lightweight editor. I have it minimally configured to disable a couple of default behaviors and add a couple aliases, and I have a crude "jump-to-definition" script set up that tries to open whatever file defines the class/function under the cursor and succeeds about 95% of the time.

Jul 21 2021, 4:29 PM
epriestley added a comment to D21697: Refactor how Mercurial runs commands that require extensions.

...the one with --onto-remote default not appearing in the trace output...

Jul 21 2021, 4:14 AM
epriestley added inline comments to D21697: Refactor how Mercurial runs commands that require extensions.
Jul 21 2021, 4:04 AM
epriestley committed rARC35c1b9bf027b: Fix an ExecFuture typo, "preprebuilt" (authored by epriestley).
Fix an ExecFuture typo, "preprebuilt"
Jul 21 2021, 3:18 AM
epriestley closed D21702: Fix an ExecFuture typo, "preprebuilt".
Jul 21 2021, 3:18 AM
epriestley requested review of D21702: Fix an ExecFuture typo, "preprebuilt".
Jul 21 2021, 3:15 AM
epriestley accepted D21697: Refactor how Mercurial runs commands that require extensions.

Thanks! Couple of minor inlines but I didn't catch anything substantial.

Jul 21 2021, 3:14 AM
epriestley accepted D21701: Suppress PHP 8 deprecation warning in startup.
Jul 21 2021, 1:03 AM
epriestley added a comment to T13648: Transaction publishing may stall during mail expansion of package recipients.

This happens when a recipient list includes an Owners package which has been destroyed. Specifically, we'll exit this section of PhabricatorMetaMTAMemberQuery with out the PHID in $package_map, and then fail to return it:

Jul 21 2021, 1:03 AM · Transactions

Jul 20 2021

epriestley accepted D21700: Display informative message when arc launches an editor.

There a bit of fancier indent/formatting stuff in some of the newer code (e.g., in ArcanistRefView->newLines()) but it hasn't really generalized into something you could easily apply here yet. I think this is fine for now and we could revisit it to make it fancier later on if the newest display stuff gets generalized a bit.

Jul 20 2021, 10:58 PM
epriestley added a comment to D21700: Display informative message when arc launches an editor.

Maybe the output could be two separate lines:

Jul 20 2021, 10:12 PM
epriestley requested review of D21699: Rename "HarbormasterRestartException" to "HarbormasterMessageException".
Jul 20 2021, 9:45 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21699: Rename "HarbormasterRestartException" to "HarbormasterMessageException".
Jul 20 2021, 9:43 PM · Harbormaster
epriestley requested review of D21698: Allow "harbormaster.sendmessage" to send control command (pause, restart, abort, resume) to Builds/Buildables.
Jul 20 2021, 9:42 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21698: Allow "harbormaster.sendmessage" to send control command (pause, restart, abort, resume) to Builds/Buildables.
Jul 20 2021, 9:41 PM · Harbormaster
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

That all looks good to me.

Jul 20 2021, 7:57 PM

Jul 19 2021

epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

We could conceivably prevent that error by making GitLocalState keep a trivial stack (just a stack of true) and return its current depth (0, 1, 2, ...), then throw if the passed $ref was not the current stack depth.

Jul 19 2021, 7:18 PM
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

Since modern Arcanist usage with Mercurial has required the arc-ls-markers for a while now and nobody else has reported issues, I think it's fine to not try to address these issues for older versions.

Jul 19 2021, 5:14 PM
epriestley accepted D21686: Update "arc diff" to amend non-head commits with Mercurial.
Jul 19 2021, 5:12 PM
epriestley added a comment to T13659: `arc land` may fail with missing rebase extension.

...perhaps we should consider a way to do this automatically or at least less-manually.

Jul 19 2021, 4:58 PM · Arcanist, Mercurial
epriestley added a comment to D21680: An assortment of fixes and updates to using arc-land with mercurial.

This makes me think that in this second scenario where running arc diff initially includes multiple commits that the revision should be updated to include the commit hashes for each, and then might result in having better detect this scenario if it's searching for revisions with associated commit hashes.

Jul 19 2021, 4:33 PM
epriestley triaged T13659: `arc land` may fail with missing rebase extension as Low priority.
Jul 19 2021, 4:32 PM · Arcanist, Mercurial

Jul 16 2021

epriestley requested review of D21696: Add a side nav to Conduit API method console pages.
Jul 16 2021, 10:16 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21696: Add a side nav to Conduit API method console pages.
Jul 16 2021, 10:15 PM · Harbormaster
epriestley added a comment to T13652: Notes on Ardunio CNC drivers.
  • Cryptographically Random Orbit Sander
Jul 16 2021, 9:43 PM
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

Sounds good. I spent 15 seconds on trying to write hg arc-amend, and this does something and does appear to produce an amended commit that looks ballpark-correct (and can amend non-heads), but emits some warnings and I have no clue if it functions across Mercurial versions:

Jul 16 2021, 5:35 PM
epriestley requested review of D21695: Add stub "harbormaster.build.edit" and "harbormaster.buildable.edit" API methods.
Jul 16 2021, 4:44 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21695: Add stub "harbormaster.build.edit" and "harbormaster.buildable.edit" API methods.
Jul 16 2021, 4:43 PM · Harbormaster
epriestley requested review of D21694: Modularize "HarbormasterBuildableTransaction".
Jul 16 2021, 4:01 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21694: Modularize "HarbormasterBuildableTransaction".
Jul 16 2021, 4:00 PM · Harbormaster
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

My only guess is you've delved into the Mercurial source base (I've poked around a little)

Jul 16 2021, 3:55 PM
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

I don't think LocalState should z̵̰̲͚͖̊͗́̓̓a̷̛̱̯̠͐̿̅̅͛̓͗ĺ̵̨̻̠͓̼̗̥̥̻̲̳̎͜ͅg̴̛͙͗̀̄̈́̃̂̓̄͝o̶̥̘̻͙̬͇͚̥̪͌̔̃͛̆̀̂̚ͅ your terminal!

Jul 16 2021, 3:54 PM
epriestley requested review of D21693: Remove "HarbormasterBuildableTransaction::TYPE_CREATE".
Jul 16 2021, 3:35 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21693: Remove "HarbormasterBuildableTransaction::TYPE_CREATE".
Jul 16 2021, 3:34 PM · Harbormaster
epriestley requested review of D21692: Remove "HarbormasterBuildCommand".
Jul 16 2021, 3:11 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21692: Remove "HarbormasterBuildCommand".
Jul 16 2021, 3:10 PM · Harbormaster
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

This is probably not a great path to walk down, but there are secret flags that disable the "cannot amend changeset with children" check:

Jul 16 2021, 2:15 AM
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

Another possible approach might be to copy the amend extension to create hg arc-amend which just lets you amend anything, assuming the "head only" rule is a safety guard rail rather than a fundamental issue with Mercurial (which it seems like it should be).

Jul 16 2021, 2:07 AM
epriestley requested review of D21691: Modularize almost all Harbormaster build message workflows and UI/UX.
Jul 16 2021, 1:32 AM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21691: Modularize almost all Harbormaster build message workflows and UI/UX.
Jul 16 2021, 1:30 AM · Harbormaster
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

Ohhhhh, sorry, I didn't connect the dots and understand that you're only sometimes running the amend (since you can't amend for non-heads).

Jul 16 2021, 1:28 AM

Jul 15 2021

epriestley added a comment to T13072: Merge Harbormaster BuildCommand into BuildMessage.

This ("Queued at") looks suspicious:

Jul 15 2021, 11:03 PM · Harbormaster
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

I'm not catching on to what you're referring to. Are you saying there are other changes which will have already stashed unsaved state away before calling amendCommit()?

Jul 15 2021, 11:02 PM
epriestley requested review of D21690: Modularize individual Harbormaster build messages.
Jul 15 2021, 10:25 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21690: Modularize individual Harbormaster build messages.
Jul 15 2021, 10:24 PM · Harbormaster
epriestley requested review of D21689: Modularize HarbormasterBuildTransaction.
Jul 15 2021, 9:07 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21689: Modularize HarbormasterBuildTransaction.
Jul 15 2021, 9:05 PM · Harbormaster
epriestley requested review of D21688: Remove "HarbormasterBuildTransaction::TYPE_CREATE".
Jul 15 2021, 6:10 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21688: Remove "HarbormasterBuildTransaction::TYPE_CREATE".
Jul 15 2021, 6:09 PM · Harbormaster
epriestley requested review of D21687: Correct the flow of edit authority when sending messages to HarbormasterBuild objects.
Jul 15 2021, 6:04 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21687: Correct the flow of edit authority when sending messages to HarbormasterBuild objects.
Jul 15 2021, 6:02 PM · Harbormaster
epriestley added a comment to D21686: Update "arc diff" to amend non-head commits with Mercurial.

After arc diff creates a revision in Phabricator it amends the commit to include a link to the revision in the commit message.

Jul 15 2021, 4:05 PM

Jul 13 2021

epriestley added a comment to T13072: Merge Harbormaster BuildCommand into BuildMessage.

In D21685, I've imposed stricter rules for which state transitions are allowed: for example, you can't issue a "pause" command to a build that is already pausing.

Jul 13 2021, 11:41 PM · Harbormaster
epriestley retitled D21685: Improve formality of "HarbormasterBuild" states from Improve formality of "HarbormasterBuild" messages to Improve formality of "HarbormasterBuild" states.
Jul 13 2021, 11:39 PM
epriestley requested review of D21685: Improve formality of "HarbormasterBuild" states.
Jul 13 2021, 11:38 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21685: Improve formality of "HarbormasterBuild" states.
Jul 13 2021, 11:37 PM · Harbormaster
epriestley requested review of D21684: Merge the "HarbormasterBuildCommand" table into "HarbormasterBuildMessage".
Jul 13 2021, 10:50 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21684: Merge the "HarbormasterBuildCommand" table into "HarbormasterBuildMessage".
Jul 13 2021, 10:48 PM · Harbormaster
epriestley requested review of D21683: Rename "HarbormasterBuild" methods to prepare for use of the "BuildMessages" table.
Jul 13 2021, 10:24 PM
epriestley added a revision to T13072: Merge Harbormaster BuildCommand into BuildMessage: D21683: Rename "HarbormasterBuild" methods to prepare for use of the "BuildMessages" table.
Jul 13 2021, 10:23 PM · Harbormaster
epriestley updated the task description for T13072: Merge Harbormaster BuildCommand into BuildMessage.
Jul 13 2021, 10:04 PM · Harbormaster
epriestley added a comment to D21682: Add a prompt to allow pruning merged branches when using --hold.

I take it these disparate trees have some limitations? Do they always apply to the working directory or something?

Jul 13 2021, 2:42 PM

Jul 12 2021

epriestley added a comment to D21682: Add a prompt to allow pruning merged branches when using --hold.

I'm still piecing together Git's handling of branches in my mind as I have used Mercurial for so long.

Jul 12 2021, 5:55 PM
epriestley added a comment to D21682: Add a prompt to allow pruning merged branches when using --hold.

Would the expectation then be that running arc land again would remove the rebased/squashed commits created by --hold, so the result would be as if they hadn't run --hold previously?

Jul 12 2021, 2:53 AM

Jul 11 2021

epriestley added a comment to T13652: Notes on Ardunio CNC drivers.

There are a bunch of libraries to help with this that I obviously won't be using because I didn't write them...

Jul 11 2021, 11:31 PM