Opened 8 years ago

Closed 8 years ago

#3191 closed task (fixed)

[patch] Missing svn:eol-style properties

Reported by: jjl@… Owned by: adrian
Component: Core (Other) Version: master
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

Some text files in SVN do not have the svn:eol-style property. That property should be set to 'native' for all of the files listed in the attached listing.

Attachments (1)

changed_files.txt (6.0 KB) - added by jjl@… 8 years ago.
List of files for which svn property svn:eol-style should be added

Download all attachments as: .zip

Change History (5)

Changed 8 years ago by jjl@…

List of files for which svn property svn:eol-style should be added

comment:1 Changed 8 years ago by jjl@…

  • Summary changed from Missing svn:eol-style properties to [patch] Missing svn:eol-style properties

This isn't strictly a patch of course, but it's the nearest thing that makes sense here since this is SVN property changes.

Note that some Python files (for example, most files in django/core) in the django SVN repository already have svn:eol-style set, so presumably at least somebody working on Django knows that this is the Right Thing.

Not having this property set on all files causes pain for people working on Windows (e.g. ports of "patch" typically crash when presented with mixed newlines -- which makes it awkward to contribute back other patches!)

comment:2 Changed 8 years ago by Gary Wilson <gary.wilson@…>

  • Type changed from defect to task

comment:3 Changed 8 years ago by adrian

For future reference, which command did you run to get a list of all files that didn't have svn:eol-style set?

comment:4 Changed 8 years ago by adrian

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

In [4265] - Fixed #3191 -- Set 'svn:eol-style native' on the files that didn't have it. Thanks, jjl@…

Note: See TracTickets for help on using tickets.
Back to Top