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
F15391555: D9620.id23065.diff
Sat, Mar 15, 10:31 AM
F15375979: D9620.id23065.diff
Thu, Mar 13, 1:08 AM
F15345589: D9620.id23070.diff
Mon, Mar 10, 11:35 AM
F15338146: D9620.id23065.diff
Sun, Mar 9, 9:34 AM
F15283128: D9620.id23065.diff
Tue, Mar 4, 5:30 AM
Unknown Object (File)
Feb 9 2025, 1:42 PM
Unknown Object (File)
Feb 9 2025, 1:42 PM
Unknown Object (File)
Feb 9 2025, 1:42 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).