unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#20401: refill long lines in generated ChangeLog
@ 2015-04-21 20:48 Glenn Morris
  2015-04-26  1:24 ` Paul Eggert
  0 siblings, 1 reply; 4+ messages in thread
From: Glenn Morris @ 2015-04-21 20:48 UTC (permalink / raw)
  To: 20401

Package: emacs
Version: 25.0.50
Severity: minor

`make ChangeLog' creates entries longer than 80 columns; see example.
I had hoped that it would refill overly long lines.

2015-04-20  Glenn Morris  <rgm@gnu.org>

	Avoid non-nil current-load-list at startup
	* src/process.c (init_process_emacs): Move Fprovide statement...
	(syms_of_process): ... to here.

	* lisp/loadup.el (custom-current-group-alist): Reset before dumping.

	* lisp/startup.el (command-line) <site-run-file>: Avoid rogue value in emacs -Q.


Many bonus points if it fills it the "smart" way that I would have done
had I written it by hand. ;)

        * lisp/startup.el (command-line) <site-run-file>:
        Avoid rogue value in emacs -Q.

Yes, it's probably my fault. I was "forced" into it by git's IMO silly
"if more than one line, second line should be blank" convention.

But it's not just me that has written lines slightly too long.





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#20401: refill long lines in generated ChangeLog
  2015-04-21 20:48 bug#20401: refill long lines in generated ChangeLog Glenn Morris
@ 2015-04-26  1:24 ` Paul Eggert
  2015-04-27 18:25   ` Glenn Morris
  0 siblings, 1 reply; 4+ messages in thread
From: Paul Eggert @ 2015-04-26  1:24 UTC (permalink / raw)
  To: Glenn Morris; +Cc: 20401

[-- Attachment #1: Type: text/plain, Size: 773 bytes --]

> it's not just me that has written lines slightly too long.

That's right, lots of people are doing it, and it makes more busywork when 
cleaning up ChangeLogs, which sucks.  I don't have much faith in fixing this 
automagically, as a lot could go wrong.

There's a much simpler fix (see attached) but Stefan nixed it (see 
<http://lists.gnu.org/archive/html/emacs-devel/2014-12/msg01640.html>) for 
reasons that I don't follow.  Perhaps we could take another run at him and he'll 
relent this time.

Another possibility is that we set the tab-width in ChangeLog files to 2, so 
that 78-character log lines fit in ChangeLog files when displayed via Emacs. 
(I'm *sure* nobody would complain about *that*. :-)

But really, it'd be simpler to shrink that "78" back to "72".

[-- Attachment #2: 0001-Encourage-shorter-commit-message-lines.patch --]
[-- Type: text/x-patch, Size: 938 bytes --]

From 9ba2c1fc35590c41fba623f892af7dd9c75e5f5b Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Sat, 25 Apr 2015 18:14:35 -0700
Subject: [PATCH] Encourage shorter commit message lines

This helps keep generated ChangeLog lines less than 80 characters
(Bug#20401).
* build-aux/git-hooks/commit-msg: Limit lines to 72 characters.
---
 build-aux/git-hooks/commit-msg | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/build-aux/git-hooks/commit-msg b/build-aux/git-hooks/commit-msg
index 9661376..5e46d9f 100755
--- a/build-aux/git-hooks/commit-msg
+++ b/build-aux/git-hooks/commit-msg
@@ -110,8 +110,8 @@ exec $awk -v at_sign="$at_sign" -v cent_sign="$cent_sign" '
     }
   }
 
-  78 < length && $0 ~ space {
-    print "Line longer than 78 characters in commit message"
+  72 < length && $0 ~ space {
+    print "Line longer than 72 characters in commit message"
     status = 1
   }
 
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* bug#20401: refill long lines in generated ChangeLog
  2015-04-26  1:24 ` Paul Eggert
@ 2015-04-27 18:25   ` Glenn Morris
  2015-04-27 18:37     ` Glenn Morris
  0 siblings, 1 reply; 4+ messages in thread
From: Glenn Morris @ 2015-04-27 18:25 UTC (permalink / raw)
  To: Paul Eggert; +Cc: 20401

Paul Eggert wrote:

> I don't have much faith in fixing this automagically, as a lot could
> go wrong.

You're probably right. (Perhaps we could use Emacs to fill the entries tho'.)





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#20401: refill long lines in generated ChangeLog
  2015-04-27 18:25   ` Glenn Morris
@ 2015-04-27 18:37     ` Glenn Morris
  0 siblings, 0 replies; 4+ messages in thread
From: Glenn Morris @ 2015-04-27 18:37 UTC (permalink / raw)
  To: Paul Eggert; +Cc: 20401


PS I'd appreciate the suggested font-lock warning for "long" VC log lines
(Though I commit from the released version, so it will be a while before
I can take advantage of it.)





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-04-27 18:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-21 20:48 bug#20401: refill long lines in generated ChangeLog Glenn Morris
2015-04-26  1:24 ` Paul Eggert
2015-04-27 18:25   ` Glenn Morris
2015-04-27 18:37     ` Glenn Morris

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.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).