Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [eclipselink-dev] Unification of line endings in the codebase

On 3/18/15 3:40 PM, David Minsky wrote:
+LF

Also consider: any plans to change the date in the copyright headers to 2015 at the same time, since a mass update is being planned for that specific day?

I'm not big fan of these mass updates just because of year has changed, I think it is better to update what actually changed only. But since I'll be touching a lot of files, it perhaps make sense to update the year as well... will think about this

thanks,
--lukas


On 18/03/2015 4:48 AM, Lukas Jungmann wrote:
Hi all,

   currently eclipselink sources contains mix of line endings - some files are using *nix style (LF), some win style (CRLF) and the rest is a mix of *nix and win style (some lines end with LF while others with CRLF). Since we are at the beginning of 2.7 development it looks like a good time to me to unify line endings in our codebase to avoid complete rewrites of files in the future.

Why we should do that?
-make it easier to review incoming as well as own (outgoing) changes
-avoid noise in git log
-use https://help.github.com/articles/dealing-with-line-endings as a guide to properly set up local development environment

Drawbacks:
-one-time full rewrite of some files in the repository
-some backports to previous streams will require manual changes/merges

When?
  if there will be an agreement, I'd like to perform the change next week, on Friday March 27, 2015

Which line endings should be used?
  either *nix or win but definitely not a mix of them in one file

How to cast your vote?
  simply reply to this email with either '+LF', '+CRLF' or 'Leave it as it is' by Thursday March 28, 23:59.

Thanks and happy voting,
--lukas
_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev


--
Oracle
David Minsky | Principal Member of Technical Staff | +1 613-288-4605
Oracle Sustaining Engineering | Application Server
45 O'Connor Street, Suite 400, Ottawa, Ontario, Canada, K1P 1A4


_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev


Back to the top