From: Damien <damien@dam.io>
To: git@vger.kernel.org
Subject: [PATCH] run-command.c: add hint when hook is not executable
Date: Tue, 3 Oct 2017 10:59:37 +0000 [thread overview]
Message-ID: <0102015ee1e41f17-927a8da1-ac14-4399-8424-fee8a82c2b0a-000000@eu-west-1.amazonses.com> (raw)
---
Documentation/config.txt | 2 ++
advice.c | 2 ++
advice.h | 1 +
contrib/completion/git-completion.bash | 1 +
run-command.c | 4 ++++
5 files changed, 10 insertions(+)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index 1ac0ae6adb046..83b1884cf22fc 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -351,6 +351,8 @@ advice.*::
addEmbeddedRepo::
Advice on what to do when you've accidentally added one
git repo inside of another.
+ hookNotExecutable::
+ Shown when an hook is there but not executable.
--
core.fileMode::
diff --git a/advice.c b/advice.c
index d81e1cb7425b0..969ba149daeec 100644
--- a/advice.c
+++ b/advice.c
@@ -17,6 +17,7 @@ int advice_set_upstream_failure = 1;
int advice_object_name_warning = 1;
int advice_rm_hints = 1;
int advice_add_embedded_repo = 1;
+int advice_hook_not_executable = 1;
static struct {
const char *name;
@@ -38,6 +39,7 @@ static struct {
{ "objectnamewarning", &advice_object_name_warning },
{ "rmhints", &advice_rm_hints },
{ "addembeddedrepo", &advice_add_embedded_repo },
+ { "hooknotexecutable", &advice_hook_not_executable},
/* make this an alias for backward compatibility */
{ "pushnonfastforward", &advice_push_update_rejected }
diff --git a/advice.h b/advice.h
index c84a44531c7d8..061492976b362 100644
--- a/advice.h
+++ b/advice.h
@@ -19,6 +19,7 @@ extern int advice_set_upstream_failure;
extern int advice_object_name_warning;
extern int advice_rm_hints;
extern int advice_add_embedded_repo;
+extern int advice_hook_not_executable;
int git_default_advice_config(const char *var, const char *value);
__attribute__((format (printf, 1, 2)))
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index d93441747523a..6324db0c44f17 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -2350,6 +2350,7 @@ _git_config ()
advice.rmHints
advice.statusHints
advice.statusUoption
+ advice.hookNotExecutable
alias.
am.keepcr
am.threeWay
diff --git a/run-command.c b/run-command.c
index b5e6eb37c0eb3..95d93a23bf87e 100644
--- a/run-command.c
+++ b/run-command.c
@@ -1174,6 +1174,10 @@ const char *find_hook(const char *name)
if (access(path.buf, X_OK) >= 0)
return path.buf;
#endif
+ if (advice_hook_not_executable) {
+ advise(_("The '%s' hook was ignored because it's not set as executable."
+ "\nYou can disable this warning with `git config advice.hookNotExecutable false`"), name);
+ }
return NULL;
}
return path.buf;
--
https://github.com/git/git/pull/411
next reply other threads:[~2017-10-03 10:59 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-03 10:59 Damien [this message]
2017-10-04 4:40 ` [PATCH] run-command.c: add hint when hook is not executable Junio C Hamano
2017-10-05 20:48 ` Damien
2017-10-05 20:53 ` [PATCH v2] run-command: add hint when a hook is ignored Damien Marié
2017-10-06 4:52 ` Junio C Hamano
2017-10-06 5:25 ` Junio C Hamano
2017-10-06 5:43 ` Junio C Hamano
2017-10-06 5:53 ` Junio C Hamano
2017-10-06 8:04 ` Damien
2017-10-06 8:07 ` [PATCH v3] " Damien Marié
2017-10-10 4:21 ` Junio C Hamano
2017-10-11 6:26 ` Junio C Hamano
2018-01-03 8:31 ` Jeff King
2018-01-05 19:36 ` Junio C Hamano
2018-03-03 7:16 ` Jeff King
2018-03-05 21:19 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0102015ee1e41f17-927a8da1-ac14-4399-8424-fee8a82c2b0a-000000@eu-west-1.amazonses.com \
--to=damien@dam.io \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).