bin/sup-add in sup-0.22.1 vs bin/sup-add in sup-0.23

- old
+ new

@@ -1,15 +1,15 @@ #!/usr/bin/env ruby $:.unshift File.join(File.dirname(__FILE__), *%w[.. lib]) require 'uri' -require 'highline/import' -require 'trollop' +require 'optimist' require "sup" +require 'sup/util/axe' -$opts = Trollop::options do +$opts = Optimist::options do version "sup-add (sup #{Redwood::VERSION})" banner <<EOS Adds a source to the Sup source list. Usage: @@ -33,11 +33,11 @@ opt :labels, "A comma-separated set of labels to apply to all messages from this source", :type => String opt :force_new, "Create a new account for this source, even if one already exists." opt :force_account, "Reuse previously defined account user@hostname.", :type => String end -Trollop::die "require one or more sources" if ARGV.empty? +Optimist::die "require one or more sources" if ARGV.empty? ## for sources that require login information, prompt the user for ## that. also provide a list of previously-defined login info to ## choose from, if any. def get_login_info uri, sources @@ -51,34 +51,34 @@ username, password = nil, nil unless accounts.empty? || $opts[:force_new] if $opts[:force_account] host, username, password = accounts.find { |h, u, p| $opts[:force_account] == "#{u}@#{h}" } unless username && password - say "No previous account #{$opts[:force_account].inspect} found." + @cli.say "No previous account #{$opts[:force_account].inspect} found." end else - say "Would you like to use the same account as for a previous source for #{uri}?" - choose do |menu| + @cli.say "Would you like to use the same account as for a previous source for #{uri}?" + @cli.choose do |menu| accounts.each do |host, olduser, oldpw| menu.choice("Use the account info for #{olduser}@#{host}") { username, password = olduser, oldpw } end menu.choice("Use a new account") { } menu.prompt = "Account selection? " end end end unless username && password - username = ask("Username for #{uri.host}: "); - password = ask("Password for #{uri.host}: ") { |q| q.echo = false } + username = @cli.ask("Username for #{uri.host}: "); + password = @cli.ask("Password for #{uri.host}: ") { |q| q.echo = false } puts # why? end [username, password] end -$terminal.wrap_at = :auto +@cli.wrap_at = :auto Redwood::start index = Redwood::Index.init index.load index.lock_interactively or exit @@ -88,11 +88,11 @@ ARGV.each do |uri| labels = $opts[:labels] ? $opts[:labels].split(/\s*,\s*/).uniq : [] if !$opts[:force_new] && Redwood::SourceManager.source_for(uri) - say "Already know about #{uri}; skipping." + @cli.say "Already know about #{uri}; skipping." next end parsed_uri = URI(URI.escape(uri)) @@ -101,14 +101,14 @@ when "maildir" Redwood::Maildir.new uri, !$opts[:unusual], $opts[:archive], $opts[:sync_back], nil, labels when "mbox" Redwood::MBox.new uri, !$opts[:unusual], $opts[:archive], nil, labels when nil - Trollop::die "Sources must be specified with an URI" + Optimist::die "Sources must be specified with an URI" else - Trollop::die "Unknown source type #{parsed_uri.scheme.inspect}" + Optimist::die "Unknown source type #{parsed_uri.scheme.inspect}" end - say "Adding #{source}..." + @cli.say "Adding #{source}..." Redwood::SourceManager.add_source source end ensure index.save index.unlock