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

	Unknown Object (File)
	Wed, Apr 17, 3:28 PM

	Unknown Object (File)
	Wed, Apr 17, 2:49 PM

	Unknown Object (File)
	Thu, Apr 11, 5:25 AM

	Unknown Object (File)
	Mon, Apr 8, 7:44 PM

	Unknown Object (File)
	Tue, Apr 2, 7:15 PM

	Unknown Object (File)
	Mon, Mar 25, 5:23 PM

	Unknown Object (File)
	Mar 8 2024, 10:50 PM

	Unknown Object (File)
	Mar 4 2024, 4:53 PM

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...