Code in CVS is saved with the wrong line ending format

DamienMcKenna - June 24, 2009 - 15:52
Project:CCK Redirection
Version:6.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

All of the files in CVS are saved with CRLF (Windows/DOS style) line endings whereas they should be saved with LF (UNIX/OSX style).

#1

DamienMcKenna - June 24, 2009 - 15:53
Status:active» needs review

The attached file *should* convert all files back to normal line endings.

AttachmentSize
cck_redirection-n501024.patch 7.31 KB

#2

DamienMcKenna - June 24, 2009 - 15:55

Due to the weirdness of dealing with line endings vs creating patch files, I'm not 100% sure if the attached patch will work, but at the very least if each file is opened & re-saved as LF format in any text editor, it will resolve the problem.

#3

DamienMcKenna - June 24, 2009 - 16:01
Priority:normal» critical

Updating the status as it is affecting our ability to do other work, e.g. #335829: No redirect if field is blank.

#4

guix - June 24, 2009 - 20:47

Oh, now I understand why your latest patch was so long ! I tried this patch but cck_redirection.module still is in DOS format and with no EOL, so I ran dos2unix on the file and here's the 30KB patch...

Anyway good idea to submit this issue, let's deal with that first. There are a lot of coder finds too, do you think we should treat them in another issue ?

AttachmentSize
cck_redirection-501024-4.patch 37.24 KB

#5

guix - June 25, 2009 - 16:44

It's corrected in the lastest CVS and 6.x-1.2. The last thing is a remaining "End of line" problem in cck_redirection.info. Then we can start the real work !

AttachmentSize
cck_redirection-501024-5.patch 796 bytes

#6

guix - June 25, 2009 - 16:45
Priority:critical» normal

#7

guix - June 26, 2009 - 05:20

In fact it's corrected in CVS HEAD but NOT in 6.x-1.2. My patch was for CVS HEAD so it still applies.

#8

DamienMcKenna - June 26, 2009 - 17:12

Could someone please fix the v6.x-1.x branch? Thanks.

 
 

Drupal is a registered trademark of Dries Buytaert.