Make WordPress Core

Opened 10 years ago

Closed 4 years ago

#28187 closed defect (bug) (fixed)

EOL of wp-config-sample is now somehow set to LF

Reported by: tai's profile tai Owned by: nacin's profile nacin
Milestone: Priority: normal
Severity: normal Version: 3.9
Component: Site Keywords:
Focuses: Cc:


EOL of wp-config-sample.php in the latest, 3.9.1 is now somehow set to LF.
Suppose to be CRLF. Please see #12775.

Change History (10)

#1 @nacin
10 years ago

Hi tai, it appears to be LF for me. Is this for a localized build of WordPress, or the English zip from

#2 @tai
10 years ago

Hi nacin, it is for the English one.
I've downloadeds it from and found it is set to LF.
Our Japanese version's EOL is set to CRLF.
Should be CRLF, right?

Version 1, edited 10 years ago by tai (previous) (next) (diff)

#4 @DrewAPicture
10 years ago

  • Component changed from General to site
  • Milestone changed from Awaiting Review to
  • Owner set to nacin
  • Status changed from new to reviewing

#5 @chriscct7
9 years ago

  • Keywords needs-nacin added

#6 in reply to: ↑ 3 @SergeyBiryukov
9 years ago

Replying to tai:

EOL of is set to CRLF,
but the one of is set to LF.

Confirmed. This appears to be an unintentional change between 3.7 and 3.8, when and were introduced: (svn:eol-style set to CRLF) (svn:eol-style set to native)

Seems to be changed with

See [2370] and [5457] for the original commits (svn:eol-style changed from native to CRLF).

@dd32, @ocean90: Should this be fixed in

#7 @dd32
9 years ago

Confirmed, this is because our sync script doesn't appear to copy over properties.

I've fixed it up for the 4.4 branch and trunk in but I'm not sure if that'll stick around or if it'll be lost in the next develop -> core sync.

This ticket was mentioned in Slack in #core by dd32. View the logs.

9 years ago

#9 @dd32
9 years ago

  • Keywords needs-nacin removed

Looks like the properties were lost because of which caused a svn rm and a svn add on the following commit.

Now that I've fixed these, I'm fairly certain it'll stay around until an accidental deletion comes along again.

To be super safe, we could also add a svn propset svn:eol-style CRLF wp-config-sample.php license.txt to the svn sync script (FYI @nacin)

#10 @dd32
4 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

I'm going to mark this as fixed, the file in question has been CRLF'd for 4 years.

Note: See TracTickets for help on using tickets.