delphi-blocks / WiRL

WiRL: RESTful Library for Delphi
https://wirl.delphiblocks.dev/
Apache License 2.0
237 stars 84 forks source link

Several sources have inconsistent line endings #54

Closed davideangeli closed 3 months ago

davideangeli commented 4 months ago

I'm recompiling the latest version I just downloaded and since I got some strange errors during compilation (Delphi 12.1) I noticed that there are several sources that only have the LF and not the CRLF at the end of each line. I fixed the sources with Project Magician and now I no longer have compilation problems. Is this a problem with my source clone from GitHub or does the problem lie at the source?

paolo-rossi commented 4 months ago

Hello Davide,

do you have a list of such files? so I can easily convert them..

Thanks Paolo.

davideangeli commented 3 months ago

Hello Paolo,

I only checked the ".pas" files and from the list below I can say that the majority have the CRLF inconsistency and I hope I haven't forgotten any files... Maybe you'd better reopen them all for safety with a tool that fix CRLFs (like Uwe Raabe's Project Magician). I've just pulled the latest available version (yesterday) and these are the pas files with the problem:

paolo-rossi commented 3 months ago

Davide,

I checked with several tools and I can't find CR or LF alone in these files, there are only CRLF, perhaps some git configuration on your part?

Paolo.

davideangeli commented 3 months ago

Paolo,

Maybe... but still I can't understand why there are some files with the correct CRLF and others not... I'm investigating on git. I use SmartGit to pull sources from GitHub but only with WiRL am I experiencing this anomaly.

Davide

davideangeli commented 3 months ago

Paolo,

I think that could be something on github. I've just downloaded the zip file from it and the files inside have the problem. This is one of them opened with the Total Commander hexadecimal viewer:

image

Davide

paolo-rossi commented 3 months ago

Davide,

yes, I don't know why but the zip download from GitHub has all the files with LF (Unix style) but I checked other repositories and it's all the same as WiRL.. so, yes probably has something to do with GitHub.

I'm going to close this with this comment given the sources itself are not affected but I'll investigate further if there is some GitHub configuration for the zip download..

Paolo.

davideangeli commented 3 months ago

Paolo,

in the zip file not all the files have the problem, I've the exact situation that I've pulling the sources with smartgit... For instance the file "WiRL.Client.Application.pas" is correct in the zip file as it is correct in my local repository. It would seem that the files stored on github bring the problem with them. I don't know if I can explain myself well!