Re: [PATCH] Add --allow-eof-whitelines to git-apply while still doing whitespace trailing-space fixing.

From: Nanako Shiraishi <nanako3@lavabit.com>
Date: Aug 4, 2009, 21:15 UTC
Message-ID: <20090805061519.6117@nanako3.lavabit.com>
In-Reply-To: <f002290c0908010957pf14d18dv7c80270eb9af3c8@mail.gmail.com>

Quoting Thell Fowler <tbfowler4@gmail.com>

Show 7 quoted lines

>   - Added --allow-eof-whitelines to git apply to short-circuit removal
> of new empty lines when tailing-space is set.
>   - Added pass-through to apply from git-am.sh
>   - Added pass-through to git-am from git-rebase.sh
>   - Added the auto-completion entries.
>
> Signed-off-by: TBFowler <thell@almostautomated.com>

I like this new feature. However, this topic came up last week, and Junio said that whitespace checking should be controlled with the attributes mechanism.

http://thread.gmane.org/gmane.comp.version-control.git/124113/focus=124138

Please do not talk "I did this, I did that" in your proposed commit log message. From Documentation/SubmittingPatches:

	- the body should provide a meaningful commit message, which:
		- uses the imperative, present tense: "change",
		  not "changed" or "changes".
		- includes motivation for the change, and contrasts
		  its implementation with previous behaviour

In other words, the messages are often written as if you are giving an order to the program to correct itself.

-- 
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/