Bug 193674

Summary:	Lua does no require pkgconfig
Product:	[Fedora] Fedora	Reporter:	Matthias Saou <matthias>
Component:	lua	Assignee:	Michael J Knox <michael>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	extras-qa
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-05-31 20:02:16 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	193444

Description Matthias Saou 2006-05-31 15:55:12 UTC

The current lua package now uses the pkgconfig .pc file instead of the no longer
included lua-config script. A requirement on pkgconfig is now needed, but it
might then be a good time to split lua and lua-devel into two since it would
currently mean adding a development type of requirement to a main package.

I've made this block the FE-BuildReq bug since I discovered this problem when
looking at the lighttpd build logs, which indicated a failure because of
pkgconfig not being installed when trying to enable lua support.

Comment 1 Michael J Knox 2006-05-31 19:33:56 UTC

Could you provide some more details? maybe a build log etc?

If you go here:

http://linux.dell.com/files/fedora/FixBuildRequires/mock-results-extras/i386/lua-5.1-3.fc6.src.rpm/result/

lua builds without the need for pkg-config. At no time during the build is
pkg-config called.

Comment 2 Michael J Knox 2006-05-31 19:43:05 UTC

Cancel that. I just clicked to what you meant. 

Will push a rebuild tonight.

Comment 3 Michael J Knox 2006-05-31 20:02:16 UTC

Fixed in lua-5.1-4.

Comment 4 Matthias Saou 2006-06-01 00:12:09 UTC

Thanks.
But what about the idea of splitting off a real -devel sub-package? When there
were no dependencies and only a static library, it didn't make much sense, but
now I really think it does.

Comment 5 Michael J Knox 2006-06-06 02:34:03 UTC

OK, I have split it out and it should be in the repos in the next push. Thanks!