[PATCH 4/4] refs: remove the last remnants of core.preferSymlinkRefs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This step is for some time after Git 3.0.

Now it is almost N years since we removed the support for the
variable at Git 3.0 boundary, it is time to remove the warning about
this ancient variable and the behaviour change we went through with
it.

This concludes the journey to make sure we no longer create a
symbolic link to represent a symref, while still recognising
existing ones.

Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx>
---
 Documentation/config/core.txt |  5 -----
 config.c                      |  5 -----
 environment.c                 |  1 -
 environment.h                 |  1 -
 refs/files-backend.c          |  5 -----
 t/t0600-reffiles-backend.sh   | 15 ++-------------
 6 files changed, 2 insertions(+), 30 deletions(-)

diff --git a/Documentation/config/core.txt b/Documentation/config/core.txt
index a6f67cab27..1887da78e3 100644
--- a/Documentation/config/core.txt
+++ b/Documentation/config/core.txt
@@ -285,11 +285,6 @@ CIFS/Microsoft Windows.
 +
 False by default.
 
-core.preferSymlinkRefs (removed)::
-	Instead of the default "symref" format for HEAD and other
-	symbolic reference files, use symbolic links.  The support
-	for this variable was dropped in Git 3.0.
-
 core.alternateRefsCommand::
 	When advertising tips of available history from an alternate, use the shell to
 	execute the specified command instead of linkgit:git-for-each-ref[1]. The
diff --git a/config.c b/config.c
index 56b5862e59..8693cd510d 100644
--- a/config.c
+++ b/config.c
@@ -1445,11 +1445,6 @@ static int git_default_core_config(const char *var, const char *value,
 		return 0;
 	}
 
-	if (!strcmp(var, "core.prefersymlinkrefs")) {
-		prefer_symlink_refs = git_config_bool(var, value);
-		return 0;
-	}
-
 	if (!strcmp(var, "core.logallrefupdates")) {
 		if (value && !strcasecmp(value, "always"))
 			log_all_ref_updates = LOG_REFS_ALWAYS;
diff --git a/environment.c b/environment.c
index 1d6c48b52d..2c3a60149b 100644
--- a/environment.c
+++ b/environment.c
@@ -40,7 +40,6 @@ int has_symlinks = 1;
 int minimum_abbrev = 4, default_abbrev = -1;
 int ignore_case;
 int assume_unchanged;
-int prefer_symlink_refs;
 int is_bare_repository_cfg = -1; /* unspecified */
 int warn_ambiguous_refs = 1;
 int warn_on_object_refname_ambiguity = 1;
diff --git a/environment.h b/environment.h
index 0148738ed6..6b595bd210 100644
--- a/environment.h
+++ b/environment.h
@@ -127,7 +127,6 @@ extern int has_symlinks;
 extern int minimum_abbrev, default_abbrev;
 extern int ignore_case;
 extern int assume_unchanged;
-extern int prefer_symlink_refs;
 extern int warn_ambiguous_refs;
 extern int warn_on_object_refname_ambiguity;
 extern char *apply_default_whitespace;
diff --git a/refs/files-backend.c b/refs/files-backend.c
index 1296272252..eef01fee06 100644
--- a/refs/files-backend.c
+++ b/refs/files-backend.c
@@ -2983,11 +2983,6 @@ static int files_transaction_finish(struct ref_store *ref_store,
 			}
 		}
 
-		/* Warn against core.preferSymlinkRefs set to true */
-		if (update->new_target && prefer_symlink_refs)
-			/* we used to, but no longer, create a symlink here */
-			warning("core.preferSymlinkRefs was removed in Git 3.0");
-
 		if (update->flags & REF_NEEDS_COMMIT) {
 			clear_loose_ref_cache(refs);
 			if (commit_ref(lock)) {
diff --git a/t/t0600-reffiles-backend.sh b/t/t0600-reffiles-backend.sh
index 4e517cdc13..4db86757ac 100755
--- a/t/t0600-reffiles-backend.sh
+++ b/t/t0600-reffiles-backend.sh
@@ -480,22 +480,12 @@ test_expect_success SYMLINKS 'symlinks used as symrefs are still supported' '
 	test_cmp actual expect
 '
 
-test_expect_success 'core.prefersymlinkrefs gets a warning' '
+test_expect_success 'core.prefersymlinkrefs no longer gets a warning' '
 	test_when_finished "git symbolic-ref -d TEST_SYMREF_HEAD || :" &&
 	git update-ref refs/heads/new HEAD &&
 
 	test_config core.prefersymlinkrefs true &&
 	git symbolic-ref TEST_SYMREF_HEAD refs/heads/new 2>stderr &&
-	test_grep "core\.preferSymlinkRefs was removed" stderr &&
-
-	git symbolic-ref -d TEST_SYMREF_HEAD &&
-	git config core.prefersymlinkrefs false &&
-	git symbolic-ref TEST_SYMREF_HEAD refs/heads/new 2>stderr &&
-	test_grep ! "core\.preferSymlinkRefs was removed" stderr &&
-
-	git symbolic-ref -d TEST_SYMREF_HEAD &&
-	git config --unset core.prefersymlinkrefs &&
-	git symbolic-ref TEST_SYMREF_HEAD refs/heads/new 2>stderr &&
 	test_grep ! "core\.preferSymlinkRefs was removed" stderr
 '
 
@@ -512,8 +502,7 @@ test_expect_success 'symref transaction' '
 	test_path_is_file .git/TEST_SYMREF_HEAD &&
 	git symbolic-ref TEST_SYMREF_HEAD >actual &&
 	echo refs/heads/new >expect &&
-	test_cmp expect actual &&
-	test_grep ! "core\.preferSymlinkRefs was removed" stderr
+	test_cmp expect actual
 '
 
 test_done
-- 
2.46.1-742-g4240f61078





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux