Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 308503 - dev-python/dbus-python unnecessarily depends on pyrex
Summary: dev-python/dbus-python unnecessarily depends on pyrex
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Doug Goldstein (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-08 19:15 UTC by m_zwart
Modified: 2010-03-31 11:54 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description m_zwart 2010-03-08 19:15:14 UTC
dbus-python (0.83.0-r1 and 0.81.1) currently both have an RDEPEND on pyrex. There are three problems with this dependency:

- pyrex is a preprocessor generating c code from .pyx code at build time. It should almost never be used as an RDEPEND: the generated c code does not depend on pyrex anymore, just on python. If necessary at all it is just a DEPEND.

- Upstream usually includes the c source generated by a known-compatible version of pyrex in their dist tarball. So even an RDEPEND on pyrex is usually only necessary if the .pyx source is patched from the ebuild.

- The tarball does not actually contain any .pyx source anymore: this was removed in 0.80.0 (24 January 2007). The only remaining references to pyrex are in the NEWS and ChangeLog file.

Based on the above I believe this dependency could very safely be removed :)
Comment 1 Gilles Dartiguelongue (RETIRED) gentoo-dev 2010-03-08 21:35:41 UTC
> - The tarball does not actually contain any .pyx source anymore: this was
> removed in 0.80.0 (24 January 2007). The only remaining references to pyrex are
> in the NEWS and ChangeLog file.

this argument alone was just fine :)
Comment 2 Gilles Dartiguelongue (RETIRED) gentoo-dev 2010-03-08 22:44:09 UTC
+  08 Mar 2010; Gilles Dartiguelongue <eva@gentoo.org>
+  dbus-python-0.83.0-r1.ebuild, dbus-python-0.83.1.ebuild:
+  pyrex is unnecessary, bug #308503.
+
Thanks for reporting.
Comment 3 Manuel Mommertz 2010-03-31 11:13:30 UTC
Please consider increasing revision number if you make any changes.

I'm just effected by this change as I maintain a server that provides binary packages. The binpkg for pyrex was removed as nothing depends on it but the binpkg for dbus-python wasn't rebuild as the revision didn't change. Now I wanted to install this binpkg but can't as there is this missing dependency.
Comment 4 Gilles Dartiguelongue (RETIRED) gentoo-dev 2010-03-31 11:54:32 UTC
That's unfortunate but dependency changes like this don't warrant a version bump.