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

[PATCH 1/3] add: use advise_if_enabled for ADVICE_ADD_IGNORED_FILE

From
Rubén Justo <rjusto@gmail.com>
Date
Mar 29, 2024, 04:19 UTC
Message-ID
<37f79186-b849-496a-ae91-f5e05a280dae@gmail.com>
In-Reply-To
<06c9b422-b22e-4310-ad5b-1686616ab860@gmail.com>
Use the newer advise_if_enabled() machinery to show the advice.
Signed-off-by: Rubén Justo <rjusto@gmail.com>
---
 builtin/add.c              | 6 ++----
 t/t3700-add.sh             | 3 +--
 t/t7400-submodule-basic.sh | 3 +--
 3 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/builtin/add.c b/builtin/add.c
index 393c10cbcf..8f148987f7 100644
--- a/builtin/add.c
+++ b/builtin/add.c
@@ -328,10 +328,8 @@ static int add_files(struct dir_struct *dir, int flags)
 		fprintf(stderr, _(ignore_error));
 		for (i = 0; i < dir->ignored_nr; i++)
 			fprintf(stderr, "%s\n", dir->ignored[i]->name);
-		if (advice_enabled(ADVICE_ADD_IGNORED_FILE))
-			advise(_("Use -f if you really want to add them.\n"
-				"Turn this message off by running\n"
-				"\"git config advice.addIgnoredFile false\""));
+		advise_if_enabled(ADVICE_ADD_IGNORED_FILE,
+				  _("Use -f if you really want to add them."));
 		exit_status = 1;
 	}
 
diff --git a/t/t3700-add.sh b/t/t3700-add.sh
index f23d39f0d5..76c2c9e7b0 100755
--- a/t/t3700-add.sh
+++ b/t/t3700-add.sh
@@ -370,8 +370,7 @@ cat >expect.err <<\EOF
 The following paths are ignored by one of your .gitignore files:
 ignored-file
 hint: Use -f if you really want to add them.
-hint: Turn this message off by running
-hint: "git config advice.addIgnoredFile false"
+hint: Disable this message with "git config advice.addIgnoredFile false"
 EOF
 cat >expect.out <<\EOF
 add 'track-this'
diff --git a/t/t7400-submodule-basic.sh b/t/t7400-submodule-basic.sh
index 00c1f1aab1..5c4a89df5c 100755
--- a/t/t7400-submodule-basic.sh
+++ b/t/t7400-submodule-basic.sh
@@ -212,8 +212,7 @@ test_expect_success 'submodule add to .gitignored path fails' '
 		The following paths are ignored by one of your .gitignore files:
 		submod
 		hint: Use -f if you really want to add them.
-		hint: Turn this message off by running
-		hint: "git config advice.addIgnoredFile false"
+		hint: Disable this message with "git config advice.addIgnoredFile false"
 		EOF
 		# Does not use test_commit due to the ignore
 		echo "*" > .gitignore &&
-- 
2.44.0.371.gf9813d4ed5
Previous: Rubén JustoNext: Junio C Hamano
Message 2 of 17 in “add: use advise_if_enabled”
  1. 0/3 add: use advise_if_enabledRubén Justo, Mar 29, 2024
  2. 1/3 add: use advise_if_enabled for ADVICE_ADD_IGNORED_FILERubén Justo, Mar 29, 2024
  3. Junio C HamanoMar 29, 2024
  4. 2/3 add: use advise_if_enabled for ADVICE_ADD_EMPTY_PATHSPECRubén Justo, Mar 29, 2024
  5. 3/3 add: use advise_if_enabled for ADVICE_ADD_EMBEDDED_REPORubén Justo, Mar 29, 2024
  6. Junio C HamanoMar 29, 2024
  7. Rubén JustoMar 29, 2024
  8. Junio C HamanoMar 29, 2024
  9. Rubén JustoMar 29, 2024
  10. Junio C HamanoMar 29, 2024
  11. Rubén JustoMar 30, 2024
  12. Junio C HamanoMar 29, 2024
  13. Rubén JustoMar 29, 2024
  14. 0/3 add: use advise_if_enabledRubén Justo, Mar 30, 2024
  15. 1/3 add: use advise_if_enabled for ADVICE_ADD_IGNORED_FILERubén Justo, Mar 30, 2024
  16. 2/3 add: use advise_if_enabled for ADVICE_ADD_EMPTY_PATHSPECRubén Justo, Mar 30, 2024
  17. 3/3 add: use advise_if_enabled for ADVICE_ADD_EMBEDDED_REPORubén Justo, Mar 30, 2024

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.