Bug 1949318

Summary:	php has mod_php still enabled and built even though it should not be present
Product:	Red Hat Enterprise Linux 9	Reporter:	Neal Gompa <ngompa13>
Component:	php	Assignee:	Remi Collet <rcollet>
Status:	CLOSED CURRENTRELEASE	QA Contact:	icesalov
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	CentOS Stream	CC:	bnater, carl, icesalov, jorton
Target Milestone:	beta
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2021-06-22 13:09:39 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Neal Gompa 2021-04-14 01:37:53 UTC

Description of problem:
I noticed that php in EL9 has the "php" package built and present with mod_php[0]. This is not supposed to be there, as mod_php was removed in Fedora 33[1].

This is happening because the conditionals are set wrong for non-Fedora. I doubt you want to support mod_php through 2032 when it's not even enabled for Fedora, so this should probably be fixed.

That said, I'm a bit surprised this wasn't caught during the ELN bringup, since this was one of the purposes of ELN.

[0]: https://gitlab.com/redhat/centos-stream/rpms/php/-/blob/4f95d29bee28e32eafa78a7209ab5d4f6d1370ad/php.spec#L47
[1]: https://fedoraproject.org/wiki/Changes/drop_mod_php

Comment 1 Remi Collet 2021-04-14 13:03:30 UTC

Already disabled in 

Fedora 35 (PHP 8.0) https://src.fedoraproject.org/rpms/php/c/67b2f663aa650feb3aa3676d3235227d484c5d1d?branch=rawhide
Fedora 34 (PHP 7.4) https://src.fedoraproject.org/rpms/php/c/41e02955a36c23872575831c32c62c33de6eb825?branch=f34

So should be pulled on next sync

Comment 3 Neal Gompa 2021-04-14 23:18:14 UTC

(In reply to Remi Collet from comment #1)
> Already disabled in 
> 
> Fedora 35 (PHP 8.0)
> https://src.fedoraproject.org/rpms/php/c/
> 67b2f663aa650feb3aa3676d3235227d484c5d1d?branch=rawhide
> Fedora 34 (PHP 7.4)
> https://src.fedoraproject.org/rpms/php/c/
> 41e02955a36c23872575831c32c62c33de6eb825?branch=f34
> 
> So should be pulled on next sync

That was a month ago, so I would have expected it to sync by now. Either the sync broke (which has happened before) or syncing was switched off after Fedora 34 Beta was released.

Comment 4 Remi Collet 2021-04-16 14:05:05 UTC

WIP on https://gitlab.com/remirh/centos_rpms_php/-/commits/bz1949319

Comment 5 Remi Collet 2021-05-05 11:33:35 UTC

mod_php have been dropped during 8.0 rebase

See bug #1949319

Comment 6 Joe Orton 2021-06-22 13:09:39 UTC

This is resolved in current builds, thanks for the report.