Make WordPress Core

Opened 7 years ago

Closed 21 months ago

#28187 closed defect (bug) (fixed)

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

Reported by: tai Owned by: nacin
Milestone: WordPress.org Priority: normal
Severity: normal Version: 3.9
Component: WordPress.org 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
7 years ago

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

#2 @tai
7 years ago

Hi nacin, it is for the English one.
I've downloaded it from http://wordpress.org/latest.zip and found it is set to LF.
Our Japanese version's EOL is set to CRLF.
Should be CRLF, right?

Last edited 7 years ago by tai (previous) (diff)

#4 @DrewAPicture
7 years ago

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

#5 @chriscct7
6 years ago

  • Keywords needs-nacin added

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

Replying to tai:

EOL of https://develop.svn.wordpress.org/tags/3.9.1/wp-config-sample.php is set to CRLF,
but the one of http://core.svn.wordpress.org/tags/3.9.1/wp-config-sample.php is set to LF.

Confirmed. This appears to be an unintentional change between 3.7 and 3.8, when develop.svn.wordpress.org and build.trac.wordpress.org were introduced:

https://build.trac.wordpress.org/browser/tags/3.7/wp-config-sample.php (svn:eol-style set to CRLF)
https://build.trac.wordpress.org/browser/tags/3.8/wp-config-sample.php (svn:eol-style set to native)

Seems to be changed with https://build.trac.wordpress.org/browser/trunk/wp-config-sample.php?rev=25861.

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

@dd32, @ocean90: Should this be fixed in https://core.svn.wordpress.org/?

#7 @dd32
6 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 https://build.trac.wordpress.org/changeset/35925 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.

6 years ago

#9 @dd32
6 years ago

  • Keywords needs-nacin removed

Looks like the properties were lost because of https://build.trac.wordpress.org/changeset/25860/ 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
21 months 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.