[#436] Adapt rake task to also clean trailing whitespace

b72785b2 · Holger Just · 40ce66d0 · b72785b2
Commit b72785b2 authored May 30, 2011 by Holger Just
Show whitespace changes
Inline Side-by-side

Showing with 17 additions and 11 deletions

code.rake lib/tasks/code.rake +17 -11

No files found.
--- a/lib/tasks/code.rake
+++ b/lib/tasks/code.rake
@@ -14,14 +14,20 @@
 namespace :code do
  desc "Fix line endings of all source files"
  task :fix_line_endings do
-    unless `which fromdos`.present?
+    Dir.chdir(File.join(File.dirname(__FILE__), "../..")) do
-      raise "fromdos command not found"
+      files = Dir['**/**{.rb,.html.erb,.rhtml,.rjs,.rsb,.plain.erb,.rxml,.yml,.rake,.eml}']
-    end
+      files.reject!{ |f|
+        f.include?("vendor") ||
+        f.include?("lib/SVG") ||
+        f.include?("lib/faster_csv") ||
+        f.include?("lib/redcloth") ||
+        f.include?("lib/diff")
+      }
-    Dir['**/**{.rb,.html.erb,.rhtml,.rjs,.rsb,.plain.erb,.rxml,.yml,.rake,.eml}'].each do |file_name|
+      # handle files in chunks of 50 to avoid too long command lines
-      next if file_name.include?("vendor")
+      while (slice = files.slice!(0, 50)).present?
-      system("fromdos #{file_name}")
+        system('ruby', '-i', '-pe', 'gsub(/\s+$/,"\n")', *slice)
+      end
    end
  end
 end