* [sup-devel] [PATCH] fix up options for sup-sync-back, tweak-labels
@ 2010-01-05 17:20 Anthony Martinez
2010-01-05 21:58 ` William Morgan
0 siblings, 1 reply; 2+ messages in thread
From: Anthony Martinez @ 2010-01-05 17:20 UTC (permalink / raw)
To: sup-devel
These two violate the convention of the other scripts inside bin/ by
using a local variable for their Trollop results.
---
bin/sup-sync-back | 2 +-
bin/sup-tweak-labels | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/bin/sup-sync-back b/bin/sup-sync-back
index 19e6230..40c1e1c 100755
--- a/bin/sup-sync-back
+++ b/bin/sup-sync-back
@@ -66,7 +66,7 @@ EOS
end
Redwood::start
-index = Redwood::Index.init $opts[:index]
+index = Redwood::Index.init opts[:index]
index.lock_interactively or exit
deleted_fp, spam_fp = nil
diff --git a/bin/sup-tweak-labels b/bin/sup-tweak-labels
index f603502..bf36a32 100755
--- a/bin/sup-tweak-labels
+++ b/bin/sup-tweak-labels
@@ -61,7 +61,7 @@ remove_labels = opts[:remove].to_set_of_symbols ","
Trollop::die "nothing to do: no labels to add or remove" if add_labels.empty? && remove_labels.empty?
Redwood::start
-index = Redwood::Index.init $opts[:index]
+index = Redwood::Index.init opts[:index]
index.lock_interactively or exit
begin
index.load
--
1.6.5
_______________________________________________
Sup-devel mailing list
Sup-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-01-05 21:58 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-01-05 17:20 [sup-devel] [PATCH] fix up options for sup-sync-back, tweak-labels Anthony Martinez
2010-01-05 21:58 ` William Morgan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox