From 594783cba79960f7a887c498558927f5c47e01f9 Mon Sep 17 00:00:00 2001
From: Stephen Haberman <stephen@exigencecorp.com>
Date: Fri, 14 Nov 2008 22:55:10 -0600
Subject: [PATCH] Remove candidate/topic auto-locking as it was too aggressive.

---
 server/update-stable                   |  31 -------
 tests/t2102-update-stable-candidate.sh | 109 -------------------------
 2 files changed, 140 deletions(-)
 delete mode 100644 tests/t2102-update-stable-candidate.sh

diff --git a/server/update-stable b/server/update-stable
index e7688db..e7e57f2 100644
--- a/server/update-stable
+++ b/server/update-stable
@@ -100,36 +100,5 @@ if [ "$short_refname" == "stable" ] ; then
 		display_error_message "Moving stable must not result in any changes from $second_parent"
 		exit 1
 	fi
-else
-	# Check if candidate/topic is already in stable
-	git branch --contains "$oldrev" | grep --quiet stable
-	if [ $? -eq 0 ] ; then
-		# Okay, it's in stable...but was it previously just a 0-commit initial banch?
-		git rev-list --first-parent stable | grep --quiet "$oldrev"
-		if [ $? -ne 0 ] ; then
-			display_error_message "$short_refname has been merged into stable"
-			exit 1
-		fi
-	fi
-
-	# For now candidates can mix amongst each other so early exit
-	case "$refname" in
-		refs/heads/candidate*)
-			exit 0
-			;;
-		*)
-			;;
-	esac
-
-	# Check if topic is already in candidates
-	candidate=$(git branch --contains "$oldrev" | grep -oP "candidate.*" --max-count=1)
-	if [ $? -eq 0 ] ; then
-		# Okay, it's in stable...but was it previously just a 0-commit initial banch?
-		git rev-list --first-parent stable | grep --quiet "$oldrev"
-		if [ $? -ne 0 ] ; then
-			display_error_message "$short_refname has been merged into $candidate"
-			exit 1
-		fi
-	fi
 fi
 
diff --git a/tests/t2102-update-stable-candidate.sh b/tests/t2102-update-stable-candidate.sh
deleted file mode 100644
index d07af4a..0000000
--- a/tests/t2102-update-stable-candidate.sh
+++ /dev/null
@@ -1,109 +0,0 @@
-#!/bin/sh
-
-test_description='server update candidate enforcer'
-
-. ./test-lib.sh
-
-test_expect_success 'setup' '
-	echo setup >a &&
-	git add a &&
-	git commit -m "a" &&
-	git clone ./. server &&
-	git remote add origin ./server &&
-	rm -fr server/.git/hooks
-'
-
-# setup the update hook
-install_update_hook 'update-stable'
-
-test_expect_success 'initial stable commit works' '
-	# do one stable-less commit
-	echo $test_name >a &&
-	git commit -a -m "$test_name" &&
-	git push origin master &&
-
-	git checkout -b stable &&
-	git push origin stable &&
-	git config --add branch.stable.remote origin &&
-	git config --add branch.stable.merge refs/heads/stable
-'
-
-test_expect_success 'create topic1 and topic2' '
-	git checkout -b topic1 stable &&
-	echo "$test_name topic1" >a.topic1 &&
-	git add a.topic1 &&
-	git commit -m "start topic1" &&
-
-	git checkout -b topic2 stable &&
-	echo "$test_name topic2" >a.topic2 &&
-	git add a.topic2 &&
-	git commit -m "start topic2" &&
-
-	git push origin topic2 topic1
-'
-
-test_expect_success 'create candidate1' '
-	git checkout -b candidate1 stable &&
-	git merge topic1 topic2 &&
-	git push origin candidate1
-'
-
-test_expect_success '0-commit branches are not caught by future stable' '
-	git checkout -b topic4 stable &&
-	git push origin topic4 &&
-
-	echo "$test_name" > a &&
-	git commit -a -m "$test_name" &&
-	git push origin topic4
-'
-
-test_expect_success 'topic1 cannot be changed' '
-	git checkout topic1 &&
-	echo "$test_name" >a.topic1 &&
-	git commit -a -m "$test_name" &&
-	! git push origin topic1 2>push.err &&
-	cat push.err | grep "topic1 has been merged into candidate1"
-'
-
-test_expect_success 'candidate1 can be changed' '
-	git checkout candidate1 &&
-	echo "$test_name" >a.topic1 &&
-	git commit -a -m "$test_name" &&
-	git push origin candidate1
-'
-
-test_expect_success 'merge candidate into stable' '
-	git checkout stable &&
-	git merge candidate1 --no-ff &&
-	git push origin stable
-'
-
-test_expect_success 'candidate cannot be changed' '
-	git checkout candidate1 &&
-	echo "$test_name" >a.topic1 &&
-	git commit -a -m "$test_name" &&
-	! git push origin candidate1 2>push.err &&
-	cat push.err | grep "candidate1 has been merged into stable"
-	! cat push.err | grep "candidate1 has been merged into candidate1"
-'
-
-test_expect_success 'topic1 cannot be changed' '
-	# It is already changed but error message should chagne
-	git checkout topic1 &&
-	! git push origin topic1 2>push.err &&
-	cat push.err | grep "topic1 has been merged into stable" &&
-	git reset --hard HEAD^
-'
-
-test_expect_success 'topic3 can initially be created on stable and then moved' '
-	git checkout -b topic3 stable &&
-	git push origin topic3 &&
-
-	echo "$test_name" >a.topic3 &&
-	git add a.topic3 &&
-	git commit -m "$test_name" &&
-	git push origin topic3
-'
-
-test_done
-
-- 
2.20.1