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

	F13364214: D8512.id20192.diff
	Wed, Jun 26, 6:44 AM

	F13357787: D8512.id20204.diff
	Tue, Jun 25, 2:45 AM

	F13354213: D8512.diff
	Mon, Jun 24, 3:12 AM

	F13352150: D8512.diff
	Sun, Jun 23, 11:43 AM

	F13338910: D8512.id20204.diff
	Wed, Jun 19, 1:10 PM

	F13333003: D8512.id20204.diff
	Mon, Jun 17, 11:50 PM

	F13329747: D8512.id20204.diff
	Mon, Jun 17, 1:27 AM

	F13329746: D8512.id20192.diff
	Mon, Jun 17, 1:26 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

Branch

hgbookmark

Lint

Lint Passed

Unit

No Test Coverage

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 20192

View Options

src/workflow/ArcanistPatchWorkflow.php

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