Paths

Table of Contentst

Differential D10197

Don't interpret `arc browse .` as a commit
ClosedPublic
Actions

Authored by epriestley on Aug 8 2014, 6:03 PM.

Tags

None

Referenced Files

	F15469937: D10197.id24532.diff
	Fri, Apr 4, 6:18 PM

	F15467942: D10197.diff
	Thu, Apr 3, 6:33 PM

	F15463692: D10197.id24534.diff
	Wed, Apr 2, 2:12 AM

	F15453487: D10197.diff
	Sat, Mar 29, 12:11 PM

	F15422220: D10197.id24531.diff
	Sat, Mar 22, 6:44 AM

	F15420006: D10197.id24533.diff
	Fri, Mar 21, 10:32 AM

	F15334452: D10197.diff
	Sat, Mar 8, 9:20 AM

	Unknown Object (File)
	Feb 28 2025, 1:39 AM

Subscribers

Details

Reviewers

btrahan
avivey
chad
• avive

Maniphest Tasks

T5781: Moar functionality for `arc browse`!

Commits

rARC377eb5d1e0a2: Don't interpret `arc browse .` as a commit

Summary

Ref T5781. git show . works like HEAD, but that isn't what arc browse . means.

Test Plan

Ran arc browse . with a repository at a published commit.

Diff Detail

Repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24531.Aug 8 2014, 6:03 PM

epriestley retitled this revision from to Don't interpret `arc browse .` as a commit.

epriestley updated this object.

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

epriestley added reviewers: chad, btrahan, • avive.

epriestley added a task: T5781: Moar functionality for `arc browse`!.

Herald added subscribers: avivey, epriestley. · View Herald TranscriptAug 8 2014, 6:03 PM

Harbormaster completed remote builds in B2114: Diff 24531.Aug 8 2014, 6:05 PM

Also use git show ... -- to prevent interpretation of the commit argument as a path.

Harbormaster completed remote builds in B2115: Diff 24532.Aug 8 2014, 6:08 PM

this now fails for me:

$ arc browse app
Exception
Command failed with error #128!
COMMAND
git show -s --format='%H' 'app' --

STDOUT
(empty)

STDERR
fatal: bad revision 'app'

(Run with --trace for a full exception trace.)

Better?

Harbormaster completed remote builds in B2116: Diff 24533.Aug 8 2014, 6:14 PM

Yes :)

(I'm still not sure what's the deal with git. It only does that trick for HEAD).

This revision is now accepted and ready to land.Aug 8 2014, 6:15 PM

Closed by commit rARC377eb5d1e0a2 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

repository/

api/

ArcanistGitAPI.php

2 lines

workflow/

ArcanistBrowseWorkflow.php

16 lines

Diff 24534

src/repository/api/ArcanistGitAPI.php

Loading...

src/workflow/ArcanistBrowseWorkflow.php

Loading...