Paths

Table of Contentst

Use hgsprintf() when managing bookmarks in arc patch in mercurial
ClosedPublic
Actions

Authored by epriestley on Mar 12 2014, 11:11 PM.

Tags

None

Referenced Files

	F15466303: D8512.diff
	Thu, Apr 3, 4:07 AM

	F15459490: D8512.diff
	Mon, Mar 31, 1:29 PM

	F15451133: D8512.diff
	Fri, Mar 28, 8:19 PM

	F15440916: D8512.diff
	Wed, Mar 26, 3:24 PM

	F15436303: D8512.id20192.diff
	Tue, Mar 25, 1:17 PM

	F15432109: D8512.id.diff
	Mon, Mar 24, 4:10 PM

	F15426079: D8512.diff
	Sun, Mar 23, 6:58 AM

	F15425038: D8512.id20192.diff
	Sun, Mar 23, 2:10 AM

View All Files

Subscribers

aran

epriestley

Details

Reviewers

btrahan
durham

Maniphest Tasks

T4596: arc patch should use hgsprintf for bookmarks

Commits

rARC67239a08a58f: Use hgsprintf() when managing bookmarks in arc patch in mercurial

Summary

Fixes T4596. I couldn't immediately reproduce this, but the hgsprintf() version is clearly more correct.

Test Plan

Used arc patch --trace to examine hg commands.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20192.Mar 12 2014, 11:11 PM

epriestley retitled this revision from to Use hgsprintf() when managing bookmarks in arc patch in mercurial.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: btrahan, durham.

epriestley added a task: T4596: arc patch should use hgsprintf for bookmarks.

Herald added subscribers: epriestley, aran.Mar 12 2014, 11:11 PM

durham accepted this revision.Mar 12 2014, 11:16 PM

durham edited edge metadata.

This revision is now accepted and ready to land.Mar 12 2014, 11:16 PM

Closed by commit rARC67239a08a58f (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistPatchWorkflow.php

4 lines

Diff 20204

View Options

src/workflow/ArcanistPatchWorkflow.php

Use hgsprintf() when managing bookmarks in arc patch in mercurialClosedPublicActions