Bug 193674 - Lua does no require pkgconfig
Summary: Lua does no require pkgconfig
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: lua
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michael J Knox
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-BuildReq
TreeView+ depends on / blocked
 
Reported: 2006-05-31 15:55 UTC by Matthias Saou
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-31 20:02:16 UTC


Attachments (Terms of Use)

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!


Note You need to log in before you can comment on or make changes to this bug.