zend-patterns / ZendServerPuppet

ZendServerPuppet
Apache License 2.0
6 stars 16 forks source link

redhat-lsb #7

Closed ubellavance closed 7 years ago

ubellavance commented 9 years ago

Thanks,

mkherlakian commented 9 years ago

@ubellavance Sorry for the delay. Actually I think we can. We don't use it with our Chef cookbook and it seems to work fine. Our standard installer script falls back to detecting it another way if detection through lsb_release fails

I'll look into it. Thanks!

ubellavance commented 7 years ago

This issue can be closed as #15 fixes it. Thanks,

ubellavance commented 7 years ago

I think I closed the issue too fast. #15 changes the name of the package, which is good because redhat-lsb installs a lot of unneeded packages, but it doesn't change the fact that I use redhat-lsb-core in my base profile. I'll try to submit a PR that would add a condition based on a new a variable named something like "lsb_package_install" that would be set to true by default. BTW #15 broke RHEL5 compatibility (redhat-lsb-core doesn't exist on RHEL5), but apparently, no one noticed.