]> git.eshelyaron.com Git - emacs.git/commitdiff
Encourage https: in commit messages
authorPaul Eggert <eggert@day>
Sun, 15 Oct 2017 02:16:12 +0000 (19:16 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Sun, 15 Oct 2017 02:19:24 +0000 (19:19 -0700)
* CONTRIBUTE: Prefer https: when citing.
* build-aux/git-hooks/commit-msg:
Diagnose http: and ftp: URLs to FSF or GNU hosts.

CONTRIBUTE
build-aux/git-hooks/commit-msg

index 8995bb3eec44c4cc798a551e317d46b60194035a..c7f3330ab01901d831c690114c7078ca83e54a91 100644 (file)
@@ -175,6 +175,9 @@ them right the first time, so here are guidelines for formatting them:
   bug number NNNNN in the debbugs database.  This string is often
   parenthesized, as in "(Bug#19003)".
 
+- When citing URLs, prefer https: to http: when either will do.  In
+  particular, gnu.org and fsf.org URLs should start with "https:".
+
 - Commit messages should contain only printable UTF-8 characters.
 
 - Commit messages should not contain the "Signed-off-by:" lines that
index 39450865cb80be6fed8e55178a7f5b93e1b13fa2..e21aaf074c68e3405c4cab0787c6e340c3c307dc 100755 (executable)
@@ -66,6 +66,8 @@ exec $awk -v at_sign="$at_sign" -v cent_sign="$cent_sign" '
         non_print = "[^[:print:]]"
       }
     }
+    c_lower = "abcdefghijklmnopqrstuvwxyz"
+    unsafe_gnu_url = "(http|ftp)://([" c_lower ".]*\\.)?(gnu|fsf)\\.org"
   }
 
   /^#/ {
@@ -125,6 +127,14 @@ exec $awk -v at_sign="$at_sign" -v cent_sign="$cent_sign" '
     status = 1
   }
 
+  {
+    if (match($0, unsafe_gnu_url)) {
+      url = substr($0, RSTART, RLENGTH)
+      printf "Use https: URL instead of '\''%s'\'' in commit message\n", url
+      status = 1
+    }
+  }
+
   $0 ~ non_print {
     print "Unprintable character in commit message"
     status = 1