Page MenuHomePhabricator

Fix an issue with `arc branch` and very old branches
ClosedPublic

Authored by epriestley on Jun 18 2014, 12:32 PM.
Tags
None
Referenced Files
F19822002: D9620.id.diff
Sat, Mar 7, 2:06 AM
F19675402: D9620.id23070.diff
Sun, Feb 8, 9:31 AM
F19615834: D9620.id23065.diff
Feb 4 2026, 1:22 AM
F18919153: D9620.id23065.diff
Nov 9 2025, 11:35 AM
F18863654: D9620.id23065.diff
Nov 3 2025, 12:26 AM
F18625743: D9620.id23070.diff
Sep 15 2025, 11:26 PM
F18622611: D9620.id23065.diff
Sep 15 2025, 12:27 PM
F18622575: D9620.id.diff
Sep 15 2025, 12:22 PM
Subscribers

Details

Summary

In D9595, we stopped parsing short-form "Differential Revision:" fields in commit messages, and only accept URLs.

I have one of the older style commit messages in my local arcanist/, so now we go down this parse failure branch in arc branch. This has never worked quite correctly, and if the parse fails we end up with a bad branch dictionary that is missing fields.

Test Plan

Ran arc branch in a working copy with an old Differential Revision: field at the head of a branch. Before patch: explosions; after patch: works great.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Fix an issue with `arc branch` and very old branches.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: joshuaspence.
This revision is now accepted and ready to land.Jun 18 2014, 3:32 PM
epriestley updated this revision to Diff 23070.

Closed by commit rARC2198cc2849c8 (authored by @epriestley).