Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: Line endings (a thrilling topic, I know)

Re: Line endings (a thrilling topic, I know)

From: Thomas Martitz <kugel_at_rockbox.org>
Date: Thu, 07 Jul 2011 08:00:36 +0200

Am 07.07.2011 00:53, schrieb Michael Sparmann:
>> I guess this ultimately comes down to: do we want to force anyone who
>> checks out the tree on Windows to work with UNIX line endings so that
>> we don't have to care if our build process handles CRLFs, or do we
>> want to let people configure their Windows git client to taste (it
>> will, by default, convert to CRLF, but individual git users can turn
>> that off) and maybe get potential build issues from it?
>>
> I'm also strongly in favor of forcing LFs on checkout. I've had much
> more trouble with autoconverted files in the past than with software
> that can't deal with LFs. The only editor that I've run across that
> doesn't like LFs is plain old notepad, and also things like VBS files
> seem to work well with LF line endings.
>
> Autoconversion being enabled by default has caused the following problem
> for me several times:
> - I usually use Windows for editing as my compilation box is headless.
> - I tend to use TortoiseSVN from time to time
> - I sometimes do an SVN update on a network share on the linux box,
> using TortoiseSVN running on the windows box.
> - Then I try to compile that checkout on linux, and guess what, all
> those perl scripts are broken, because TortoiseSVN did convert them to
> CRLF, which fails on linux for various reasons.
>
> So actually I expect more trouble from autoconversion than from forcing
> LFs even for windows users.
>
> Just my two cents...
> TheSeven
>

TortoiseGit has more settings in that regard so that shouldn't be a problem.

I'm not sure we need to force anything? Can't we just batch-convert to
LF and then leave things as is?

Best regards.
Received on 2011-07-07


Page was last modified "Jan 10 2012" The Rockbox Crew
aaa