Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] git-rebase.sh: Use POSIX/Susv command substitution instead of backticks

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:44:10

Johannes Schindelin [off-list ref] writes:
And then you have to do it for all scripts in one go.  Mind you, it is not 
really complicated: just one call to perl.
Please do not do this.  If other people have pending changes,
"cleanup for clean-up's sake" would create conflicts for no good
reason.

There are only two cases such a clean-up patch is good:

 (1) When the maintainer is not yet accepting any patches after
     a release-freeze and there is no pending patches from the
     community, and/or if you can convince people with pending
     patches to rebase on top of the clean-up because the
     current codebase is so unmaintainably bad, then a
     whole-tree clean-up patch should go in before anything
     else, forcing everybody to rebase on top of it;

 (2) If you will be working on the code in an area, you may want
     to have the first one in the series a "pure clean-up and
     nothing else" of the whole area, and then build your real
     changes on top.  You still need to coordinate with people
     whose patches may get hit by your clean-ups, but you have
     to do this anyway because you will have conflicts from your
     "real changes".

Any other "clean-up patch" would result in a not-so-appreciated
code churn.  Please don't encourage it.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help