Opened 12 months ago

Last modified 12 months ago

#20890 new defect (bug)

Standardize line endings of wp-config-sample.php and wp-fullscreen.css

Reported by: xknown Owned by:
Priority: normal Milestone: Awaiting Review
Component: General Version:
Severity: trivial Keywords:
Cc:

Description

Two core files seem to have invalid line endings (\r).

~/data/wp$ ack -l "\r"
wp-includes/js/tinymce/plugins/wpfullscreen/css/wp-fullscreen.css
wp-config-sample.php

~/data/wp$ cat -vT wp-config-sample.php | head
<?php^M
/**^M
 * The base configurations of the WordPress.^M
 *^M
 * This file has the following configurations: MySQL settings, Table Prefix,^M
 * Secret Keys, WordPress Language, and ABSPATH. You can find more information^M
 * by visiting {@link http://codex.wordpress.org/Editing_wp-config.php Editing^M
 * wp-config.php} Codex page. You can get the MySQL settings from your web host.^M
 *^M
 * This file is used by the wp-config.php creation script during the^M

Change History (4)

  • Summary changed from Standardize line endings to Standardize line endings of wp-config-sample.php and wp-fullscreen.css

Add a more descriptive summary.

wp-fullscreen.css definitely should be LF line endings (perhaps svn:eol-style: native is missing on some new files) but wp-config-sample.php is deliberately CRLF.

More files (other than wp-config-sample.php) that probably need svn:eol-style:native.

/data/wp$ find . -type f | grep -v .svn | xargs svn propget svn:eol-style | grep CRLF
wp-includes/images/crystal/license.txt - CRLF
wp-includes/js/scriptaculous/MIT-LICENSE - CRLF
license.txt - CRLF
wp-config-sample.php - CRLF

I didn't think about that some Windows users use notepad to edit their wp-config.php file

Any user-readable files should be CRLF for maximum compatibility, ie. wp-config-sample.php and any license files. Not all license files are CRLF though i don't think, The only change here should be to make all the license files consistent.

Note: See TracTickets for help on using tickets.