Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Augeas lenses #30

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

lazyfrosch
Copy link

The lenses weren't working on an Ubuntu 14.04, so I pulled the latest versions
from Augeas' GitHub.

I'm not sure if that might cause problems on other OSes.

@mjhas
Copy link
Owner

mjhas commented Apr 26, 2015

Hey, well apparently it does ;) I think the better solution would be to not install them for Ubuntu 14.04 and Debian 8.0 fordwards. The configuration lies in manifests/config/augeas.pp may be you want to give it a try?

@lazyfrosch
Copy link
Author

AFAIR this didn't work, because the lenses shipped with 14.04 where too old, but I have to verify that.

@lazyfrosch
Copy link
Author

Well, long time no see, it seems to work with the shipped lenses in Ubuntu 14.04 (augeas 1.2.0).

So should we only deploy them on old versions?

I'll change that PR to a conditional!

 NOTE: we should consider removing this in the future, since most distros are on Augeas > 1 now, and even PuppetLabs ships those packages.
@lazyfrosch
Copy link
Author

Here you go. I think travis still has problems with it.

@hdeadman
Copy link
Contributor

Putting these old augeas files out is breaking Centos 7 augeas too (because yum.aug relies on some features in build.aug that aren't in this old version). This PR looks like it should fix the issue.

@timogoebel
Copy link

This breaks augeas on Debian Jessie as well. @mjhas: What do you think? Can this get merged?

@raoulbhatia
Copy link
Contributor

Works for me on Ubuntu 14.04 LTS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants