Thread (2 messages) flat view 2 messages, 2 authors, 2016-06-15

Re: [PATCH] fast-import: add 'ls' command

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:50:09

Possibly related (same subject, not in this thread)

Jonathan Nieder [off-list ref] writes:
I somewhat agree.  Actually I would go further: the word "index" brings
to mind .git/index and its in-core counterpart, so at the same time as
documenting it better, we might look into making it more self-
explanatory.  Maybe a syntax like

	ls current "path/to/entry"

would make it clearer that this is about directory entries in
fast-import's active commit and not necessarily the usual index file?
I think that explains the feature better.  I was wondering if the stream
somehow wanted to access the state the index of the repository happens to
be.
I am not sure what syntax other vcs-es use for tree-ishes.  To avoid
name clashes (what if 'current' is the low-level name of a tree-ish?),
an alternative might be

	ls-tree :11 "path/to/historical/entry"
	ls "path/to/current/entry"
Is it an option to use "ls" for both cases and treat one-arg and two-arg
cases differently?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help