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

	Unknown Object (File)
	Sat, Nov 23, 1:29 AM

	Unknown Object (File)
	Fri, Nov 22, 5:44 PM

	Unknown Object (File)
	Tue, Nov 19, 12:48 PM

	Unknown Object (File)
	Sat, Nov 16, 11:41 PM

	Unknown Object (File)
	Thu, Nov 14, 5:50 PM

	Unknown Object (File)
	Sun, Nov 10, 7:44 PM

	Unknown Object (File)
	Wed, Nov 6, 9:54 PM

	Unknown Object (File)
	Tue, Oct 29, 1:30 PM

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