Merge lp:~lifeless/ubuntu/maverick/pyrex/fixdepends into lp:ubuntu/maverick/pyrex

[Robert Collins]

Please be depending on things needed to use pyrex. For the meerkat. Not the market.

[James Westby (james-w)]

Hi,

I think the Recommends: python-all-dev is sufficient. Most people will
have it installed, but in some unusual cases.

What makes you think it is insufficient?

Thanks,

James

review: Needs Information

[Robert Collins (lifeless)]

On Wed, Jun 16, 2010 at 10:50 PM, James Westby
<email address hidden> wrote:
> Review: Needs Information
> Hi,
>
> I think the Recommends: python-all-dev is sufficient. Most people will
> have it installed, but in some unusual cases.
>
> What makes you think it is insufficient?

So there are two things;
* python-all-dev is about having *all supported* python headers
installed, python-dev is about the default python's headers and static
library, according to their packages.
 * is recommends enough?

For the former point, if someone has unusual needs, they can obviously
do whatever, so it's not really a big deal, for packages I'd be
build-deping on -all-dev, but this isn't for package builds.

For the latter point, pyrex really is quite useless without the python
headers installed. Yes you can in theory build a pyrex file to use
elsewhere, but I've never encountered someone doing that - and you'd
want to QA the thing thats generated anway.

[James Westby (james-w)]

On Wed, 16 Jun 2010 11:13:24 -0000, Robert Collins <email address hidden> wrote:
> For the latter point, pyrex really is quite useless without the python
> headers installed. Yes you can in theory build a pyrex file to use
> elsewhere, but I've never encountered someone doing that - and you'd
> want to QA the thing thats generated anway.

It seems to me that Recommends is entirely appropriate then.

Thanks,

James

Unmerged revisions

15. By Robert Collins on 2010-06-16

Depend on python-dev - it supplies the python headers needed to build the C
files pyrex generates.

=== modified file 'debian/changelog'
--- debian/changelog	2009-05-24 01:03:02 +0000
+++ debian/changelog	2010-06-16 04:08:26 +0000
@@ -1,3 +1,10 @@
+pyrex (0.9.8.5-1ubuntu3) lucid; urgency=low
+
+  * Depend on python-dev - it supplies the python headers needed to build the C
+    files pyrex generates. 
+
+ -- Robert Collins <[e-mail]>  Wed, 16 Jun 2010 16:04:16 +1200
+
 pyrex (0.9.8.5-1ubuntu2) karmic; urgency=low
 
   * Fix deprecation warning in Pyrex/Compiler/Errors.py (LP: #379754)

=== modified file 'debian/control'
--- debian/control	2009-05-04 21:48:56 +0000
+++ debian/control	2010-06-16 04:08:26 +0000
@@ -13,7 +13,7 @@
 
 Package: python-pyrex
 Architecture: all
-Depends: ${python:Depends}, ${misc:Depends}
+Depends: ${python:Depends}, ${misc:Depends}, python-dev
 Provides: ${python:Provides}
 Recommends: python-all-dev
 XB-Python-Version: all