use STDOUT for errors, avoid unnecessary raise and exception

lodestone · Jul 13, 2011 · 0fdfe5d · 0fdfe5d
1 parent 711585f
commit 0fdfe5d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,5 +1,9 @@
 = Recent Changes in HCl
 
+== v0.2.4
+
+* use STDERR instead of STDOUT for error reporting
+
 == v0.2.3 Sun Aug 23 21:39:34 2009 -0700
 
 * Allow decimal time offset without a dot, closes #29.

diff --git a/lib/hcl/app.rb b/lib/hcl/app.rb
@@ -39,8 +39,6 @@ class App
     SETTINGS_FILE = "#{ENV['HOME']}/.hcl_settings"
     CONFIG_FILE = "#{ENV['HOME']}/.hcl_config"
 
-    class UnknownCommand < StandardError; end
-
     def initialize
       read_config
       read_settings
@@ -73,16 +71,17 @@ def run
               end
             end
           else
-            raise UnknownCommand, "unrecognized command `#{@command}'"
+            STDERR.puts "unrecognized command `#{@command}'"
+            exit 1
           end
         else
           show
         end
       rescue RuntimeError => e
-        puts "Error: #{e}"
+        STDERR.puts "Error: #{e}"
         exit 1
       rescue TimesheetResource::Failure => e
-        puts "Internal failure. #{e}"
+        STDERR.puts "Internal failure. #{e}"
         exit 1
       end
     end