[PATCH aeolus-configure] BZ# 768123: Configure should not write /root/.aeolus-cli
John Eckersberg
jeckersb at redhat.com
Fri Dec 16 19:07:16 UTC 2011
On Thu, 15 Dec 2011 15:11:50 -0500, Jason Guiditta <jguiditt at redhat.com> wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=768123
>
> This was written incorrectly to use a specific version of aeolus-cli,
> as well as not being needed. If there is a need in future to have
> configure do something with this file, it should be done in a more
> robust way.
> ---
> recipes/aeolus/manifests/image-factory.pp | 3 ---
> 1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/recipes/aeolus/manifests/image-factory.pp b/recipes/aeolus/manifests/image-factory.pp
> index d6e37ec..79cfd1b 100644
> --- a/recipes/aeolus/manifests/image-factory.pp
> +++ b/recipes/aeolus/manifests/image-factory.pp
> @@ -19,9 +19,6 @@ class aeolus::image-factory inherits aeolus {
> # image factory client
> package { 'rubygem-aeolus-image': ensure => 'installed' }
> package { 'rubygem-aeolus-cli': ensure => 'installed' }
> - file{"/root/.aeolus-cli":
> - source => "/usr/lib/ruby/gems/1.8/gems/aeolus-cli-0.2.0/examples/aeolus-cli",
> - require => Package['rubygem-aeolus-image', 'rubygem-aeolus-cli'] }
>
> # image factory services
> package { 'libvirt':
> --
> 1.7.7.4
>
ACK
More information about the aeolus-devel
mailing list