moving argv.dup to top of parse_options to preserve return behavior

1 files changed, 2 insertions, 1 deletions

diff --git a/lib/mixlib/cli.rb b/lib/mixlib/cli.rb
index e5daf14..31eb3d9 100644
--- a/lib/mixlib/cli.rb
+++ b/lib/mixlib/cli.rb
@@ -118,6 +118,7 @@ module Mixlib
     # === Returns
     # argv<Array>:: Returns any un-parsed elements.
     def parse_options(argv=ARGV)
+      argv = argv.dup
       @opt_parser = OptionParser.new do |opts|  
         # Set the banner
         opts.banner = banner        
@@ -158,7 +159,7 @@ module Mixlib
           opts.send(*full_opt)
         end
       end
-      @opt_parser.parse!(argv.dup)
+      @opt_parser.parse!(argv)
       
       # Deal with any required values
       options.each do |opt_key, opt_value|