git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* diff-index --cached on bare repository
@ 2009-01-30 16:13 Johannes Gilger
  2009-01-30 20:06 ` Jeff King
  0 siblings, 1 reply; 2+ messages in thread
From: Johannes Gilger @ 2009-01-30 16:13 UTC (permalink / raw)
  To: git

Hi list,

today I stumbled over the output of "git diff-index --cached HEAD" in a 
bare repository. While diff-index HEAD just reminds me that I got to 
have a work-tree, --cached outputs all the files in HEAD as deleted. The 
question is if this is intended behaviour? Or is it irrelevant because 
diff-index is only plumbing? I found a mail of the gitk-team working 
around this, which is funny because I discovered this while working on 
GitX.

So please, enlighten me ;)

Greetings,
Jojo

-- 
Johannes Gilger <heipei@hackvalue.de>
http://hackvalue.de/heipei/
GPG-Key: 0x42F6DE81
GPG-Fingerprint: BB49 F967 775E BB52 3A81  882C 58EE B178 42F6 DE81

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: diff-index --cached on bare repository
  2009-01-30 16:13 diff-index --cached on bare repository Johannes Gilger
@ 2009-01-30 20:06 ` Jeff King
  0 siblings, 0 replies; 2+ messages in thread
From: Jeff King @ 2009-01-30 20:06 UTC (permalink / raw)
  To: Johannes Gilger; +Cc: git

On Fri, Jan 30, 2009 at 04:13:53PM +0000, Johannes Gilger wrote:

> today I stumbled over the output of "git diff-index --cached HEAD" in a 
> bare repository. While diff-index HEAD just reminds me that I got to 
> have a work-tree, --cached outputs all the files in HEAD as deleted. The 
> question is if this is intended behaviour? Or is it irrelevant because 
> diff-index is only plumbing? I found a mail of the gitk-team working 
> around this, which is funny because I discovered this while working on 
> GitX.

Generally commands which don't make sense in a bare repository are
forbidden. But diff-index _can_ work in a bare repository, since there
is no reason you can't do index-only operations. However, a bare repo
has no index by default, in which case diff-index shows the results you
saw.

So no, it's not a bug, but it is potentially confusing. I think nobody
ever worried about it because diff-index is plumbing, so users are
unlikely to stumble into it.

-Peff

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-01-30 20:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-01-30 16:13 diff-index --cached on bare repository Johannes Gilger
2009-01-30 20:06 ` Jeff King

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).