Skip to content
This repository has been archived by the owner on Oct 11, 2018. It is now read-only.

Version switch #165

Open
wants to merge 6 commits into
base: php7.0
Choose a base branch
from
Open

Version switch #165

wants to merge 6 commits into from

Conversation

hp197
Copy link

@hp197 hp197 commented Feb 26, 2017

Changed most (if not all) hardcoded php version numbers into case statements.
This way the module will be compatible with php5 and other distro's while keeping
compatible with the new php7.0 and possible future upgrades.

** Tested package names & version numbers only for Debian Jessie and Stretch.

Henry Paulissen added 6 commits February 25, 2017 22:34
All tested with Debian Jessie and Stretch.
Most likely broke for RedHat based systems (but this module was
probably broke for RedHat systems already - Feel free to extend).

Signed-off-by: Henry Paulissen <[email protected]>
Stretch seems to use php7.0- prefix for extensions shipped
with php7.0 and php- prefix for other php extensions. I
updated the code to reflect this.

Cross checked Jessie, every package on there is prefixed with php5-.

Signed-off-by: Henry Paulissen <[email protected]>
In Jessie the php-apc is a virtual package for php5-apcu.
In Stretch this package is named php-apcu and as last, for
Wheezy we want to install the real php-apc.

So I added more conditionals in the apc package.

Signed-off-by: Henry Paulissen <[email protected]>
All "7.0" in puppet code should be gone now.

Signed-off-by: Henry Paulissen <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant