search

voxpupuli / puppet-augeas

Helper for using augeas with puppet
Apache License 2.0
44 stars 82 forks source link

Remove ordering that puts Augeas packages before Augeas resources #58

Closed treydock closed 3 years ago

treydock commented 8 years ago

Remove ordering that puts Augeas packages before Augeas resources

ruby-augeas is a dependency of puppet, so it will already be installed

Relates to #56

This change is Reviewable

raphink commented 8 years ago

With AIO/PE yes, but there's still lots of people using Puppet 3 (hopefully not 2, but you never know…)

treydock commented 7 years ago

@raphink I think Puppet 3 also has the dependency, at least I never ran into a problem with this PR in a Puppet 3.8 environment.

djjudas21 commented 7 years ago

I'm running into this problem on Puppet 3.8.7 too, and can confirm that this PR fixes it for me

djjudas21 commented 7 years ago

Nudge :smile_cat:

raphink commented 7 years ago

Puppet 3 is now end-of-life since the end of December. @treydock @djjudas21 do we still need this?

djjudas21 commented 7 years ago

I migrated to Puppet 4 yesterday, so I no longer need this. Thanks!

raphink commented 7 years ago

@djjudas21 congrats on your migration!

djjudas21 commented 7 years ago

Well, it's not working properly yet :stuck_out_tongue_closed_eyes:

treydock commented 7 years ago

@raphink I'm on Puppet 3 with future parser and this is still an issue. I recently used exported augeas resources and had them incorrectly collected by this module, which may be a bug in Puppet.

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

treydock commented 3 years ago

This is no longer needed.