For leaving trailing blanks around - I plead guilty.
I am also an emacs user.
I have added:
(add-hook 'before-save-hook 'delete-trailing-whitespace)
to my emacs initialization. That should help in the future
Philip
On 2/21/22 01:19, mark_at_yahoo via Rosegarden-devel wrote:
On 2/20/22 3:19 PM, Ted Felix wrote:
On 2/20/22 4:03 PM, mark_at_yahoo via Rosegarden-devel wrote:
2) "Hmmm, grumble, OK. But first commit each file with just the
whitespace changes so we can run git-diff (diff, kompare, meld,
whatever) without seeing spurious changes or having to add the "-w"
commandline option."
This sounds good. That way we don't have to look at whitespace
issues when bisecting to find problems.
Will do.
Again, just the files you are doing work on. Leave the rest
alone. Want to keep git blame as relevant as possible in case we need
it.
Yes, absolutely. Apologies for leaving that out. My Emacs stuff
doesn't modify files (whitespace or anything else) if they're simply
being viewed, only if they're being edited and re-written to disk. And
it's certainly not run by default on every the file in the repository.
No. Bring on the code. I can't promise very fast turnaround (we
have two top priority user requests in play right now), but I will
get to it. If you can get someone else on the team reviewing and
testing, that would help me immensely. I need two sets of eyes on
every change at the user and developer levels.
Not a problem. It's going to take me some time to implement my
proposal. However long that is, schedule the Rosegarden team's review,
test, and possible merge at your convenience. I'll also advertise for
review and test volunteers on the mailing list and/or feature request
ticket.
_______________________________________________
Rosegarden-devel mailing list
Rosegarden-devel@lists.sourceforge.net - use the link below to
unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
_______________________________________________
Rosegarden-devel mailing list
Rosegarden-devel@lists.sourceforge.net - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
