unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
From: Thien-Thi Nguyen <ttn@gnuvola.org>
To: guile-devel@gnu.org
Subject: Re: rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace)
Date: Sat, 23 Jan 2010 01:38:30 +0100	[thread overview]
Message-ID: <87636tr6yh.fsf@ambire.localdomain> (raw)
In-Reply-To: <87eilj2nkt.fsf@ossau.uklinux.net> (Neil Jerram's message of "Thu, 21 Jan 2010 20:46:26 +0000")

() Neil Jerram <neil@ossau.uklinux.net>
() Thu, 21 Jan 2010 20:46:26 +0000

   > Jim Meyering gives a nice rationale in
   > <http://old.nabble.com/whitespace-cleanup-td6828228.html>.

   I'm afraid those rationales don't persuade me: [...]

Well, i respect the work Jim Meyering does and read his rationale
w/ a more general mindset.  I agree Guile is not GNU coreutils,
and have elided the "does not apply" parts, w/ the exception of:

   > [merge conflicts]

   Hasn't been a problem in practice AFAIK.  I've been doing a lot
   of merging at work recently, with a lot of real merge
   conflicts.  The few that are caused by whitespace are easy to
   spot, and give me a pleasant interlude between the ones that
   are really difficult.

If there is a whitespace policy, we can eliminate those conflicts
(by rejecting non-compliant patches).  This is good!

   If you are concerned that your hook is going to generate diffs
   that other developers might think are spurious - I'd say don't
   worry about that.  I personally won't object to that, and I
   don't think Andy or Ludo would either.

I would rather clarify the policy (either way) and then adopt it,
than go w/o policy.  If the policy (either way) is clear, i know
how to avoid introducing spurious diffs and being inconvenienced
by them from others, and can help others do the same.

   If you anticipate some other practical problem, can you say
   more about what it is?

The practical problem is that w/o policy spurious diffs come and
go, instead of being addressed once (one-shot big-commit if policy
is "deny", a blurb in HACKING to justly orient newbies if "allow").
This is "practical" in the sense of "what people will do in
practice".  It is a "problem" in the sense of "i hate burning my
geezer brain-cells on administrivia over and over and over...".

Revised proposal: Let's decide one way or the other, record the
decision (including link to this thread), and bask in the time and
effort saved by our forward thinking when the issue resurfaces w/
new programmers.

thi




  reply	other threads:[~2010-01-23  0:38 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-13  9:06 rfc: (add-hook 'before-save-hook 'delete-trailing-whitespace) Thien-Thi Nguyen
2010-01-13 10:53 ` Ludovic Courtès
2010-01-13 14:22   ` Thien-Thi Nguyen
2010-01-21 20:31   ` Neil Jerram
2010-01-21 20:44     ` Ludovic Courtès
2010-01-13 20:02 ` Andy Wingo
2010-01-13 20:45   ` Thien-Thi Nguyen
2010-01-14  0:10     ` Ludovic Courtès
2010-01-21 20:46 ` Neil Jerram
2010-01-23  0:38   ` Thien-Thi Nguyen [this message]
2010-01-23 16:10     ` Ludovic Courtès
2010-02-09 13:30       ` Thien-Thi Nguyen
2010-02-14 14:20         ` Ludovic Courtès

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: https://www.gnu.org/software/guile/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87636tr6yh.fsf@ambire.localdomain \
    --to=ttn@gnuvola.org \
    --cc=guile-devel@gnu.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.
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).