@@ -695,6 +695,7 @@ X =
PROGRAMS += $(patsubst %.o,git-%$X,$(PROGRAM_OBJS))
+TEST_BUILTINS_OBJS += test-advise.o
TEST_BUILTINS_OBJS += test-chmtime.o
TEST_BUILTINS_OBJS += test-config.o
TEST_BUILTINS_OBJS += test-ctype.o
@@ -32,6 +32,40 @@ int advice_checkout_ambiguous_remote_branch_name = 1;
int advice_nested_tag = 1;
int advice_submodule_alternate_error_strategy_die = 1;
+const char ADVICE_ADD_EMBEDDED_REPO[] = "advice.addEmbeddedRepo";
+const char ADVICE_AM_WORK_DIR[] = "advice.amWorkDir";
+const char ADVICE_CHECKOUT_AMBIGUOUS_REMOTE_BRANCH_NAME[] = "advice.checkoutAmbiguousRemoteBranchName";
+const char ADVICE_COMMIT_BEFORE_MERGE[] = "advice.commitBeforeMerge";
+const char ADVICE_DETACHED_HEAD[] = "advice.detachedHead";
+const char ADVICE_FETCH_SHOW_FORCED_UPDATES[] = "advice.fetchShowForcedUpdates";
+const char ADVICE_GRAFT_FILE_DEPRECATED[] = "advice.graftFileDeprecated";
+const char ADVICE_IGNORED_HOOK[] = "advice.ignoredHook";
+const char ADVICE_IMPLICIT_IDENTITY[] = "advice.implicitIdentity";
+const char ADVICE_NESTED_TAG[] = "advice.nestedTag";
+const char ADVICE_OBJECT_NAME_WARNING[] = "advice.objectNameWarning";
+const char ADVICE_PUSH_ALREADY_EXISTS[] = "advice.pushAlreadyExists";
+const char ADVICE_PUSH_FETCH_FIRST[] = "advice.pushFetchFirst";
+const char ADVICE_PUSH_NEEDS_FORCE[] = "advice.pushNeedsForce";
+const char ADVICE_PUSH_NON_FF_CURRENT[] = "advice.pushNonFFCurrent";
+const char ADVICE_PUSH_NON_FF_MATCHING[] = "advice.pushNonFFMatching";
+const char ADVICE_PUSH_UNQUALIFIED_REF_NAME[] = "advice.pushUnqualifiedRefName";
+const char ADVICE_PUSH_UPDATE_REJECTED[] = "advice.pushUpdateRejected";
+
+/* make this an alias for backward compatibility */
+const char ADVICE_PUSH_UPDATE_REJECTED_ALIAS[] = "advice.pushNonFastForward";
+
+const char ADVICE_RESET_QUIET_WARNING[] = "advice.resetQuiet";
+const char ADVICE_RESOLVE_CONFLICT[] = "advice.resolveConflict";
+const char ADVICE_RM_HINTS[] = "advice.rmHints";
+const char ADVICE_SEQUENCER_IN_USE[] = "advice.sequencerInUse";
+const char ADVICE_SET_UPSTREAM_FAILURE[] = "advice.setUpstreamFailure";
+const char ADVICE_STATUS_AHEAD_BEHIND_WARNING[] = "advice.statusAheadBehindWarning";
+const char ADVICE_STATUS_HINTS[] = "advice.statusHints";
+const char ADVICE_STATUS_U_OPTION[] = "advice.statusUoption";
+const char ADVICE_SUBMODULE_ALTERNATE_ERROR_STRATEGY_DIE[] = "advice.submoduleAlternateErrorStrategyDie";
+const char ADVICE_WAITING_FOR_EDITOR[] = "advice.waitingForEditor";
+
+
static int advice_use_color = -1;
static char advice_colors[][COLOR_MAXLEN] = {
GIT_COLOR_RESET,
@@ -96,13 +130,53 @@ static struct {
{ "pushNonFastForward", &advice_push_update_rejected }
};
-static void vadvise(const char *advice, va_list params)
+static const char *advice_config_keys[] = {
+ ADVICE_ADD_EMBEDDED_REPO,
+ ADVICE_AM_WORK_DIR,
+ ADVICE_CHECKOUT_AMBIGUOUS_REMOTE_BRANCH_NAME,
+ ADVICE_COMMIT_BEFORE_MERGE,
+ ADVICE_DETACHED_HEAD,
+ ADVICE_FETCH_SHOW_FORCED_UPDATES,
+ ADVICE_GRAFT_FILE_DEPRECATED,
+ ADVICE_IGNORED_HOOK,
+ ADVICE_IMPLICIT_IDENTITY,
+ ADVICE_NESTED_TAG,
+ ADVICE_OBJECT_NAME_WARNING,
+ ADVICE_PUSH_ALREADY_EXISTS,
+ ADVICE_PUSH_FETCH_FIRST,
+ ADVICE_PUSH_NEEDS_FORCE,
+ ADVICE_PUSH_UPDATE_REJECTED_ALIAS,
+ ADVICE_PUSH_NON_FF_CURRENT,
+ ADVICE_PUSH_NON_FF_MATCHING,
+ ADVICE_PUSH_UNQUALIFIED_REF_NAME,
+ ADVICE_PUSH_UPDATE_REJECTED,
+ ADVICE_RESET_QUIET_WARNING,
+ ADVICE_RESOLVE_CONFLICT,
+ ADVICE_RM_HINTS,
+ ADVICE_SEQUENCER_IN_USE,
+ ADVICE_SET_UPSTREAM_FAILURE,
+ ADVICE_STATUS_AHEAD_BEHIND_WARNING,
+ ADVICE_STATUS_HINTS,
+ ADVICE_STATUS_U_OPTION,
+ ADVICE_SUBMODULE_ALTERNATE_ERROR_STRATEGY_DIE,
+ ADVICE_WAITING_FOR_EDITOR,
+};
+
+static const char turn_off_instructions[] =
+N_("\n"
+ "Disable this message with \"git config %s false\"");
+
+static void vadvise(const char *advice, int display_instructions,
+ const char *key, va_list params)
{
struct strbuf buf = STRBUF_INIT;
const char *cp, *np;
strbuf_vaddf(&buf, advice, params);
+ if (display_instructions)
+ strbuf_addf(&buf, turn_off_instructions, key);
+
for (cp = buf.buf; *cp; cp = np) {
np = strchrnul(cp, '\n');
fprintf(stderr, _("%shint: %.*s%s\n"),
@@ -119,7 +193,36 @@ void advise(const char *advice, ...)
{
va_list params;
va_start(params, advice);
- vadvise(advice, params);
+ vadvise(advice, 0, "", params);
+ va_end(params);
+}
+
+static int get_config_value(const char *advice_key)
+{
+ int value = 1;
+
+ git_config_get_bool(advice_key, &value);
+ return value;
+}
+
+int advice_enabled(const char *advice_key)
+{
+ if (advice_key == ADVICE_PUSH_UPDATE_REJECTED)
+ return get_config_value(ADVICE_PUSH_UPDATE_REJECTED) &&
+ get_config_value(ADVICE_PUSH_UPDATE_REJECTED_ALIAS);
+ else
+ return get_config_value(advice_key);
+}
+
+void advise_if_enabled(const char *advice_key, const char *advice, ...)
+{
+ va_list params;
+
+ if (!advice_enabled(advice_key))
+ return;
+
+ va_start(params, advice);
+ vadvise(advice, 1, advice_key, params);
va_end(params);
}
@@ -159,8 +262,8 @@ void list_config_advices(struct string_list *list, const char *prefix)
{
int i;
- for (i = 0; i < ARRAY_SIZE(advice_config); i++)
- list_config_item(list, prefix, advice_config[i].name);
+ for (i = 0; i < ARRAY_SIZE(advice_config_keys); i++)
+ list_config_item(list, prefix, advice_config_keys[i]);
}
int error_resolve_conflict(const char *me)
@@ -32,9 +32,58 @@ extern int advice_checkout_ambiguous_remote_branch_name;
extern int advice_nested_tag;
extern int advice_submodule_alternate_error_strategy_die;
+/*
+ * To add a new advice, you need to:
+ * Define a new const char array.
+ * Add a new entry to advice_config_keys list.
+ * Add the new config variable to Documentation/config/advice.txt.
+ * Call advise_if_enabled to print your advice.
+ */
+extern const char ADVICE_ADD_EMBEDDED_REPO[];
+extern const char ADVICE_AM_WORK_DIR[];
+extern const char ADVICE_CHECKOUT_AMBIGUOUS_REMOTE_BRANCH_NAME[];
+extern const char ADVICE_COMMIT_BEFORE_MERGE[];
+extern const char ADVICE_DETACHED_HEAD[];
+extern const char ADVICE_FETCH_SHOW_FORCED_UPDATES[];
+extern const char ADVICE_GRAFT_FILE_DEPRECATED[];
+extern const char ADVICE_IGNORED_HOOK[];
+extern const char ADVICE_IMPLICIT_IDENTITY[];
+extern const char ADVICE_NESTED_TAG[];
+extern const char ADVICE_OBJECT_NAME_WARNING[];
+extern const char ADVICE_PUSH_ALREADY_EXISTS[];
+extern const char ADVICE_PUSH_FETCH_FIRST[];
+extern const char ADVICE_PUSH_NEEDS_FORCE[];
+extern const char ADVICE_PUSH_NON_FF_CURRENT[];
+extern const char ADVICE_PUSH_NON_FF_MATCHING[];
+extern const char ADVICE_PUSH_UNQUALIFIED_REF_NAME[];
+extern const char ADVICE_PUSH_UPDATE_REJECTED_ALIAS[];
+extern const char ADVICE_PUSH_UPDATE_REJECTED[];
+extern const char ADVICE_RESET_QUIET_WARNING[];
+extern const char ADVICE_RESOLVE_CONFLICT[];
+extern const char ADVICE_RM_HINTS[];
+extern const char ADVICE_SEQUENCER_IN_USE[];
+extern const char ADVICE_SET_UPSTREAM_FAILURE[];
+extern const char ADVICE_STATUS_AHEAD_BEHIND_WARNING[];
+extern const char ADVICE_STATUS_HINTS[];
+extern const char ADVICE_STATUS_U_OPTION[];
+extern const char ADVICE_SUBMODULE_ALTERNATE_ERROR_STRATEGY_DIE[];
+extern const char ADVICE_WAITING_FOR_EDITOR[];
+
int git_default_advice_config(const char *var, const char *value);
__attribute__((format (printf, 1, 2)))
void advise(const char *advice, ...);
+
+/**
+ * Checks if advice type is enabled (can be printed to the user).
+ * Should be called before advise().
+ */
+int advice_enabled(const char *advice_key);
+
+/**
+ * Checks the visibility of the advice before printing.
+ */
+void advise_if_enabled(const char *advice_key, const char *advice, ...);
+
int error_resolve_conflict(const char *me);
void NORETURN die_resolve_conflict(const char *me);
void NORETURN die_conclude_merge(void);
new file mode 100644
@@ -0,0 +1,19 @@
+#include "test-tool.h"
+#include "cache.h"
+#include "advice.h"
+
+int cmd__advise_if_enabled(int argc, const char **argv)
+{
+ if (!argv[1])
+ die("usage: %s <advice>", argv[0]);
+
+ setup_git_directory();
+
+ /*
+ Any advice type can be used for testing, but NESTED_TAG was selected
+ here and in t0018 where this command is being executed.
+ */
+ advise_if_enabled(ADVICE_NESTED_TAG, argv[1]);
+
+ return 0;
+}
@@ -14,6 +14,7 @@ struct test_cmd {
};
static struct test_cmd cmds[] = {
+ { "advise", cmd__advise_if_enabled },
{ "chmtime", cmd__chmtime },
{ "config", cmd__config },
{ "ctype", cmd__ctype },
@@ -4,6 +4,7 @@
#define USE_THE_INDEX_COMPATIBILITY_MACROS
#include "git-compat-util.h"
+int cmd__advise_if_enabled(int argc, const char **argv);
int cmd__chmtime(int argc, const char **argv);
int cmd__config(int argc, const char **argv);
int cmd__ctype(int argc, const char **argv);
new file mode 100755
@@ -0,0 +1,32 @@
+#!/bin/sh
+
+test_description='Test advise_if_enabled functionality'
+
+. ./test-lib.sh
+
+test_expect_success 'advice should be printed when config variable is unset' '
+ cat >expect <<-\EOF &&
+ hint: This is a piece of advice
+ hint: Disable this message with "git config advice.nestedTag false"
+ EOF
+ test-tool advise "This is a piece of advice" 2>actual &&
+ test_i18ncmp expect actual
+'
+
+test_expect_success 'advice should be printed when config variable is set to true' '
+ cat >expect <<-\EOF &&
+ hint: This is a piece of advice
+ hint: Disable this message with "git config advice.nestedTag false"
+ EOF
+ test_config advice.nestedTag true &&
+ test-tool advise "This is a piece of advice" 2>actual &&
+ test_i18ncmp expect actual
+'
+
+test_expect_success 'advice should not be printed when config variable is set to false' '
+ test_config advice.nestedTag false &&
+ test-tool advise "This is a piece of advice" 2>actual &&
+ test_must_be_empty actual
+'
+
+test_done