On Sat, Dec 05, 2009 at 10:24:46AM -0800, Junio C Hamano wrote: > > ... The test already checks that the change won't break soft reset, > > which is good, but it does not seem to check/specify what should happen in > > the mixed reset in this case (I think it should be allowed). > > Heh, I was not thinking straight. A bare repository does not have the > index, so allowing (cd .git && git reset) is Ok but mixed in a bare > repository (cd bare.git && git reset) is not. Hmm. I would have thought it would be allowed in a bare repository, to explicitly let people treat the bare repo as a pseudo-database, just pulling out the files when they want to. And I was all set to argue against restricting it, but looking at your tests, it seems we already disallow it. So I don't see a harm in verifying the current behavior. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html