Bug 130200 - Pyrex has missing dependency
Summary: Pyrex has missing dependency
Alias: None
Product: Fedora
Classification: Fedora
Component: Pyrex
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: John (J5) Palmieri
QA Contact:
Depends On:
Blocks: FC3Target
TreeView+ depends on / blocked
Reported: 2004-08-17 23:36 UTC by Steve Grubb
Modified: 2013-03-13 04:46 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2004-08-18 16:53:53 UTC

Attachments (Terms of Use)
Patch that adds the Requires (398 bytes, patch)
2004-08-17 23:38 UTC, Steve Grubb
no flags Details | Diff

Description Steve Grubb 2004-08-17 23:36:37 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i586; en-US; rv:1.4.2)

Description of problem:
When Pyrex is used, it needs to have python-devel available to it.
Building dbus, for example, doesn't work unless there is a Requires:
python-devel against the package.

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. rpm -e python-devel
2. rpmbuild -bb dbus.spec

Actual Results:  Message saying headers weren't found.

Expected Results:  A message saying where the dbus rpm was written.

Additional info: I will attach a patch that addresses this.

Comment 1 Steve Grubb 2004-08-17 23:38:10 UTC
Created attachment 102814 [details]
Patch that adds the Requires

Please apply

Comment 2 John (J5) Palmieri 2004-08-18 14:28:09 UTC
This should be a BuildRequires for dbus but since Pyrex is a
development library I see no problem requiring python-devel for Pyrex.
 Patch accepted. Thanks.

Comment 3 Steve Grubb 2004-08-18 15:13:11 UTC
Can you think of any other package that uses Pyrex in its build
process? Does Pyrex complete its job for that package even without
python-devel being installed? If the answer is yes, a package does
indeed use Pyrex to build and does not need python-devel, then I would
agree that it is a BuildRequires against dbus.

However, if no package that uses Pyrex can be built without also
needing python-devel, then its far easier to put the Requires here and
just be done with it. Otherwise you have to add it to all packages
that use Pyrex.

Comment 4 John (J5) Palmieri 2004-08-18 16:53:53 UTC
No other packages in Fedora use Pyrex.  Anyway it is a moot point
since I have accepted the patch and feel there are good reasons for
having it in here. 

Built. Closing bug.

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