On Wed, Aug 4, 2010 at 9:58 PM, Christian Couder <chriscool@xxxxxxxxxxxxx> wrote: >> > I will >> > try to have a deeper look at that, but it would help if you could give >> > an example of a command that triggers this behavior. >> >> The following patch add "sha1" command. These commands give different sha1: >> >> git sha1 `git rev-parse HEAD` `git rev-parse HEAD^` A >> git sha1 `git rev-parse HEAD` `git rev-parse HEAD^` B > > Yes, but that does not mean that the content of the object returned by > lookup_commit(A) is not the content of A. > > Or do you have an example where the content of the object returned by > lookup_commit(A) is not the content of A? Both return the content of B. I modified my patch a bit to also show the content, ((struct commit*)obj)->buffer. $ ./git cat-file -p HEAD tree 13109340ff2bd55bd16271bbad7a9232f9052923 parent 9a3028b987d73e4c85e8db9980720feba6a1b5c6 parent 2aedccd3d51ec922020f7c7e39df5d2c4d3af515 author Junio C Hamano <gitster@xxxxxxxxx> 1280274846 -0700 committer Junio C Hamano <gitster@xxxxxxxxx> 1280274905 -0700 Sync with 1.7.2.1 Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> $ ./git cat-file -p HEAD^ tree 4ffcd88bed7a675e2d130a41203a7ebe026b6462 parent 9a9fb5d3c4c8601beb2d7b8e3b9283c6c3815a2d author Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> 1279925924 +0000 committer Junio C Hamano <gitster@xxxxxxxxx> 1280124888 -0700 tests: Ignore the Test::Harness .prove file We document how to run prove with the --state option in t/README. This produces a .prove YAML file in the current directory. Change the t/ gitignore to ignore it, and clean it up on `make clean'. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> $ ./git sha1 `git rev-parse HEAD` `git rev-parse HEAD^` A old = ba9523ea809dc496a14f3644bdc1dd6f486983c0 new = 9a3028b987d73e4c85e8db9980720feba6a1b5c6 sha1 = 9a3028b987d73e4c85e8db9980720feba6a1b5c6 tree 4ffcd88bed7a675e2d130a41203a7ebe026b6462 parent 9a9fb5d3c4c8601beb2d7b8e3b9283c6c3815a2d author Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> 1279925924 +0000 committer Junio C Hamano <gitster@xxxxxxxxx> 1280124888 -0700 tests: Ignore the Test::Harness .prove file We document how to run prove with the --state option in t/README. This produces a .prove YAML file in the current directory. Change the t/ gitignore to ignore it, and clean it up on `make clean'. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> $ ./git sha1 `git rev-parse HEAD` `git rev-parse HEAD^` B old = ba9523ea809dc496a14f3644bdc1dd6f486983c0 new = 9a3028b987d73e4c85e8db9980720feba6a1b5c6 sha1 = ba9523ea809dc496a14f3644bdc1dd6f486983c0 tree 4ffcd88bed7a675e2d130a41203a7ebe026b6462 parent 9a9fb5d3c4c8601beb2d7b8e3b9283c6c3815a2d author Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> 1279925924 +0000 committer Junio C Hamano <gitster@xxxxxxxxx> 1280124888 -0700 tests: Ignore the Test::Harness .prove file We document how to run prove with the --state option in t/README. This produces a .prove YAML file in the current directory. Change the t/ gitignore to ignore it, and clean it up on `make clean'. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> -- Duy -- 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