Re: [TopGit PATCH] pre-commit: check .top* files in the to-be-commited tree
flat view
From: Uwe Kleine-König <hidden>
Date: 2016-06-15 22:46:54
Hi Bert, On Tue, Jun 02, 2009 at 10:40:38PM +0200, Bert Wesarg wrote:
We currently check fo these files only in the working tree. But we should check what would be commited. We use write-tree to be able to utilize ls-tree and check the result.
<nitpick> I prefer not speaking about "us" in the commit log. Better use passive voice. </nitpick> So what about: The pre-commit hook used to check that the working copy has the .top* files. Instead of that assert that the tree that is about to be commited has these files.
quoted hunk ↗ jump to hunk
+# I suspect this can't fail, but who knows +tree=$(git write-tree) || + die "Can't write tree"
Actually you don't need to write the tree. You can use: git cat-file -t :0:.topdeps -> blob git cat-file blob :0:.topdeps | grep . Up to now I only shortly looked at your patches, but they seem to be OK. I hope I get around to push the changes later today. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ |