Ewoud Kohl van Wijngaarden

Results 1159 comments of Ewoud Kohl van Wijngaarden

Relevant previous effort: https://github.com/puppetlabs/puppetlabs-apache/pull/2163 > Tested with the following Puppet code: > > ``` > class { 'apache': > mpm_module => 'prefork', > } > class { 'apache::mod::php': } >...

@xorpaul I recognize the pain you have. Sadly PDK is in a horrible place regarding modern Ruby support. https://tickets.puppetlabs.com/browse/MODULES-11161 has been open for a long time. I always end up...

Somehow this doesn't link back to https://github.com/puppetlabs/puppetlabs-stdlib/pull/1259 but that's where this came up. In https://github.com/puppetlabs/puppetlabs-stdlib/pull/1259#issuecomment-1197839997 I stated pretty much the same as @b4ldr that I do feel stdlib is special...

Can you come up with a patch? It should be here: https://github.com/puppetlabs/puppetlabs-apache/blob/3839c93572110e9a6ed61edf0fdc4a32e0770137/manifests/params.pp#L365-L370 Right now there isn't proper CI support for Ubuntu 22.04 so we can't claim full support but having...

Related, there's #2261 which may have the same cause.

> I think the Red Hat family has dropped support for PHP modules since RHEL/CentOS 8.x so I'm not sure this is related It looks like you're right for at...

On CentOS Stream 8 you can try (on a fresh system) `dnf module enable php:8.0` to get PHP 8. If you already have a module enabled, `dnf module reset php...

Could you come up with a patch to change it to `String[1]` instead? Unless you can come up with a better pattern of course.

The EL 8 test failures can be ignored (they also fail without this PR), but I do wonder if you can come up with a test that shows the new...

Looks like the tests fail on this. I also wonder if this should be a type alias.