git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[TopGit PATCH] hooks/pre-commit: check for deps repititions

From
Bert Wesarg <bert.wesarg@googlemail.com>
Date
Oct 4, 2010, 21:22 UTC
Message-ID
<1286227337-8934-1-git-send-email-bert.wesarg@googlemail.com>
In-Reply-To
<1286227130-8306-1-git-send-email-bert.wesarg@googlemail.com>
A dep should only be listed once in .topdeps, force this.
Signed-off-by: Bert Wesarg <bert.wesarg@googlemail.com>
---
 hooks/pre-commit.sh |   12 ++++++++++++
 1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/hooks/pre-commit.sh b/hooks/pre-commit.sh
index f6901e4..e8986d6 100644 hooks/pre-commit.sh
--- a/hooks/pre-commit.sh
+++ b/hooks/pre-commit.sh
@@ -67,3 +67,15 @@ BEGIN      { in_hunk = 0; }
 		# therefore no endless loop in the cycle-check
 		no_remotes=1 recurse_deps check_cycle_name "$newly_added"
 	done
+
+# check for repititions of deps
+depdir="$(mktemp -t -d tg-depdir.XXXXXX)" ||
+	die "Can't check for mulitple occurrences of deps"
+trap "rm -rf '$depdir'" 0
+cat_file "(i):.topdeps" |
+	while read dep; do
+		[ ! -d "$depdir/$dep" ] ||
+			die "Mulitple occurrences of the same dep: $dep"
+		mkdir -p "$depdir/$dep" ||
+			die "Can't check for mulitple occurrences of deps"
+	done
-- 
tg: (a2bfc32..) bw/check-for-repetitions (depends on: bw/check-valid-deps)
Previous: Bert Wesarg
Message 3 of 3 in “hooks/pre-commit: check for cycles in dependencies”
  1. hooks/pre-commit: check for cycles in dependenciesBert Wesarg, Oct 4, 2010
  2. pre-commit: check .topdeps for valid branchesBert Wesarg, Oct 4, 2010
  3. hooks/pre-commit: check for deps repititionsBert Wesarg, Oct 4, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.