Bug 2378931

Summary:	miniz-devel is missing miniz.pc file
Product:	[Fedora] Fedora	Reporter:	Dominik 'Rathann' Mierzejewski <dominik>
Component:	miniz	Assignee:	Paul Howarth <paul>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	paul, ppisar
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	miniz-3.0.2-9.fc43	Doc Type:	---
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2025-08-03 00:13:14 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 Dominik 'Rathann' Mierzejewski 2025-07-08 23:58:44 UTC

Description of problem:
miniz.pc file is not included in miniz-devel subpackage (https://src.fedoraproject.org/rpms/miniz/blob/rawhide/f/miniz.spec#_129) despite upstream providing one (https://github.com/richgel999/miniz/blob/master/miniz.pc.in).

Version-Release number of selected component (if applicable):
miniz-3.0.2-7.fc42

How reproducible:
Always.

Steps to Reproduce:
1. dnf install miniz-devel
2. pkg-config --path miniz

Actual results:
Package miniz was not found in the pkg-config search path.
Perhaps you should add the directory containing `miniz.pc'
to the PKG_CONFIG_PATH environment variable
Package 'miniz' not found

Expected results:
/usr/lib64/pkgconfig/miniz.pc

Additional info:
It looks like upstream has a working CMake and meson build systems now, though meson-based one doesn't install the pkgconfig file.

Comment 1 Paul Howarth 2025-07-11 14:09:55 UTC

Hi Dominik, are you actively using miniz? If so, perhaps you'd like to co-maintain the package? My only interest in it is as a dependency of perl-Sereal.

Comment 2 Dominik 'Rathann' Mierzejewski 2025-07-11 18:40:19 UTC

(In reply to Paul Howarth from comment #1)
> Hi Dominik, are you actively using miniz? If so, perhaps you'd like to
> co-maintain the package? My only interest in it is as a dependency of
> perl-Sereal.

I'm in the same position. It's a bundled dependency of one of my packages,
ags. So I don't really need it, just want it unbundled to follow the rules.
But sure, I'd be happy to co-maintain. You can add me (rathann) to the package.

Looks like there are two more consumers of miniz: Pencil2D and tinyexr.

Comment 3 Paul Howarth 2025-07-12 13:02:31 UTC

(In reply to Dominik 'Rathann' Mierzejewski from comment #2)
> (In reply to Paul Howarth from comment #1)
> > Hi Dominik, are you actively using miniz? If so, perhaps you'd like to
> > co-maintain the package? My only interest in it is as a dependency of
> > perl-Sereal.
> 
> I'm in the same position. It's a bundled dependency of one of my packages,
> ags. So I don't really need it, just want it unbundled to follow the rules.
> But sure, I'd be happy to co-maintain. You can add me (rathann) to the
> package.

Done, thanks.

Comment 4 Dominik 'Rathann' Mierzejewski 2025-07-20 11:02:53 UTC

PR submitted. Ok to merge?

Comment 5 Paul Howarth 2025-07-22 07:39:36 UTC

Go ahead and merge. I'm not a fan of autosetup personally as I find it makes managing downstream patches harder, but there are no patches here.

Comment 6 Dominik 'Rathann' Mierzejewski 2025-07-22 08:51:04 UTC

(In reply to Paul Howarth from comment #5)
> Go ahead and merge.

Thanks.

> I'm not a fan of autosetup personally as I find it makes
> managing downstream patches harder, but there are no patches here.

No problem, I will leave %setup in place.

Comment 7 Fedora Update System 2025-08-03 00:08:39 UTC

FEDORA-2025-23762e1e5f (ags-3.6.2.12-3.fc43, miniz-3.0.2-9.fc43, and 3 more) has been submitted as an update to Fedora 43.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-23762e1e5f

Comment 8 Fedora Update System 2025-08-03 00:13:14 UTC

FEDORA-2025-23762e1e5f (ags-3.6.2.12-3.fc43, miniz-3.0.2-9.fc43, and 3 more) has been pushed to the Fedora 43 stable repository.
If problem still persists, please make note of it in this bug report.