On Monday, July 16, 2012 05:29:24 PM mzatko(a)redhat.com wrote:
From: Maros Zatko <mzatko(a)redhat.com>
https://bugzilla.redhat.com/show_bug.cgi?id=818750
"aeolus-image list --images --environments=<env>" can handle spaces in
<env> now
---
lib/aeolus_cli/command/list_command.rb | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/aeolus_cli/command/list_command.rb
b/lib/aeolus_cli/command/list_command.rb
index 479b961..a45d5a5 100644
--- a/lib/aeolus_cli/command/list_command.rb
+++ b/lib/aeolus_cli/command/list_command.rb
@@ -29,7 +29,7 @@ module Aeolus
headers[:os_version] = "OS Version"
headers[:arch] = "Arch"
headers[:description] = "Description"
- collection = @options[:environment].nil? ? Aeolus::CLI::Image.all :
Aeolus::CLI::Image.find(:all, :from => Aeolus::CLI::Base.site.path +
"/environments/" + @options[:environment] + "/images.xml")
+ collection = @options[:environment].nil? ? Aeolus::CLI::Image.all :
Aeolus::CLI::Image.find(:all, :from => URI::encode(Aeolus::CLI::Base.site.path +
"/environments/" + @options[:environment] + "/images.xml"))
print_collection(collection, headers)
quit(0)
rescue => e
Works for me but I see no test.
So COND ACK - add test to avoid regression in the future and then push.
--
With regards
Petr Blaho