Hi all,
changes are in master. I've just check on windows that it should
be enough to follow
https://help.github.com/articles/dealing-with-line-endings/#platform-windows
:
git pull -r
git config --global core.autocrlf true
git rm --cached -r .
git reset --hard
on linux/mac:
git pull -r
git config --global core.autocrlf input
git rm --cached -r .
git reset --hard
...or just create a new fresh clone.
Thanks,
--lukas
On 3/30/15 3:01 AM, Lukas Jungmann
wrote:
Hi,
since all were for LF, I went ahead and prepared the patch,
everything is attached in
https://bugs.eclipse.org/bugs/show_bug.cgi?id=463403
What has been covered:
-all files except of *.cmd were converted to unix format (LF)
-99% of tabs has been replaced by 4 spaces - 1% are moxy specific
test files
-trailing whitespaces has been removed in 99% cases, again some
had to stay due to moxy tests
-copyright year has been updated to 2015 in all files
as for code reformatting - I haven't found simple way to run
checkstyle and set up everything around it, so I'd prefer to not
do that as part of this, it can be done incrementally during the
time
If there will be no comments and/or no one will disagree with this
change as is, I'll apply it to master as soon as the review
request in the bug gets at least 2 '+'.
Thanks,
--lukas
On 3/18/15 9: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
_______________________________________________
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
|
