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

[RFC PATCH] Makefile: add deprecation message for strip target

From
Bagas Sanjaya <bagasdotme@gmail.com>
Date
Nov 23, 2021, 12:29 UTC
Message-ID
<20211123122934.639428-1-bagasdotme@gmail.com>

Now that $INSTALL_STRIP variable can be defined since 3231f41009 (make: add INSTALL_STRIP option variable, 2021-09-05), deprecate 'strip' target to encourage users to move to $INSTALL_STRIP. The target will eventually be removed in Git 2.35+1.

Only deprecation message is printed.
Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 Makefile | 2 ++
 1 file changed, 2 insertions(+)
diff --git a/Makefile b/Makefile
index 12be39ac49..ee83860f7d 100644
--- a/Makefile
+++ b/Makefile
@@ -2159,6 +2159,8 @@ please_set_SHELL_PATH_to_a_more_modern_shell:
 shell_compatibility_test: please_set_SHELL_PATH_to_a_more_modern_shell
 
 strip: $(PROGRAMS) git$X
+	@echo "The 'strip' target is deprecated, define INSTALL_STRIP if you want to"
+	@echo "install Git with stripped binaries."
 	$(STRIP) $(STRIP_OPTS) $^
 
 ### Flags affecting all rules

base-commit: cd3e606211bb1cf8bc57f7d76bab98cc17a150bc
-- 
An old man doll... just what I always wanted! - Clara
Next: Ævar Arnfjörð Bjarmason
Message 1 of 2 in “Makefile: add deprecation message for strip target”
  1. Makefile: add deprecation message for strip targetBagas Sanjaya, Nov 23, 2021
  2. Ævar Arnfjörð BjarmasonNov 23, 2021

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.