Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #4634 (new defect)

Opened 3 years ago

[PATCH] Substitute path separator in svn status paths

Reported by: drodrigueztroitino@yahoo.es Assigned to: David
Priority: normal Milestone:
Component: Railties Version: 1.1.0
Severity: normal Keywords:
Cc:

Description

The --svn (or -c) option from the generators fails in Windows when using destroy generator.

"svn status" return in Windows a list of Windows paths (with "\" separating path directories) so when the hash get compared later with the Unix paths the script tries to use "svn rm" when it should have used "svn revert", making the script fail because svn says that the files have been locally modified.

The patch substitutes each File::SEPARATOR and File::ALT_SEPARATOR in the path returned from "svn status" with "/" so the later comparisons don't fail anymore.

I've tested that the patch works in Windows, and tested that the regexp works in Windows, Mac OS X and Linux.

Attachments

subs-path-separator.patch (0.9 kB) - added by drodrigueztroitino@yahoo.es on 04/07/06 00:39:05.
Substitute path separators patch

Change History

04/07/06 00:39:05 changed by drodrigueztroitino@yahoo.es

  • attachment subs-path-separator.patch added.

Substitute path separators patch