On 6 Nov., 13:40, Jeroen Demeyer <jdeme...@cage.ugent.be> wrote: > On 2011-10-25 08:37, Jeroen Demeyer wrote: > > > The following would be easy for me to do: remove all *newly added* > > trailing whitespace from patches (except on lines consisting only of > > spaces). Basically, s/^\(+.*[^ ]\) *$/\1/ in the patches. > > I just tried to do this for sage-4.8.alpha1 and the problem is reviewer > patches. If a ticket has a "main" patch adding new trailing whitespace > and a "reviewer" ticket changing those lines, then removing new trailing > whitespace from patches creates conflicts.
Well, of course you'd have to - first clean up the whole Sage library once (removing trailing whitespace), - then for every patch to the Sage library, remove trailing whitespace from deleted (-), added (+) as well as context ( ) lines (the latter perhaps less important, as it will mainly remove fuzz). -leif -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org
